md: don't unregister the thread in mddev_suspend
authorNeilBrown <neilb@suse.de>
Tue, 6 Apr 2010 04:23:02 +0000 (14:23 +1000)
committerNeilBrown <neilb@suse.de>
Tue, 18 May 2010 05:27:56 +0000 (15:27 +1000)
This is
 - unnecessary because mddev_suspend is always followed by a call to
   ->stop, and each ->stop unregisters the thread, and
 - a problem as it makes it awkwards to suspend and then resume a
   device as we will want later.

Signed-off-by: NeilBrown <neilb@suse.de>
drivers/md/md.c

index d3579fc9efedd3afd4392fc75b7d4f5fa39d86b4..af0780ae56b572a8a49c882ca112eabb62f5ffd7 100644 (file)
@@ -254,6 +254,12 @@ static int md_make_request(struct request_queue *q, struct bio *bio)
        return rv;
 }
 
+/* mddev_suspend makes sure no new requests are submitted
+ * to the device, and that any requests that have been submitted
+ * are completely handled.
+ * Once ->stop is called and completes, the module will be completely
+ * unused.
+ */
 static void mddev_suspend(mddev_t *mddev)
 {
        BUG_ON(mddev->suspended);
@@ -261,13 +267,6 @@ static void mddev_suspend(mddev_t *mddev)
        synchronize_rcu();
        wait_event(mddev->sb_wait, atomic_read(&mddev->active_io) == 0);
        mddev->pers->quiesce(mddev, 1);
-       md_unregister_thread(mddev->thread);
-       mddev->thread = NULL;
-       /* we now know that no code is executing in the personality module,
-        * except possibly the tail end of a ->bi_end_io function, but that
-        * is certain to complete before the module has a chance to get
-        * unloaded
-        */
 }
 
 static void mddev_resume(mddev_t *mddev)