scsi: move the nr_phys_segments assert into scsi_init_io
authorChristoph Hellwig <hch@lst.de>
Sat, 28 Jun 2014 09:51:01 +0000 (11:51 +0200)
committerChristoph Hellwig <hch@lst.de>
Thu, 17 Jul 2014 20:10:53 +0000 (22:10 +0200)
scsi_init_io should only be called for requests that transfer data,
so move the assert that a request has segments from the callers into
scsi_init_io.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Martin K. Petersen <martin.petersen@oracle.com>
Reviewed-by: Hannes Reinecke <hare@suse.de>
drivers/scsi/scsi_lib.c

index 03076b2b5debf07915a0e3635b2becdf473d17d9..150d7bb675a1ce505e475edcf2cdbb7c721f4d76 100644 (file)
@@ -963,8 +963,11 @@ int scsi_init_io(struct scsi_cmnd *cmd, gfp_t gfp_mask)
 {
        struct scsi_device *sdev = cmd->device;
        struct request *rq = cmd->request;
+       int error;
 
-       int error = scsi_init_sgtable(rq, &cmd->sdb, gfp_mask);
+       BUG_ON(!rq->nr_phys_segments);
+
+       error = scsi_init_sgtable(rq, &cmd->sdb, gfp_mask);
        if (error)
                goto err_exit;
 
@@ -1055,11 +1058,7 @@ int scsi_setup_blk_pc_cmnd(struct scsi_device *sdev, struct request *req)
         * submit a request without an attached bio.
         */
        if (req->bio) {
-               int ret;
-
-               BUG_ON(!req->nr_phys_segments);
-
-               ret = scsi_init_io(cmd, GFP_ATOMIC);
+               int ret = scsi_init_io(cmd, GFP_ATOMIC);
                if (unlikely(ret))
                        return ret;
        } else {
@@ -1098,11 +1097,6 @@ int scsi_setup_fs_cmnd(struct scsi_device *sdev, struct request *req)
                        return ret;
        }
 
-       /*
-        * Filesystem requests must transfer data.
-        */
-       BUG_ON(!req->nr_phys_segments);
-
        memset(cmd->cmnd, 0, BLK_MAX_CDB);
        return scsi_init_io(cmd, GFP_ATOMIC);
 }