From: Ming Lin <min...@ssi.samsung.com>

Parameter "bool mq" is block driver specific.
Change it to "first_chunk" to make it more generic.

Reviewed-by: Christoph Hellwig <h...@lst.de>
Signed-off-by: Ming Lin <min...@ssi.samsung.com>
---
 drivers/scsi/scsi_lib.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
index 4229c18..9675353 100644
--- a/drivers/scsi/scsi_lib.c
+++ b/drivers/scsi/scsi_lib.c
@@ -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;
                }
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to