scsi: target: core: Get rid of warning in compare_and_write_do_cmp()
authorChaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Sun, 28 Feb 2021 05:56:27 +0000 (21:56 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 10 Mar 2021 04:47:18 +0000 (23:47 -0500)
Rename function local variable i to sg_cnt so we can get rid of the
shadow variable compilation warning:

unsigned int i;
     ^
int i;
^

Link: https://lore.kernel.org/r/20210228055645.22253-6-chaitanya.kulkarni@wdc.com
Reviewed-by: Mike Christie <michael.christie@oracle.com>
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/target/target_core_sbc.c

index f7c527a..7b07e55 100644 (file)
@@ -448,7 +448,7 @@ compare_and_write_do_cmp(struct scatterlist *read_sgl, unsigned int read_nents,
        sense_reason_t ret;
        unsigned int offset;
        size_t rc;
-       int i;
+       int sg_cnt;
 
        buf = kzalloc(cmp_len, GFP_KERNEL);
        if (!buf) {
@@ -467,7 +467,7 @@ compare_and_write_do_cmp(struct scatterlist *read_sgl, unsigned int read_nents,
         */
        offset = 0;
        ret = TCM_NO_SENSE;
-       for_each_sg(read_sgl, sg, read_nents, i) {
+       for_each_sg(read_sgl, sg, read_nents, sg_cnt) {
                unsigned int len = min(sg->length, cmp_len);
                unsigned char *addr = kmap_atomic(sg_page(sg));