Commit d63b328f authored by Quinn Tran's avatar Quinn Tran Committed by Martin K. Petersen

scsi: qla2xxx: Remove extra register read

Remove extra register read in each interrupt processing to improve
performance.
Signed-off-by: default avatarQuinn Tran <quinn.tran@cavium.com>
Signed-off-by: default avatarHimanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent ba68a635
...@@ -464,7 +464,9 @@ qla2x00_start_iocbs(struct scsi_qla_host *vha, struct req_que *req) ...@@ -464,7 +464,9 @@ qla2x00_start_iocbs(struct scsi_qla_host *vha, struct req_que *req)
req->ring_ptr++; req->ring_ptr++;
/* Set chip new ring index. */ /* Set chip new ring index. */
if (ha->mqenable || IS_QLA83XX(ha) || IS_QLA27XX(ha)) { if (ha->mqenable || IS_QLA27XX(ha)) {
WRT_REG_DWORD(req->req_q_in, req->ring_index);
} else if (IS_QLA83XX(ha)) {
WRT_REG_DWORD(req->req_q_in, req->ring_index); WRT_REG_DWORD(req->req_q_in, req->ring_index);
RD_REG_DWORD_RELAXED(&ha->iobase->isp24.hccr); RD_REG_DWORD_RELAXED(&ha->iobase->isp24.hccr);
} else if (IS_QLAFX00(ha)) { } else if (IS_QLAFX00(ha)) {
......
...@@ -2252,11 +2252,10 @@ static void qlt_unmap_sg(struct scsi_qla_host *vha, struct qla_tgt_cmd *cmd) ...@@ -2252,11 +2252,10 @@ static void qlt_unmap_sg(struct scsi_qla_host *vha, struct qla_tgt_cmd *cmd)
static int qlt_check_reserve_free_req(struct scsi_qla_host *vha, static int qlt_check_reserve_free_req(struct scsi_qla_host *vha,
uint32_t req_cnt) uint32_t req_cnt)
{ {
uint32_t cnt, cnt_in; uint32_t cnt;
if (vha->req->cnt < (req_cnt + 2)) { if (vha->req->cnt < (req_cnt + 2)) {
cnt = (uint16_t)RD_REG_DWORD(vha->req->req_q_out); cnt = (uint16_t)RD_REG_DWORD(vha->req->req_q_out);
cnt_in = (uint16_t)RD_REG_DWORD(vha->req->req_q_in);
if (vha->req->ring_index < cnt) if (vha->req->ring_index < cnt)
vha->req->cnt = cnt - vha->req->ring_index; vha->req->cnt = cnt - vha->req->ring_index;
...@@ -2264,14 +2263,8 @@ static int qlt_check_reserve_free_req(struct scsi_qla_host *vha, ...@@ -2264,14 +2263,8 @@ static int qlt_check_reserve_free_req(struct scsi_qla_host *vha,
vha->req->cnt = vha->req->length - vha->req->cnt = vha->req->length -
(vha->req->ring_index - cnt); (vha->req->ring_index - cnt);
if (unlikely(vha->req->cnt < (req_cnt + 2))) { if (unlikely(vha->req->cnt < (req_cnt + 2)))
ql_dbg(ql_dbg_io, vha, 0x305a,
"qla_target(%d): There is no room in the request ring: vha->req->ring_index=%d, vha->req->cnt=%d, req_cnt=%d Req-out=%d Req-in=%d Req-Length=%d\n",
vha->vp_idx, vha->req->ring_index,
vha->req->cnt, req_cnt, cnt, cnt_in,
vha->req->length);
return -EAGAIN; return -EAGAIN;
}
} }
vha->req->cnt -= req_cnt; vha->req->cnt -= req_cnt;
......
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