mtip32xx: move the blk_rq_map_sg call to mtip_hw_submit_io
authorChristoph Hellwig <hch@lst.de>
Fri, 9 Nov 2018 13:48:54 +0000 (14:48 +0100)
committerJens Axboe <axboe@kernel.dk>
Fri, 9 Nov 2018 15:39:21 +0000 (08:39 -0700)
We have all arguments at hand in mtip_hw_submit_io, so keep the
rq to sg mapping close to the dma_map_sg call.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/mtip32xx/mtip32xx.c

index 947aa10107a684e852e020469614c65b4f35df3d..1964af8a187b232a56a28a7ee738925439e3c384 100644 (file)
@@ -2171,7 +2171,6 @@ static int mtip_hw_ioctl(struct driver_data *dd, unsigned int cmd,
  * @dd       Pointer to the driver data structure.
  * @start    First sector to read.
  * @nsect    Number of sectors to read.
- * @nents    Number of entries in scatter list for the read command.
  * @tag      The tag of this read command.
  * @callback Pointer to the function that should be called
  *          when the read completes.
@@ -2183,7 +2182,7 @@ static int mtip_hw_ioctl(struct driver_data *dd, unsigned int cmd,
  *     None
  */
 static void mtip_hw_submit_io(struct driver_data *dd, struct request *rq,
-                             struct mtip_cmd *command, int nents,
+                             struct mtip_cmd *command,
                              struct blk_mq_hw_ctx *hctx)
 {
        struct host_to_dev_fis  *fis;
@@ -2191,8 +2190,10 @@ static void mtip_hw_submit_io(struct driver_data *dd, struct request *rq,
        int dma_dir = rq_data_dir(rq) == READ ? DMA_FROM_DEVICE : DMA_TO_DEVICE;
        u64 start = blk_rq_pos(rq);
        unsigned int nsect = blk_rq_sectors(rq);
+       unsigned int nents;
 
        /* Map the scatter list for DMA access */
+       nents = blk_rq_map_sg(hctx->queue, rq, command->sg);
        nents = dma_map_sg(&dd->pdev->dev, command->sg, nents, dma_dir);
 
        prefetch(&port->flags);
@@ -3548,7 +3549,6 @@ static int mtip_submit_request(struct blk_mq_hw_ctx *hctx, struct request *rq)
 {
        struct driver_data *dd = hctx->queue->queuedata;
        struct mtip_cmd *cmd = blk_mq_rq_to_pdu(rq);
-       unsigned int nents;
 
        if (is_se_active(dd))
                return -ENODATA;
@@ -3579,11 +3579,8 @@ static int mtip_submit_request(struct blk_mq_hw_ctx *hctx, struct request *rq)
                return 0;
        }
 
-       /* Create the scatter list for this request. */
-       nents = blk_rq_map_sg(hctx->queue, rq, cmd->sg);
-
        /* Issue the read/write. */
-       mtip_hw_submit_io(dd, rq, cmd, nents, hctx);
+       mtip_hw_submit_io(dd, rq, cmd, hctx);
        return 0;
 }