]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
scsi: scsi_debug: Use scsi_[gs]et_resid() where appropriate
authorBart Van Assche <bvanassche@acm.org>
Fri, 8 Feb 2019 21:25:00 +0000 (13:25 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 12 Feb 2019 16:13:41 +0000 (11:13 -0500)
This patch does not change any functionality.

Cc: Douglas Gilbert <dgilbert@interlog.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Acked-by: Douglas Gilbert <dgilbert@interlog.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/scsi_debug.c

index 90b1b94a7772ed98a34947a75b4638a185414b0b..4c9339ea455de3d37ec60cd37a3e94acd4207a0f 100644 (file)
@@ -1015,7 +1015,7 @@ static int fill_from_dev_buffer(struct scsi_cmnd *scp, unsigned char *arr,
 
        act_len = sg_copy_from_buffer(sdb->table.sgl, sdb->table.nents,
                                      arr, arr_len);
-       sdb->resid = scsi_bufflen(scp) - act_len;
+       scsi_set_resid(scp, scsi_bufflen(scp) - act_len);
 
        return 0;
 }
@@ -1040,9 +1040,10 @@ static int p_fill_from_dev_buffer(struct scsi_cmnd *scp, const void *arr,
        act_len = sg_pcopy_from_buffer(sdb->table.sgl, sdb->table.nents,
                                       arr, arr_len, skip);
        pr_debug("%s: off_dst=%u, scsi_bufflen=%u, act_len=%u, resid=%d\n",
-                __func__, off_dst, scsi_bufflen(scp), act_len, sdb->resid);
+                __func__, off_dst, scsi_bufflen(scp), act_len,
+                scsi_get_resid(scp));
        n = (int)scsi_bufflen(scp) - ((int)off_dst + act_len);
-       sdb->resid = min(sdb->resid, n);
+       scsi_set_resid(scp, min(scsi_get_resid(scp), n));
        return 0;
 }
 
@@ -2768,7 +2769,7 @@ static int resp_read_dt0(struct scsi_cmnd *scp, struct sdebug_dev_info *devip)
        if (unlikely(ret == -1))
                return DID_ERROR << 16;
 
-       scp->sdb.resid = scsi_bufflen(scp) - ret;
+       scsi_set_resid(scp, scsi_bufflen(scp) - ret);
 
        if (unlikely(sqcp)) {
                if (sqcp->inj_recovered) {