scsi: qla2xxx: Remove extra register read
authorQuinn Tran <quinn.tran@cavium.com>
Fri, 2 Jun 2017 16:12:07 +0000 (09:12 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 13 Jun 2017 00:48:08 +0000 (20:48 -0400)
Remove extra register read in each interrupt processing to improve
performance.

Signed-off-by: Quinn Tran <quinn.tran@cavium.com>
Signed-off-by: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla2xxx/qla_iocb.c
drivers/scsi/qla2xxx/qla_target.c

index ea027f6a7fd4e949c1a9a53aad0de00b0a7ee361..8404f17f3c6cfb7c3bd5c3a6910541d9e87046db 100644 (file)
@@ -464,7 +464,9 @@ qla2x00_start_iocbs(struct scsi_qla_host *vha, struct req_que *req)
                        req->ring_ptr++;
 
                /* 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);
                        RD_REG_DWORD_RELAXED(&ha->iobase->isp24.hccr);
                } else if (IS_QLAFX00(ha)) {
index a8e57072019be2002a17de90ed3a736a7d83fe8a..88eea4d3448734d3e6ecdac5b9a7d19ed7185513 100644 (file)
@@ -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,
        uint32_t req_cnt)
 {
-       uint32_t cnt, cnt_in;
+       uint32_t cnt;
 
        if (vha->req->cnt < (req_cnt + 2)) {
                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)
                        vha->req->cnt = cnt - vha->req->ring_index;
@@ -2264,14 +2263,8 @@ static int qlt_check_reserve_free_req(struct scsi_qla_host *vha,
                        vha->req->cnt = vha->req->length -
                            (vha->req->ring_index - cnt);
 
-               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);
+               if (unlikely(vha->req->cnt < (req_cnt + 2)))
                        return -EAGAIN;
-               }
        }
 
        vha->req->cnt -= req_cnt;