pmem: make_request cleanups
authorDan Williams <dan.j.williams@intel.com>
Sat, 16 May 2015 16:28:51 +0000 (12:28 -0400)
committerDan Williams <dan.j.williams@intel.com>
Fri, 26 Jun 2015 15:23:38 +0000 (11:23 -0400)
Various cleanups:

1/ Kill the BUG_ON since we've already told the block layer we don't
   support DISCARD on all these drivers.

2/ Kill the 'rw' variable, no need to cache it.

3/ Kill the local 'sector' variable.  bio_for_each_segment() is already
   advancing the iterator's sector number by the bio_vec length.

4/ Kill the check for accessing past the end of device
   generic_make_request_checks() already does that.

Suggested-by: Christoph Hellwig <hch@lst.de>
[hch: kill access past end of the device check]
Reviewed-by: Vishal Verma <vishal.l.verma@linux.intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/nvdimm/pmem.c

index d29a42adb95a8fcfb776be449b2634bdf36dad10..e846a627ebdf9952352c4db1b7aa69988ac1a57f 100644 (file)
@@ -58,31 +58,15 @@ static void pmem_do_bvec(struct pmem_device *pmem, struct page *page,
 
 static void pmem_make_request(struct request_queue *q, struct bio *bio)
 {
-       struct block_device *bdev = bio->bi_bdev;
-       struct pmem_device *pmem = bdev->bd_disk->private_data;
-       int rw;
        struct bio_vec bvec;
-       sector_t sector;
        struct bvec_iter iter;
-       int err = 0;
-
-       if (bio_end_sector(bio) > get_capacity(bdev->bd_disk)) {
-               err = -EIO;
-               goto out;
-       }
-
-       BUG_ON(bio->bi_rw & REQ_DISCARD);
+       struct block_device *bdev = bio->bi_bdev;
+       struct pmem_device *pmem = bdev->bd_disk->private_data;
 
-       rw = bio_data_dir(bio);
-       sector = bio->bi_iter.bi_sector;
-       bio_for_each_segment(bvec, bio, iter) {
+       bio_for_each_segment(bvec, bio, iter)
                pmem_do_bvec(pmem, bvec.bv_page, bvec.bv_len, bvec.bv_offset,
-                            rw, sector);
-               sector += bvec.bv_len >> 9;
-       }
-
-out:
-       bio_endio(bio, err);
+                               bio_data_dir(bio), iter.bi_sector);
+       bio_endio(bio, 0);
 }
 
 static int pmem_rw_page(struct block_device *bdev, sector_t sector,