[SCSI] qla2xxx: convert to use the data buffer accessors
authorFUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Fri, 25 May 2007 16:55:38 +0000 (01:55 +0900)
committerJames Bottomley <jejb@mulgrave.il.steeleye.com>
Thu, 31 May 2007 17:23:49 +0000 (12:23 -0500)
- remove the unnecessary map_single path.

- convert to use the new accessors for the sg lists and the
parameters.

Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Acked-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
drivers/scsi/qla2xxx/qla_dbg.c
drivers/scsi/qla2xxx/qla_iocb.c
drivers/scsi/qla2xxx/qla_isr.c
drivers/scsi/qla2xxx/qla_os.c

index f6ed6962bc2b71617b631d422e0221eb8c190488..996c47a6307406d653ee095e1cacbd3cd24c0918 100644 (file)
@@ -1411,9 +1411,9 @@ qla2x00_print_scsi_cmd(struct scsi_cmnd * cmd)
                printk("0x%02x ", cmd->cmnd[i]);
        }
        printk("\n  seg_cnt=%d, allowed=%d, retries=%d\n",
-           cmd->use_sg, cmd->allowed, cmd->retries);
+              scsi_sg_count(cmd), cmd->allowed, cmd->retries);
        printk("  request buffer=0x%p, request buffer len=0x%x\n",
-           cmd->request_buffer, cmd->request_bufflen);
+              scsi_sglist(cmd), scsi_bufflen(cmd));
        printk("  tag=%d, transfersize=0x%x\n",
            cmd->tag, cmd->transfersize);
        printk("  serial_number=%lx, SP=%p\n", cmd->serial_number, sp);
