md/raid5: flush out all pending requests before proceeding with reshape.
authorNeilBrown <neilb@suse.de>
Tue, 27 Aug 2013 05:57:47 +0000 (15:57 +1000)
committerNeilBrown <neilb@suse.de>
Wed, 28 Aug 2013 06:58:44 +0000 (16:58 +1000)
Some requests - particularly 'discard' and 'read' are handled
differently depending on whether a reshape is active or not.

It is harmless to assume reshape is active if it isn't but wrong
to act as though reshape is not active when it is.

So when we start reshape - after making clear to all requests that
reshape has started - use mddev_suspend/mddev_resume to flush out all
requests.  This will ensure that no requests will be assuming the
absence of reshape once it really starts.

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

index 4263df11d5976413d2bf7d5ec173d40dd09674d6..663a8e58d4390b223a13a84645bc98f795c63f07 100644 (file)
@@ -6279,6 +6279,13 @@ static int raid5_start_reshape(struct mddev *mddev)
        write_seqcount_end(&conf->gen_lock);
        spin_unlock_irq(&conf->device_lock);
 
+       /* Now make sure any requests that proceeded on the assumption
+        * the reshape wasn't running - like Discard or Read - have
+        * completed.
+        */
+       mddev_suspend(mddev);
+       mddev_resume(mddev);
+
        /* Add some new drives, as many as will fit.
         * We know there are enough to make the newly sized array work.
         * Don't add devices if we are reducing the number of