10/19

From: Joe Thornber (joe@fib011235813.fsnet.co.uk)
Date: Mon Dec 16 2002 - 05:12:36 EST


dm_suspend(): Stop holding the read lock around the while loop that
waits for pending io to complete.
--- diff/drivers/md/dm.c 2002-12-16 09:41:03.000000000 +0000
+++ source/drivers/md/dm.c 2002-12-16 09:41:08.000000000 +0000
@@ -715,15 +715,13 @@
         }
 
         set_bit(DMF_BLOCK_IO, &md->flags);
+ add_wait_queue(&md->wait, &wait);
         up_write(&md->lock);
 
         /*
          * Then we wait for the already mapped ios to
          * complete.
          */
- down_read(&md->lock);
-
- add_wait_queue(&md->wait, &wait);
         while (1) {
                 set_current_state(TASK_INTERRUPTIBLE);
 
@@ -734,11 +732,11 @@
         }
 
         current->state = TASK_RUNNING;
- remove_wait_queue(&md->wait, &wait);
- up_read(&md->lock);
 
- /* set_bit is atomic */
+ down_write(&md->lock);
+ remove_wait_queue(&md->wait, &wait);
         set_bit(DMF_SUSPENDED, &md->flags);
+ up_write(&md->lock);
 
         return 0;
 }
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Dec 23 2002 - 22:00:13 EST