index c5b3c610a32a2968219ca29c8a572cc35b0ac605..c517a1478e442b781936e10f809c9271eedc6dbe 100644 (file)
@@ -155,6 +155,8 @@ void qla2x00_build_scsi_iocbs_32(srb_t *sp, cmd_entry_t *cmd_pkt,
        uint32_t        *cur_dsd;
        scsi_qla_host_t *ha;
        struct scsi_cmnd *cmd;
+       struct scatterlist *sg;
+       int i;
 
        cmd = sp->cmd;
 
@@ -163,7 +165,7 @@ void qla2x00_build_scsi_iocbs_32(srb_t *sp, cmd_entry_t *cmd_pkt,
            __constant_cpu_to_le32(COMMAND_TYPE);
 
        /* No data transfer */
-       if (cmd->request_bufflen == 0 || cmd->sc_data_direction == DMA_NONE) {
+       if (!scsi_bufflen(cmd) || cmd->sc_data_direction == DMA_NONE) {
                cmd_pkt->byte_count = __constant_cpu_to_le32(0);
                return;
        }
@@ -177,35 +179,24 @@ void qla2x00_build_scsi_iocbs_32(srb_t *sp, cmd_entry_t *cmd_pkt,
        cur_dsd = (uint32_t *)&cmd_pkt->dseg_0_address;
 
        /* Load data segments */
-       if (cmd->use_sg != 0) {
-               struct  scatterlist *cur_seg;
-               struct  scatterlist *end_seg;
-
-               cur_seg = (struct scatterlist *)cmd->request_buffer;
-               end_seg = cur_seg + tot_dsds;
-               while (cur_seg < end_seg) {
-                       cont_entry_t    *cont_pkt;
-
-                       /* Allocate additional continuation packets? */
-                       if (avail_dsds == 0) {
-                               /*
-                                * Seven DSDs are available in the Continuation
-                                * Type 0 IOCB.
-                                */
-                               cont_pkt = qla2x00_prep_cont_type0_iocb(ha);
-                               cur_dsd = (uint32_t *)&cont_pkt->dseg_0_address;
-                               avail_dsds = 7;
-                       }
-
-                       *cur_dsd++ = cpu_to_le32(sg_dma_address(cur_seg));
-                       *cur_dsd++ = cpu_to_le32(sg_dma_len(cur_seg));
-                       avail_dsds--;
-
-                       cur_seg++;
+
+       scsi_for_each_sg(cmd, sg, tot_dsds, i) {
+               cont_entry_t *cont_pkt;
+
+               /* Allocate additional continuation packets? */
+               if (avail_dsds == 0) {
+                       /*
+                        * Seven DSDs are available in the Continuation
+                        * Type 0 IOCB.
+                        */
+                       cont_pkt = qla2x00_prep_cont_type0_iocb(ha);
+                       cur_dsd = (uint32_t *)&cont_pkt->dseg_0_address;
+                       avail_dsds = 7;
                }
-       } else {
-               *cur_dsd++ = cpu_to_le32(sp->dma_handle);
-               *cur_dsd++ = cpu_to_le32(cmd->request_bufflen);
+
+               *cur_dsd++ = cpu_to_le32(sg_dma_address(sg));
+               *cur_dsd++ = cpu_to_le32(sg_dma_len(sg));
+               avail_dsds--;
        }
 }
 
@@ -224,6 +215,8 @@ void qla2x00_build_scsi_iocbs_64(srb_t *sp, cmd_entry_t *cmd_pkt,
        uint32_t        *cur_dsd;
        scsi_qla_host_t *ha;
        struct scsi_cmnd *cmd;
+       struct scatterlist *sg;
+       int i;
 
        cmd = sp->cmd;
 
@@ -232,7 +225,7 @@ void qla2x00_build_scsi_iocbs_64(srb_t *sp, cmd_entry_t *cmd_pkt,
            __constant_cpu_to_le32(COMMAND_A64_TYPE);
 
        /* No data transfer */
-       if (cmd->request_bufflen == 0 || cmd->sc_data_direction == DMA_NONE) {
+       if (!scsi_bufflen(cmd) || cmd->sc_data_direction == DMA_NONE) {
                cmd_pkt->byte_count = __constant_cpu_to_le32(0);
                return;
        }
@@ -246,39 +239,26 @@ void qla2x00_build_scsi_iocbs_64(srb_t *sp, cmd_entry_t *cmd_pkt,
        cur_dsd = (uint32_t *)&cmd_pkt->dseg_0_address;
 
        /* Load data segments */
-       if (cmd->use_sg != 0) {
-               struct  scatterlist *cur_seg;
-               struct  scatterlist *end_seg;
-
-               cur_seg = (struct scatterlist *)cmd->request_buffer;
-               end_seg = cur_seg + tot_dsds;
-               while (cur_seg < end_seg) {
-                       dma_addr_t      sle_dma;
-                       cont_a64_entry_t *cont_pkt;
-
-                       /* Allocate additional continuation packets? */
-                       if (avail_dsds == 0) {
-                               /*
-                                * Five DSDs are available in the Continuation
-                                * Type 1 IOCB.
-                                */
-                               cont_pkt = qla2x00_prep_cont_type1_iocb(ha);
-                               cur_dsd = (uint32_t *)cont_pkt->dseg_0_address;
-                               avail_dsds = 5;
-                       }
-
-                       sle_dma = sg_dma_address(cur_seg);
-                       *cur_dsd++ = cpu_to_le32(LSD(sle_dma));
-                       *cur_dsd++ = cpu_to_le32(MSD(sle_dma));
-                       *cur_dsd++ = cpu_to_le32(sg_dma_len(cur_seg));
-                       avail_dsds--;
-
-                       cur_seg++;
+       scsi_for_each_sg(cmd, sg, tot_dsds, i) {
+               dma_addr_t      sle_dma;
+               cont_a64_entry_t *cont_pkt;
+
+               /* Allocate additional continuation packets? */
+               if (avail_dsds == 0) {
+                       /*
+                        * Five DSDs are available in the Continuation
+                        * Type 1 IOCB.
+                        */
+                       cont_pkt = qla2x00_prep_cont_type1_iocb(ha);
+                       cur_dsd = (uint32_t *)cont_pkt->dseg_0_address;
+                       avail_dsds = 5;
                }
-       } else {
-               *cur_dsd++ = cpu_to_le32(LSD(sp->dma_handle));
-               *cur_dsd++ = cpu_to_le32(MSD(sp->dma_handle));
-               *cur_dsd++ = cpu_to_le32(cmd->request_bufflen);
+
+               sle_dma = sg_dma_address(sg);
+               *cur_dsd++ = cpu_to_le32(LSD(sle_dma));
+               *cur_dsd++ = cpu_to_le32(MSD(sle_dma));
+               *cur_dsd++ = cpu_to_le32(sg_dma_len(sg));
+               avail_dsds--;
        }
 }
 
@@ -291,7 +271,7 @@ void qla2x00_build_scsi_iocbs_64(srb_t *sp, cmd_entry_t *cmd_pkt,
 int
 qla2x00_start_scsi(srb_t *sp)
 {
-       int             ret;
+       int             ret, nseg;
        unsigned long   flags;
        scsi_qla_host_t *ha;
        struct scsi_cmnd *cmd;
@@ -299,7 +279,6 @@ qla2x00_start_scsi(srb_t *sp)
        uint32_t        index;
        uint32_t        handle;
        cmd_entry_t     *cmd_pkt;
-       struct scatterlist *sg;
        uint16_t        cnt;
        uint16_t        req_cnt;
        uint16_t        tot_dsds;
@@ -337,23 +316,10 @@ qla2x00_start_scsi(srb_t *sp)
                goto queuing_error;
 
        /* Map the sg table so we have an accurate count of sg entries needed */
-       if (cmd->use_sg) {
-               sg = (struct scatterlist *) cmd->request_buffer;
-               tot_dsds = pci_map_sg(ha->pdev, sg, cmd->use_sg,
-                   cmd->sc_data_direction);
-               if (tot_dsds == 0)
-                       goto queuing_error;
-       } else if (cmd->request_bufflen) {
-               dma_addr_t      req_dma;
-
-               req_dma = pci_map_single(ha->pdev, cmd->request_buffer,
-                   cmd->request_bufflen, cmd->sc_data_direction);
-               if (dma_mapping_error(req_dma))
-                       goto queuing_error;
-
-               sp->dma_handle = req_dma;
-               tot_dsds = 1;
-       }
+       nseg = scsi_dma_map(cmd);
+       if (nseg < 0)
+               goto queuing_error;
+       tot_dsds = nseg;
 
        /* Calculate the number of request entries needed. */
        req_cnt = ha->isp_ops.calc_req_entries(tot_dsds);
@@ -391,7 +357,7 @@ qla2x00_start_scsi(srb_t *sp)
 
        /* Load SCSI command packet. */
        memcpy(cmd_pkt->scsi_cdb, cmd->cmnd, cmd->cmd_len);
-       cmd_pkt->byte_count = cpu_to_le32((uint32_t)cmd->request_bufflen);
+       cmd_pkt->byte_count = cpu_to_le32((uint32_t)scsi_bufflen(cmd));
 
        /* Build IOCB segments */
        ha->isp_ops.build_iocbs(sp, cmd_pkt, tot_dsds);
@@ -423,14 +389,9 @@ qla2x00_start_scsi(srb_t *sp)
        return (QLA_SUCCESS);
 
 queuing_error:
-       if (cmd->use_sg && tot_dsds) {
-               sg = (struct scatterlist *) cmd->request_buffer;
-               pci_unmap_sg(ha->pdev, sg, cmd->use_sg,
-                   cmd->sc_data_direction);
-       } else if (tot_dsds) {
-               pci_unmap_single(ha->pdev, sp->dma_handle,
-                   cmd->request_bufflen, cmd->sc_data_direction);
-       }
+       if (tot_dsds)
+               scsi_dma_unmap(cmd);
+
        spin_unlock_irqrestore(&ha->hardware_lock, flags);
 
        return (QLA_FUNCTION_FAILED);
@@ -642,6 +603,8 @@ qla24xx_build_scsi_iocbs(srb_t *sp, struct cmd_type_7 *cmd_pkt,
        uint32_t        *cur_dsd;
        scsi_qla_host_t *ha;
        struct scsi_cmnd *cmd;
+       struct scatterlist *sg;
+       int i;
 
        cmd = sp->cmd;
 
@@ -650,7 +613,7 @@ qla24xx_build_scsi_iocbs(srb_t *sp, struct cmd_type_7 *cmd_pkt,
            __constant_cpu_to_le32(COMMAND_TYPE_7);
 
        /* No data transfer */
-       if (cmd->request_bufflen == 0 || cmd->sc_data_direction == DMA_NONE) {
+       if (!scsi_bufflen(cmd) || cmd->sc_data_direction == DMA_NONE) {
                cmd_pkt->byte_count = __constant_cpu_to_le32(0);
                return;
        }
@@ -670,39 +633,27 @@ qla24xx_build_scsi_iocbs(srb_t *sp, struct cmd_type_7 *cmd_pkt,
        cur_dsd = (uint32_t *)&cmd_pkt->dseg_0_address;
 
        /* Load data segments */
-       if (cmd->use_sg != 0) {
-               struct  scatterlist *cur_seg;
-               struct  scatterlist *end_seg;
-
-               cur_seg = (struct scatterlist *)cmd->request_buffer;
-               end_seg = cur_seg + tot_dsds;
-               while (cur_seg < end_seg) {
-                       dma_addr_t      sle_dma;
-                       cont_a64_entry_t *cont_pkt;
-
-                       /* Allocate additional continuation packets? */
-                       if (avail_dsds == 0) {
-                               /*
-                                * Five DSDs are available in the Continuation
-                                * Type 1 IOCB.
-                                */
-                               cont_pkt = qla2x00_prep_cont_type1_iocb(ha);
-                               cur_dsd = (uint32_t *)cont_pkt->dseg_0_address;
-                               avail_dsds = 5;
-                       }
-
-                       sle_dma = sg_dma_address(cur_seg);
-                       *cur_dsd++ = cpu_to_le32(LSD(sle_dma));
-                       *cur_dsd++ = cpu_to_le32(MSD(sle_dma));
-                       *cur_dsd++ = cpu_to_le32(sg_dma_len(cur_seg));
-                       avail_dsds--;
-
-                       cur_seg++;
+
+       scsi_for_each_sg(cmd, sg, tot_dsds, i) {
+               dma_addr_t      sle_dma;
+               cont_a64_entry_t *cont_pkt;
+
+               /* Allocate additional continuation packets? */
+               if (avail_dsds == 0) {
+                       /*
+                        * Five DSDs are available in the Continuation
+                        * Type 1 IOCB.
+                        */
+                       cont_pkt = qla2x00_prep_cont_type1_iocb(ha);
+                       cur_dsd = (uint32_t *)cont_pkt->dseg_0_address;
+                       avail_dsds = 5;
                }
-       } else {
-               *cur_dsd++ = cpu_to_le32(LSD(sp->dma_handle));
-               *cur_dsd++ = cpu_to_le32(MSD(sp->dma_handle));
-               *cur_dsd++ = cpu_to_le32(cmd->request_bufflen);
+
+               sle_dma = sg_dma_address(sg);
+               *cur_dsd++ = cpu_to_le32(LSD(sle_dma));
+               *cur_dsd++ = cpu_to_le32(MSD(sle_dma));
+               *cur_dsd++ = cpu_to_le32(sg_dma_len(sg));
+               avail_dsds--;
        }
 }
 
@@ -716,7 +667,7 @@ qla24xx_build_scsi_iocbs(srb_t *sp, struct cmd_type_7 *cmd_pkt,
 int
 qla24xx_start_scsi(srb_t *sp)
 {
-       int             ret;
+       int             ret, nseg;
        unsigned long   flags;
        scsi_qla_host_t *ha;
        struct scsi_cmnd *cmd;
@@ -724,7 +675,6 @@ qla24xx_start_scsi(srb_t *sp)
        uint32_t        index;
        uint32_t        handle;
        struct cmd_type_7 *cmd_pkt;
-       struct scatterlist *sg;
        uint16_t        cnt;
        uint16_t        req_cnt;
        uint16_t        tot_dsds;
@@ -762,23 +712,10 @@ qla24xx_start_scsi(srb_t *sp)
                goto queuing_error;
 
        /* Map the sg table so we have an accurate count of sg entries needed */
-       if (cmd->use_sg) {
-               sg = (struct scatterlist *) cmd->request_buffer;
-               tot_dsds = pci_map_sg(ha->pdev, sg, cmd->use_sg,
-                   cmd->sc_data_direction);
-               if (tot_dsds == 0)
+       nseg = scsi_dma_map(cmd);
+       if (nseg < 0)
                        goto queuing_error;
-       } else if (cmd->request_bufflen) {
-               dma_addr_t      req_dma;
-
-               req_dma = pci_map_single(ha->pdev, cmd->request_buffer,
-                   cmd->request_bufflen, cmd->sc_data_direction);
-               if (dma_mapping_error(req_dma))
-                       goto queuing_error;
-
-               sp->dma_handle = req_dma;
-               tot_dsds = 1;
-       }
+       tot_dsds = nseg;
 
        req_cnt = qla24xx_calc_iocbs(tot_dsds);
        if (ha->req_q_cnt < (req_cnt + 2)) {
@@ -821,7 +758,7 @@ qla24xx_start_scsi(srb_t *sp)
        memcpy(cmd_pkt->fcp_cdb, cmd->cmnd, cmd->cmd_len);
        host_to_fcp_swap(cmd_pkt->fcp_cdb, sizeof(cmd_pkt->fcp_cdb));
 
-       cmd_pkt->byte_count = cpu_to_le32((uint32_t)cmd->request_bufflen);
+       cmd_pkt->byte_count = cpu_to_le32((uint32_t)scsi_bufflen(cmd));
 
        /* Build IOCB segments */
        qla24xx_build_scsi_iocbs(sp, cmd_pkt, tot_dsds);
@@ -853,14 +790,9 @@ qla24xx_start_scsi(srb_t *sp)
        return QLA_SUCCESS;
 
 queuing_error:
-       if (cmd->use_sg && tot_dsds) {
-               sg = (struct scatterlist *) cmd->request_buffer;
-               pci_unmap_sg(ha->pdev, sg, cmd->use_sg,
-                   cmd->sc_data_direction);
-       } else if (tot_dsds) {
-               pci_unmap_single(ha->pdev, sp->dma_handle,
-                   cmd->request_bufflen, cmd->sc_data_direction);
-       }
+       if (tot_dsds)
+               scsi_dma_unmap(cmd);
+
        spin_unlock_irqrestore(&ha->hardware_lock, flags);
 
        return QLA_FUNCTION_FAILED;
index 0f04258becbf0483c0abf524fe65a5e097d3c674..6ce532cdc4c1ee6b148ddea22f8c08ee33f1a205 100644 (file)
@@ -889,19 +889,19 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt)
                }
                if (scsi_status & (SS_RESIDUAL_UNDER | SS_RESIDUAL_OVER)) {
                        resid = resid_len;
-                       cp->resid = resid;
+                       scsi_set_resid(cp, resid);
                        CMD_RESID_LEN(cp) = resid;
 
                        if (!lscsi_status &&
-                           ((unsigned)(cp->request_bufflen - resid) <
+                           ((unsigned)(scsi_bufflen(cp) - resid) <
                             cp->underflow)) {
                                qla_printk(KERN_INFO, ha,
-                                   "scsi(%ld:%d:%d:%d): Mid-layer underflow "
-                                   "detected (%x of %x bytes)...returning "
-                                   "error status.\n", ha->host_no,
-                                   cp->device->channel, cp->device->id,
-                                   cp->device->lun, resid,
-                                   cp->request_bufflen);
+                                          "scsi(%ld:%d:%d:%d): Mid-layer underflow "
+                                          "detected (%x of %x bytes)...returning "
+                                          "error status.\n", ha->host_no,
+                                          cp->device->channel, cp->device->id,
+                                          cp->device->lun, resid,
+                                          scsi_bufflen(cp));
 
                                cp->result = DID_ERROR << 16;
                                break;
@@ -963,7 +963,7 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt)
                        resid = fw_resid_len;
 
                if (scsi_status & SS_RESIDUAL_UNDER) {
-                       cp->resid = resid;
+                       scsi_set_resid(cp, resid);
                        CMD_RESID_LEN(cp) = resid;
                } else {
                        DEBUG2(printk(KERN_INFO
@@ -1042,26 +1042,26 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt)
                         */
                        if (!(scsi_status & SS_RESIDUAL_UNDER)) {
                                DEBUG2(printk("scsi(%ld:%d:%d:%d) Dropped "
-                                   "frame(s) detected (%x of %x bytes)..."
-                                   "retrying command.\n", ha->host_no,
-                                   cp->device->channel, cp->device->id,
-                                   cp->device->lun, resid,
-                                   cp->request_bufflen));
+                                             "frame(s) detected (%x of %x bytes)..."
+                                             "retrying command.\n", ha->host_no,
+                                             cp->device->channel, cp->device->id,
+                                             cp->device->lun, resid,
+                                             scsi_bufflen(cp)));
 
                                cp->result = DID_BUS_BUSY << 16;
                                break;
                        }
 
                        /* Handle mid-layer underflow */
-                       if ((unsigned)(cp->request_bufflen - resid) <
+                       if ((unsigned)(scsi_bufflen(cp) - resid) <
                            cp->underflow) {
                                qla_printk(KERN_INFO, ha,
-                                   "scsi(%ld:%d:%d:%d): Mid-layer underflow "
-                                   "detected (%x of %x bytes)...returning "
-                                   "error status.\n", ha->host_no,
-                                   cp->device->channel, cp->device->id,
-                                   cp->device->lun, resid,
-                                   cp->request_bufflen);
+                                          "scsi(%ld:%d:%d:%d): Mid-layer underflow "
+                                          "detected (%x of %x bytes)...returning "
+                                          "error status.\n", ha->host_no,
+                                          cp->device->channel, cp->device->id,
+                                          cp->device->lun, resid,
+                                          scsi_bufflen(cp));
 
                                cp->result = DID_ERROR << 16;
                                break;
@@ -1084,7 +1084,7 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt)
                DEBUG2(printk(KERN_INFO
                    "PID=0x%lx req=0x%x xtra=0x%x -- returning DID_ERROR "
                    "status!\n",
-                   cp->serial_number, cp->request_bufflen, resid_len));
+                   cp->serial_number, scsi_bufflen(cp), resid_len));
 
                cp->result = DID_ERROR << 16;
                break;
index dd076da86a465d668d6b251512492176a65e81be..18baa5bf69c40af27c05e6debf4555c79d5c2613 100644 (file)
@@ -2426,13 +2426,7 @@ qla2x00_sp_free_dma(scsi_qla_host_t *ha, srb_t *sp)
        struct scsi_cmnd *cmd = sp->cmd;
 
        if (sp->flags & SRB_DMA_VALID) {
-               if (cmd->use_sg) {
-                       dma_unmap_sg(&ha->pdev->dev, cmd->request_buffer,
-                           cmd->use_sg, cmd->sc_data_direction);
-               } else if (cmd->request_bufflen) {
-                       dma_unmap_single(&ha->pdev->dev, sp->dma_handle,
-                           cmd->request_bufflen, cmd->sc_data_direction);
-               }
+               scsi_dma_unmap(cmd);
                sp->flags &= ~SRB_DMA_VALID;
        }
        CMD_SP(cmd) = NULL;