]> git.proxmox.com Git - mirror_ubuntu-artful-kernel.git/commitdiff
scsi: replace "mq" with "first_chunk" in SG functions
authorMing Lin <ming.l@ssi.samsung.com>
Mon, 4 Apr 2016 21:48:08 +0000 (14:48 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 15 Apr 2016 20:53:13 +0000 (16:53 -0400)
Parameter "bool mq" is block driver specific.
Change it to "first_chunk" to make it more generic.

Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Ming Lin <ming.l@ssi.samsung.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/scsi_lib.c

index 4229c183648b390c5dfbff2e46abfeb6bd09d587..9675353770e937debd276392c0a51a92f5f36051 100644 (file)
@@ -583,33 +583,32 @@ static struct scatterlist *scsi_sg_alloc(unsigned int nents, gfp_t gfp_mask)
        return mempool_alloc(sgp->pool, gfp_mask);
 }
 
-static void scsi_free_sgtable(struct sg_table *table, bool mq)
+static void scsi_free_sgtable(struct sg_table *table, bool first_chunk)
 {
-       if (mq && table->orig_nents <= SCSI_MAX_SG_SEGMENTS)
+       if (first_chunk && table->orig_nents <= SCSI_MAX_SG_SEGMENTS)
                return;
-       __sg_free_table(table, SCSI_MAX_SG_SEGMENTS, mq, scsi_sg_free);
+       __sg_free_table(table, SCSI_MAX_SG_SEGMENTS, first_chunk, scsi_sg_free);
 }
 
-static int scsi_alloc_sgtable(struct sg_table *table, int nents, bool mq)
+static int scsi_alloc_sgtable(struct sg_table *table, int nents,
+               struct scatterlist *first_chunk)
 {
-       struct scatterlist *first_chunk = NULL;
        int ret;
 
        BUG_ON(!nents);
 
-       if (mq) {
+       if (first_chunk) {
                if (nents <= SCSI_MAX_SG_SEGMENTS) {
                        table->nents = table->orig_nents = nents;
                        sg_init_table(table->sgl, nents);
                        return 0;
                }
-               first_chunk = table->sgl;
        }
 
        ret = __sg_alloc_table(table, nents, SCSI_MAX_SG_SEGMENTS,
                               first_chunk, GFP_ATOMIC, scsi_sg_alloc);
        if (unlikely(ret))
-               scsi_free_sgtable(table, mq);
+               scsi_free_sgtable(table, (bool)first_chunk);
        return ret;
 }
 
@@ -1091,7 +1090,7 @@ static int scsi_init_sgtable(struct request *req, struct scsi_data_buffer *sdb)
         * If sg table allocation fails, requeue request later.
         */
        if (unlikely(scsi_alloc_sgtable(&sdb->table, req->nr_phys_segments,
-                                       req->mq_ctx != NULL)))
+                                       sdb->table.sgl)))
                return BLKPREP_DEFER;
 
        /* 
@@ -1163,7 +1162,8 @@ int scsi_init_io(struct scsi_cmnd *cmd)
 
                ivecs = blk_rq_count_integrity_sg(rq->q, rq->bio);
 
-               if (scsi_alloc_sgtable(&prot_sdb->table, ivecs, is_mq)) {
+               if (scsi_alloc_sgtable(&prot_sdb->table, ivecs,
+                               prot_sdb->table.sgl)) {
                        error = BLKPREP_DEFER;
                        goto err_exit;
                }