scsi: qla2xxx: Remove datasegs_per_cmd and datasegs_per_cont field
authorQuinn Tran <quinn.tran@cavium.com>
Wed, 14 Jun 2017 03:47:27 +0000 (20:47 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 28 Jun 2017 01:21:41 +0000 (21:21 -0400)
These fields only hold one set of value. Replace it with macros to
reduce cache thrash.

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_target.c
drivers/scsi/qla2xxx/qla_target.h

index e8aa58dfa93c4de87e536737c2d0ef689e130cc4..84e8130029826a63ef83f1f0e9bbbf9d1f1ab34b 100644 (file)
@@ -2228,10 +2228,10 @@ static int qlt_pci_map_calc_cnt(struct qla_tgt_prm *prm)
                 * If greater than four sg entries then we need to allocate
                 * the continuation entries
                 */
-               if (prm->seg_cnt > prm->tgt->datasegs_per_cmd)
+               if (prm->seg_cnt > QLA_TGT_DATASEGS_PER_CMD_24XX)
                        prm->req_cnt += DIV_ROUND_UP(prm->seg_cnt -
-                       prm->tgt->datasegs_per_cmd,
-                       prm->tgt->datasegs_per_cont);
+                       QLA_TGT_DATASEGS_PER_CMD_24XX,
+                       QLA_TGT_DATASEGS_PER_CONT_24XX);
        } else {
                /* DIF */
                if ((cmd->se_cmd.prot_op == TARGET_PROT_DIN_INSERT) ||
@@ -2448,7 +2448,7 @@ static void qlt_load_cont_data_segments(struct qla_tgt_prm *prm)
 
                /* Load continuation entry data segments */
                for (cnt = 0;
-                   cnt < prm->tgt->datasegs_per_cont && prm->seg_cnt;
+                   cnt < QLA_TGT_DATASEGS_PER_CONT_24XX && prm->seg_cnt;
                    cnt++, prm->seg_cnt--) {
                        *dword_ptr++ =
                            cpu_to_le32(pci_dma_lo32
@@ -2492,7 +2492,7 @@ static void qlt_load_data_segments(struct qla_tgt_prm *prm)
 
        /* Load command entry data segments */
        for (cnt = 0;
-           (cnt < prm->tgt->datasegs_per_cmd) && prm->seg_cnt;
+           (cnt < QLA_TGT_DATASEGS_PER_CMD_24XX) && prm->seg_cnt;
            cnt++, prm->seg_cnt--) {
                *dword_ptr++ =
                    cpu_to_le32(pci_dma_lo32(sg_dma_address(prm->sg)));
@@ -6164,8 +6164,6 @@ int qlt_add_target(struct qla_hw_data *ha, struct scsi_qla_host *base_vha)
                base_vha->vp_idx);
        /* 3 is reserved */
        tgt->sg_tablesize = QLA_TGT_MAX_SG_24XX(base_vha->req->length - 3);
-       tgt->datasegs_per_cmd = QLA_TGT_DATASEGS_PER_CMD_24XX;
-       tgt->datasegs_per_cont = QLA_TGT_DATASEGS_PER_CONT_24XX;
 
        mutex_lock(&qla_tgt_mutex);
        list_add_tail(&tgt->tgt_list_entry, &qla_tgt_glist);
index 902685f85506a5fa98ce92fda1c09296507674e6..7fe02d036bdf1c92bff8a02c31a6b8b0fd1357bc 100644 (file)
@@ -806,7 +806,7 @@ struct qla_tgt {
         */
        int atio_irq_cmd_count;
 
-       int datasegs_per_cmd, datasegs_per_cont, sg_tablesize;
+       int sg_tablesize;
 
        /* Target's flags, serialized by pha->hardware_lock */
        unsigned int link_reinit_iocb_pending:1;