We never queue an command with this state, and only set it in a completely
bogus place in tcm_fc.
Signed-off-by: Christoph Hellwig <hch@xxxxxx>
Index: lio-core/drivers/target/target_core_transport.c
===================================================================
--- lio-core.orig/drivers/target/target_core_transport.c 2011-10-17 16:13:48.456649929 +0200
+++ lio-core/drivers/target/target_core_transport.c 2011-10-17 16:14:55.055149375 +0200
@@ -4789,9 +4789,6 @@ get_cmd:
case TRANSPORT_COMPLETE_OK:
transport_generic_complete_ok(cmd);
break;
- case TRANSPORT_REMOVE:
- transport_put_cmd(cmd);
- break;
case TRANSPORT_FREE_CMD_INTR:
transport_generic_free_cmd(cmd, 0);
break;
Index: lio-core/drivers/target/tcm_fc/tfc_cmd.c
===================================================================
--- lio-core.orig/drivers/target/tcm_fc/tfc_cmd.c 2011-10-15 15:29:46.798049801 +0200
+++ lio-core/drivers/target/tcm_fc/tfc_cmd.c 2011-10-17 16:14:55.059148327 +0200
@@ -267,7 +267,6 @@ static void ft_recv_seq(struct fc_seq *s
if (IS_ERR(fp)) {
/* XXX need to find cmd if queued */
- cmd->se_cmd.t_state = TRANSPORT_REMOVE;
cmd->seq = NULL;
transport_generic_free_cmd(&cmd->se_cmd, 0);
return;
Index: lio-core/include/target/target_core_base.h
===================================================================
--- lio-core.orig/include/target/target_core_base.h 2011-10-17 16:13:48.484651915 +0200
+++ lio-core/include/target/target_core_base.h 2011-10-17 16:14:55.067148092 +0200
@@ -95,7 +95,6 @@ enum transport_state_table {
TRANSPORT_ISTATE_PROCESSING = 11,
TRANSPORT_ISTATE_PROCESSED = 12,
TRANSPORT_KILL = 13,
- TRANSPORT_REMOVE = 14,
TRANSPORT_FREE = 15,
TRANSPORT_NEW_CMD_MAP = 16,
TRANSPORT_FREE_CMD_INTR = 17,
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
