drbd: fix schedule in atomic
authorLars Ellenberg <lars.ellenberg@linbit.com>
Mon, 2 May 2011 09:51:31 +0000 (11:51 +0200)
committerPhilipp Reisner <philipp.reisner@linbit.com>
Thu, 8 Nov 2012 15:53:01 +0000 (16:53 +0100)
commit009ba89db5ae836949009f97a00abb96feba69f4
tree34218da1ee3796a5a77603bc162eaad22cce3469
parent992d6e91d3654c11c2e4d8d5933ffbf82a0440f0
drbd: fix schedule in atomic

An administrative detach used to request a state change directly to D_DISKLESS,
first suspending IO to avoid the last put_ldev() occuring from an endio handler,
potentially in irq context.

This is not enough on the receiving side (typically secondary), we may miss
some peer_req on the way to local disk, which then may do the last put_ldev()
from their drbd_peer_request_endio().

This patch makes the detach always go through the intermediate D_FAILED state.
We may consider to rename it D_DETACHING.

Alternative approach would be to create yet an other work item to be scheduled
on the worker, do the destructor work from there, and get the timing right.

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