]> git.proxmox.com Git - mirror_ubuntu-artful-kernel.git/commit
target: Fix target_release_cmd_kref shutdown comp leak
authorHimanshu Madhani <himanshu.madhani@qlogic.com>
Tue, 15 Mar 2016 05:47:37 +0000 (22:47 -0700)
committerTim Gardner <tim.gardner@canonical.com>
Thu, 21 Apr 2016 12:53:46 +0000 (06:53 -0600)
commitade08bbfd2e90a689e8c1c567e45e59dddb15c92
tree6bb8411025226432bd3b410e6bb9531509f122f8
parent28d73507b5dd9dfd80dcaf95f3fc7a09d9f0f563
target: Fix target_release_cmd_kref shutdown comp leak

BugLink: http://bugs.launchpad.net/bugs/1572722
commit 5e47f1985d7107331c3f64fb3ec83d66fd73577e upstream.

This patch fixes an active I/O shutdown bug for fabric
drivers using target_wait_for_sess_cmds(), where se_cmd
descriptor shutdown would result in hung tasks waiting
indefinitely for se_cmd->cmd_wait_comp to complete().

To address this bug, drop the incorrect list_del_init()
usage in target_wait_for_sess_cmds() and always complete()
during se_cmd target_release_cmd_kref() put, in order to
let caller invoke the final fabric release callback
into se_cmd->se_tfo->release_cmd() code.

Reported-by: Himanshu Madhani <himanshu.madhani@qlogic.com>
Tested-by: Himanshu Madhani <himanshu.madhani@qlogic.com>
Signed-off-by: Himanshu Madhani <himanshu.madhani@qlogic.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
drivers/target/target_core_transport.c