diff options
author | John Soni Jose <sony.john@avagotech.com> | 2015-06-24 06:41:58 +0530 |
---|---|---|
committer | Ben Hutchings <ben@decadent.org.uk> | 2015-10-13 03:46:03 +0100 |
commit | 861a9633c22ee8b484e1b4054995f7e1b94a3834 (patch) | |
tree | 1129192f3121745a0b9c424dbb4b6a41fa1c445b /fs/logfs | |
parent | a11117c7b36d53cfd18c4d211ce30a4419ee477d (diff) | |
download | kernel_samsung_smdk4412-861a9633c22ee8b484e1b4054995f7e1b94a3834.zip kernel_samsung_smdk4412-861a9633c22ee8b484e1b4054995f7e1b94a3834.tar.gz kernel_samsung_smdk4412-861a9633c22ee8b484e1b4054995f7e1b94a3834.tar.bz2 |
libiscsi: Fix host busy blocking during connection teardown
commit 660d0831d1494a6837b2f810d08b5be092c1f31d upstream.
In case of hw iscsi offload, an host can have N-number of active
connections. There can be IO's running on some connections which
make host->host_busy always TRUE. Now if logout from a connection
is tried then the code gets into an infinite loop as host->host_busy
is always TRUE.
iscsi_conn_teardown(....)
{
.........
/*
* Block until all in-progress commands for this connection
* time out or fail.
*/
for (;;) {
spin_lock_irqsave(session->host->host_lock, flags);
if (!atomic_read(&session->host->host_busy)) { /* OK for ERL == 0 */
spin_unlock_irqrestore(session->host->host_lock, flags);
break;
}
spin_unlock_irqrestore(session->host->host_lock, flags);
msleep_interruptible(500);
iscsi_conn_printk(KERN_INFO, conn, "iscsi conn_destroy(): "
"host_busy %d host_failed %d\n",
atomic_read(&session->host->host_busy),
session->host->host_failed);
................
...............
}
}
This is not an issue with software-iscsi/iser as each cxn is a separate
host.
Fix:
Acquiring eh_mutex in iscsi_conn_teardown() before setting
session->state = ISCSI_STATE_TERMINATE.
Signed-off-by: John Soni Jose <sony.john@avagotech.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Reviewed-by: Chris Leech <cleech@redhat.com>
Signed-off-by: James Bottomley <JBottomley@Odin.com>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'fs/logfs')
0 files changed, 0 insertions, 0 deletions