aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/block/drbd/drbd_int.h
diff options
context:
space:
mode:
authorPhilipp Reisner <philipp.reisner@linbit.com>2010-06-23 05:20:05 -0400
committerPhilipp Reisner <philipp.reisner@linbit.com>2010-10-14 09:04:46 -0400
commit84dfb9f564208a0331131d1ab922382c7d61a553 (patch)
tree1506d6c318afa4a94b13bab6af983031eac5e3cd /drivers/block/drbd/drbd_int.h
parent65d922c33ebd359db25d5846929b2eafc4238fcc (diff)
drbd: Fixed a deadlock, probably only affected UP machines
After disconnect (most likely mdev->net_cnt == 0) and we are still in an unstable state (!drbd_state_is_stable()). When we get an IO request in drbd_get_max_buffers() (called from __inc_ap_bio_cond(), called from inc_ap_bio()) we wake up misc_wait. Misc_wait is also used in inc_ap_bio() to sleep until the outcome of __inc_ap_bio_cond() changes. => Busy loop! Solution: Have a dedicated wait queue for get_net_conf() and put_net_conf(). Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com> Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
Diffstat (limited to 'drivers/block/drbd/drbd_int.h')
-rw-r--r--drivers/block/drbd/drbd_int.h3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/block/drbd/drbd_int.h b/drivers/block/drbd/drbd_int.h
index e0e08f5e0a76..aa9bb213fe70 100644
--- a/drivers/block/drbd/drbd_int.h
+++ b/drivers/block/drbd/drbd_int.h
@@ -947,6 +947,7 @@ struct drbd_conf {
947 union drbd_state state; 947 union drbd_state state;
948 wait_queue_head_t misc_wait; 948 wait_queue_head_t misc_wait;
949 wait_queue_head_t state_wait; /* upon each state change. */ 949 wait_queue_head_t state_wait; /* upon each state change. */
950 wait_queue_head_t net_cnt_wait;
950 unsigned int send_cnt; 951 unsigned int send_cnt;
951 unsigned int recv_cnt; 952 unsigned int recv_cnt;
952 unsigned int read_cnt; 953 unsigned int read_cnt;
@@ -2018,7 +2019,7 @@ static inline void inc_unacked(struct drbd_conf *mdev)
2018static inline void put_net_conf(struct drbd_conf *mdev) 2019static inline void put_net_conf(struct drbd_conf *mdev)
2019{ 2020{
2020 if (atomic_dec_and_test(&mdev->net_cnt)) 2021 if (atomic_dec_and_test(&mdev->net_cnt))
2021 wake_up(&mdev->misc_wait); 2022 wake_up(&mdev->net_cnt_wait);
2022} 2023}
2023 2024
2024/** 2025/**