[PATCH 2/3] MEMSTICK: fix hangs on unexpected device removal in mspro_blk

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Maxim Levitsky
Date: Thursday, August 5, 2010 - 8:23 am

mspro_block_remove is called from detect thread that first calls
the mspro_block_stop, which stops the request queue.
If we call del_gendisk with queue stopped we get a deadlock.


Signed-off-by: Maxim Levitsky <maximlevitsky@gmail.com>
---
 drivers/memstick/core/mspro_block.c |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/memstick/core/mspro_block.c b/drivers/memstick/core/mspro_block.c
index 369313f..300ec15 100644
--- a/drivers/memstick/core/mspro_block.c
+++ b/drivers/memstick/core/mspro_block.c
@@ -1331,13 +1331,14 @@ static void mspro_block_remove(struct memstick_dev *card)
 	struct mspro_block_data *msb = memstick_get_drvdata(card);
 	unsigned long flags;
 
-	del_gendisk(msb->disk);
-	dev_dbg(&card->dev, "mspro block remove\n");
 	spin_lock_irqsave(&msb->q_lock, flags);
 	msb->eject = 1;
 	blk_start_queue(msb->queue);
 	spin_unlock_irqrestore(&msb->q_lock, flags);
 
+	del_gendisk(msb->disk);
+	dev_dbg(&card->dev, "mspro block remove\n");
+
 	blk_cleanup_queue(msb->queue);
 	msb->queue = NULL;
 
-- 
1.7.0.4

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/2 V3] Driver for Ricoh cardreader + few fixes, Maxim Levitsky, (Thu Aug 5, 8:23 am)
[PATCH 1/3] memstick: init sysfs attributes, Maxim Levitsky, (Thu Aug 5, 8:23 am)
[PATCH 2/3] MEMSTICK: fix hangs on unexpected device remov ..., Maxim Levitsky, (Thu Aug 5, 8:23 am)