target: simply fabric driver queue full processing
There is no need to schedule the delayed processing in a workqueue that offloads it to the target processing thread. Instead execute it directly from the workqueue. There will be a lot of future work in this area, which I'd likfe to defer for now as it is not nessecary for getting rid of the target processing thread. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
This commit is contained in:
parent
1389533ef9
commit
7a6f0a1ea5
|
@ -720,6 +720,8 @@ static void target_add_to_state_list(struct se_cmd *cmd)
|
||||||
/*
|
/*
|
||||||
* Handle QUEUE_FULL / -EAGAIN and -ENOMEM status
|
* Handle QUEUE_FULL / -EAGAIN and -ENOMEM status
|
||||||
*/
|
*/
|
||||||
|
static void transport_write_pending_qf(struct se_cmd *cmd);
|
||||||
|
static void transport_complete_qf(struct se_cmd *cmd);
|
||||||
|
|
||||||
static void target_qf_do_work(struct work_struct *work)
|
static void target_qf_do_work(struct work_struct *work)
|
||||||
{
|
{
|
||||||
|
@ -743,7 +745,10 @@ static void target_qf_do_work(struct work_struct *work)
|
||||||
(cmd->t_state == TRANSPORT_COMPLETE_QF_WP) ? "WRITE_PENDING"
|
(cmd->t_state == TRANSPORT_COMPLETE_QF_WP) ? "WRITE_PENDING"
|
||||||
: "UNKNOWN");
|
: "UNKNOWN");
|
||||||
|
|
||||||
transport_add_cmd_to_queue(cmd, cmd->t_state, true);
|
if (cmd->t_state == TRANSPORT_COMPLETE_QF_WP)
|
||||||
|
transport_write_pending_qf(cmd);
|
||||||
|
else if (cmd->t_state == TRANSPORT_COMPLETE_QF_OK)
|
||||||
|
transport_complete_qf(cmd);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -3262,12 +3267,6 @@ get_cmd:
|
||||||
case TRANSPORT_PROCESS_TMR:
|
case TRANSPORT_PROCESS_TMR:
|
||||||
transport_generic_do_tmr(cmd);
|
transport_generic_do_tmr(cmd);
|
||||||
break;
|
break;
|
||||||
case TRANSPORT_COMPLETE_QF_WP:
|
|
||||||
transport_write_pending_qf(cmd);
|
|
||||||
break;
|
|
||||||
case TRANSPORT_COMPLETE_QF_OK:
|
|
||||||
transport_complete_qf(cmd);
|
|
||||||
break;
|
|
||||||
default:
|
default:
|
||||||
pr_err("Unknown t_state: %d for ITT: 0x%08x "
|
pr_err("Unknown t_state: %d for ITT: 0x%08x "
|
||||||
"i_state: %d on SE LUN: %u\n",
|
"i_state: %d on SE LUN: %u\n",
|
||||||
|
|
Loading…
Reference in New Issue