* [PATCH RFC] scsi: target: iscsi: Handle abort for ICF_OOO_CMDSN cmds
@ 2023-07-25 11:33 jirong.feng
0 siblings, 0 replies; only message in thread
From: jirong.feng @ 2023-07-25 11:33 UTC (permalink / raw)
To: martin.petersen, michael.christie, mlombard, d.bogdanov
Cc: dongsheng.yang, linux-scsi, target-devel, linux-kernel, jirong.feng
If a connection closes with an ICF_OOO_CMDSN cmd being aborted, then
rx thread will wait for the abort cmd to complete. At the same time,
the abort kworker is waiting for the ooo cmd's completion. Since rx
thread is waiting too, that ooo cmd can never reach to completion, so
is the abort cmd, causing both rx thread and abort kworker hanging
indefinitely.
Add handling in TCM of the abort for the ICF_OOO_CMDSN cmds at
connection closing moment to make it possible to release them.
---
drivers/target/iscsi/iscsi_target.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/target/iscsi/iscsi_target.c b/drivers/target/iscsi/iscsi_target.c
index b516c2893420..27edc5eb9e6e 100644
--- a/drivers/target/iscsi/iscsi_target.c
+++ b/drivers/target/iscsi/iscsi_target.c
@@ -4236,7 +4236,8 @@ static void iscsit_release_commands_from_conn(struct iscsit_conn *conn)
se_cmd->transport_state |= CMD_T_FABRIC_STOP;
}
- if (cmd->se_cmd.t_state == TRANSPORT_WRITE_PENDING) {
+ if (cmd->se_cmd.t_state == TRANSPORT_WRITE_PENDING ||
+ cmd->cmd_flags & ICF_OOO_CMDSN) {
/*
* We never submitted the cmd to LIO core, so we have
* to tell LIO to perform the completion process.
--
2.25.1
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2023-07-25 11:33 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-25 11:33 [PATCH RFC] scsi: target: iscsi: Handle abort for ICF_OOO_CMDSN cmds jirong.feng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®