block: use bio_release_pages in bio_unmap_user
authorChristoph Hellwig <hch@lst.de>
Wed, 26 Jun 2019 13:49:22 +0000 (15:49 +0200)
committerJens Axboe <axboe@kernel.dk>
Sat, 29 Jun 2019 15:47:31 +0000 (09:47 -0600)
Use bio_release_pages instead of open coding it.

Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Reviewed-by: Minwoo Im <minwoo.im.dev@gmail.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/bio.c

index 8a7b315630ce7e4fec35d973c06745e1c21c1368..c759f55985139da70dfff1f3da74efa20956de16 100644 (file)
@@ -1437,24 +1437,6 @@ struct bio *bio_map_user_iov(struct request_queue *q,
        return ERR_PTR(ret);
 }
 
-static void __bio_unmap_user(struct bio *bio)
-{
-       struct bio_vec *bvec;
-       struct bvec_iter_all iter_all;
-
-       /*
-        * make sure we dirty pages we wrote to
-        */
-       bio_for_each_segment_all(bvec, bio, iter_all) {
-               if (bio_data_dir(bio) == READ)
-                       set_page_dirty_lock(bvec->bv_page);
-
-               put_page(bvec->bv_page);
-       }
-
-       bio_put(bio);
-}
-
 /**
  *     bio_unmap_user  -       unmap a bio
  *     @bio:           the bio being unmapped
@@ -1466,7 +1448,8 @@ static void __bio_unmap_user(struct bio *bio)
  */
 void bio_unmap_user(struct bio *bio)
 {
-       __bio_unmap_user(bio);
+       bio_release_pages(bio, bio_data_dir(bio) == READ);
+       bio_put(bio);
        bio_put(bio);
 }