Do not deadlock in drbd_disconnect() [bugz 258]
authorPhilipp Reisner <philipp.reisner@linbit.com>
Fri, 23 Oct 2009 11:57:45 +0000 (13:57 +0200)
committerPhilipp Reisner <philipp.reisner@linbit.com>
Wed, 4 Nov 2009 14:21:03 +0000 (15:21 +0100)
When there are many blocks on the fly (ua), and the AL gets into "starving"
mode (random IO, scattered all over the device), and the connections gets
interrupted, the receiver thread deadlocks in the drbd_disconnect() code path.

Affected are only nodes in Primary role.

The bug triggers most likely on system that mirror over "long distances"

Regression introduced shortly before 8.3.3
with git commit 31e0f1250f174ac1ee317f360943a0159e19edc8

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

index d9312b45393f08b398bd3f869dc983d804a04789..9bbc509443e5bfae8b2953f1815f91b365bae9b4 100644 (file)
@@ -3619,10 +3619,6 @@ static void drbd_disconnect(struct drbd_conf *mdev)
        set_bit(STOP_SYNC_TIMER, &mdev->flags);
        resync_timer_fn((unsigned long)mdev);
 
-       /* so we can be sure that all remote or resync reads
-        * made it at least to net_ee */
-       wait_event(mdev->misc_wait, !atomic_read(&mdev->local_cnt));
-
        /* wait for all w_e_end_data_req, w_e_end_rsdata_req, w_send_barrier,
         * w_make_resync_request etc. which may still be on the worker queue
         * to be "canceled" */