Commit 245a5754 authored by Eddie Wai's avatar Eddie Wai Committed by James Bottomley

[SCSI] bnx2fc: Fixed the handling for the SCSI retry delay

SCSI retry delay upon SAM_STAT_BUSY/_SET_FULL was not being handled
in bnx2fc.  This patch adds such handling by returning TARGET_BUSY
to the SCSI ML for the corresponding LUN until the retry timer expires.
Signed-off-by: default avatarEddie Wai <eddie.wai@broadcom.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Parallels.com>
parent 06c4f20d
...@@ -367,6 +367,7 @@ struct bnx2fc_rport { ...@@ -367,6 +367,7 @@ struct bnx2fc_rport {
atomic_t num_active_ios; atomic_t num_active_ios;
u32 flush_in_prog; u32 flush_in_prog;
unsigned long timestamp; unsigned long timestamp;
unsigned long retry_delay_timestamp;
struct list_head free_task_list; struct list_head free_task_list;
struct bnx2fc_cmd *pending_queue[BNX2FC_SQ_WQES_MAX+1]; struct bnx2fc_cmd *pending_queue[BNX2FC_SQ_WQES_MAX+1];
struct list_head active_cmd_queue; struct list_head active_cmd_queue;
......
...@@ -1871,7 +1871,15 @@ int bnx2fc_queuecommand(struct Scsi_Host *host, ...@@ -1871,7 +1871,15 @@ int bnx2fc_queuecommand(struct Scsi_Host *host,
rc = SCSI_MLQUEUE_TARGET_BUSY; rc = SCSI_MLQUEUE_TARGET_BUSY;
goto exit_qcmd; goto exit_qcmd;
} }
if (tgt->retry_delay_timestamp) {
if (time_after(jiffies, tgt->retry_delay_timestamp)) {
tgt->retry_delay_timestamp = 0;
} else {
/* If retry_delay timer is active, flow off the ML */
rc = SCSI_MLQUEUE_TARGET_BUSY;
goto exit_qcmd;
}
}
io_req = bnx2fc_cmd_alloc(tgt); io_req = bnx2fc_cmd_alloc(tgt);
if (!io_req) { if (!io_req) {
rc = SCSI_MLQUEUE_HOST_BUSY; rc = SCSI_MLQUEUE_HOST_BUSY;
...@@ -1961,6 +1969,15 @@ void bnx2fc_process_scsi_cmd_compl(struct bnx2fc_cmd *io_req, ...@@ -1961,6 +1969,15 @@ void bnx2fc_process_scsi_cmd_compl(struct bnx2fc_cmd *io_req,
" fcp_resid = 0x%x\n", " fcp_resid = 0x%x\n",
io_req->cdb_status, io_req->fcp_resid); io_req->cdb_status, io_req->fcp_resid);
sc_cmd->result = (DID_OK << 16) | io_req->cdb_status; sc_cmd->result = (DID_OK << 16) | io_req->cdb_status;
if (io_req->cdb_status == SAM_STAT_TASK_SET_FULL ||
io_req->cdb_status == SAM_STAT_BUSY) {
/* Set the jiffies + retry_delay_timer * 100ms
for the rport/tgt */
tgt->retry_delay_timestamp = jiffies +
fcp_rsp->retry_delay_timer * HZ / 10;
}
} }
if (io_req->fcp_resid) if (io_req->fcp_resid)
scsi_set_resid(sc_cmd, io_req->fcp_resid); scsi_set_resid(sc_cmd, io_req->fcp_resid);
......
...@@ -386,6 +386,7 @@ static int bnx2fc_init_tgt(struct bnx2fc_rport *tgt, ...@@ -386,6 +386,7 @@ static int bnx2fc_init_tgt(struct bnx2fc_rport *tgt,
tgt->rq_prod_idx = 0x8000; tgt->rq_prod_idx = 0x8000;
tgt->rq_cons_idx = 0; tgt->rq_cons_idx = 0;
atomic_set(&tgt->num_active_ios, 0); atomic_set(&tgt->num_active_ios, 0);
tgt->retry_delay_timestamp = 0;
if (rdata->flags & FC_RP_FLAGS_RETRY && if (rdata->flags & FC_RP_FLAGS_RETRY &&
rdata->ids.roles & FC_RPORT_ROLE_FCP_TARGET && rdata->ids.roles & FC_RPORT_ROLE_FCP_TARGET &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment