diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-21 12:48:36 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-21 12:48:36 -0400 |
commit | 70ca0a42fe1702ac0784386c44b05d18ee5f1f9e (patch) | |
tree | cd9366330e409a354aa100685f162a5da638e286 /drivers/scsi | |
parent | d7dbf4ffee1c7a17e2e5b5f01efe76fbd1671db6 (diff) |
Fix networking tree iscsi_tcp.c mis-merge
The removal of the 'waitqueue_active()' test in commit d7d05548a6
("[SCSI] iscsi_tcp: fix relogin/shutdown hang") got incorrectly resolved
by David when he back-merged the main git tree into the networking tree
in commit 278554bd65 ("Merge branch 'master' of master.kernel.org:...").
There was a content conflict due to 'sock->sk->sk_sleep' being changed
into 'sk_sleep(sock->sk)' in the networking tree, but David didn't pick
up the iscsi change from the main tree.
Reported-by: James Bottomley <James.Bottomley@suse.de>
Cc: David Miller <davem@davemloft.net>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/iscsi_tcp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/iscsi_tcp.c b/drivers/scsi/iscsi_tcp.c index 7608310e19ee..bf55d3057413 100644 --- a/drivers/scsi/iscsi_tcp.c +++ b/drivers/scsi/iscsi_tcp.c | |||
@@ -601,7 +601,7 @@ static void iscsi_sw_tcp_conn_stop(struct iscsi_cls_conn *cls_conn, int flag) | |||
601 | set_bit(ISCSI_SUSPEND_BIT, &conn->suspend_rx); | 601 | set_bit(ISCSI_SUSPEND_BIT, &conn->suspend_rx); |
602 | write_unlock_bh(&tcp_sw_conn->sock->sk->sk_callback_lock); | 602 | write_unlock_bh(&tcp_sw_conn->sock->sk->sk_callback_lock); |
603 | 603 | ||
604 | if (sk_sleep(sock->sk) && waitqueue_active(sk_sleep(sock->sk))) { | 604 | if (sk_sleep(sock->sk)) { |
605 | sock->sk->sk_err = EIO; | 605 | sock->sk->sk_err = EIO; |
606 | wake_up_interruptible(sk_sleep(sock->sk)); | 606 | wake_up_interruptible(sk_sleep(sock->sk)); |
607 | } | 607 | } |