drbd: fix race between drbdadm invalidate/verify and finishing resync
authorLars Ellenberg <lars.ellenberg@linbit.com>
Wed, 25 Apr 2012 14:27:35 +0000 (16:27 +0200)
committerPhilipp Reisner <philipp.reisner@linbit.com>
Wed, 9 May 2012 13:16:59 +0000 (15:16 +0200)
commita574daf5d722f4ca8cc18509f30b804c4d519962
tree83ae95ce9a57d605ce2a308ceb2a026e9f2ca9b2
parentba280c092e6eca8a70c502e4510061535fdce382
drbd: fix race between drbdadm invalidate/verify and finishing resync

When a resync or online verify is finished or aborted,
drbd does a bulk write-out of changed bitmap pages.

If *in that very moment* a new verify or resync is triggered,
this can race:
 ASSERT( !test_bit(BITMAP_IO, &mdev->flags) ) in drbd_main.c
 FIXME going to queue 'set_n_write from StartingSync' but 'write from resync_finished' still pending?
and similar.

This can be observed with e.g. tight invalidate loops in test scripts,
and probably has no real-life implication.

Still, that race can be solved by first quiescen the device,
before starting a new resync or verify.

Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
drivers/block/drbd/drbd_nl.c