block: consistently use GFP_NOIO instead of __GFP_NORECLAIM

Same numerical value (for now at least), but a much better documentation
of intent.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/block/scsi_ioctl.c b/block/scsi_ioctl.c
index 04b54f9a415245c0c9c7beacdd41b98deaed3043..533f4aee8567a0f434ab268089652302b098f312 100644 (file)
--- a/block/scsi_ioctl.c
+++ b/block/scsi_ioctl.c
@@ -499,7 +499,7 @@ int sg_scsi_ioctl(struct request_queue *q, struct gendisk *disk, fmode_t mode,
                break;
        }
 
-       if (bytes && blk_rq_map_kern(q, rq, buffer, bytes, __GFP_RECLAIM)) {
+       if (bytes && blk_rq_map_kern(q, rq, buffer, bytes, GFP_NOIO)) {
                err = DRIVER_ERROR << 24;
                goto error;
        }

diff --git a/drivers/block/drbd/drbd_bitmap.c b/drivers/block/drbd/drbd_bitmap.c
index 9f4e6f502b8460e88fb3a2fb1df2cac31f5edaf8..d82237d534cf058a47c6cc7a13b7bb478c3c69bc 100644 (file)
--- a/drivers/block/drbd/drbd_bitmap.c
+++ b/drivers/block/drbd/drbd_bitmap.c
@@ -1014,7 +1014,8 @@ static void bm_page_io_async(struct drbd_bm_aio_ctx *ctx, int page_nr) __must_ho
        bm_set_page_unchanged(b->bm_pages[page_nr]);
 
        if (ctx->flags & BM_AIO_COPY_PAGES) {
-               page = mempool_alloc(drbd_md_io_page_pool, __GFP_HIGHMEM|__GFP_RECLAIM);
+               page = mempool_alloc(drbd_md_io_page_pool,
+                               GFP_NOIO | __GFP_HIGHMEM);
                copy_highpage(page, b->bm_pages[page_nr]);
                bm_store_page_idx(page, page_nr);
        } else

diff --git a/drivers/block/pktcdvd.c b/drivers/block/pktcdvd.c
index 4880a4a9f52d488749d2766bcce1d08cc72cbd92..ccfcf544830fc398d271ddf056d7667d2737792e 100644 (file)
--- a/drivers/block/pktcdvd.c
+++ b/drivers/block/pktcdvd.c
@@ -710,7 +710,7 @@ static int pkt_generic_packet(struct pktcdvd_device *pd, struct packet_command *
 
        if (cgc->buflen) {
                ret = blk_rq_map_kern(q, rq, cgc->buffer, cgc->buflen,
-                                     __GFP_RECLAIM);
+                                     GFP_NOIO);
                if (ret)
                        goto out;
        }

diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 66661031f3f17906496b691c610ddafced970695..62c1a19a9aed4b25a6506bd407ae46f70e1ae9f4 100644 (file)
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -862,7 +862,7 @@ static int idetape_queue_rw_tail(ide_drive_t *drive, int cmd, int size)
 
        if (size) {
                ret = blk_rq_map_kern(drive->queue, rq, tape->buf, size,
-                                     __GFP_RECLAIM);
+                                     GFP_NOIO);
                if (ret)
                        goto out_put;
        }

diff --git a/drivers/ide/ide-taskfile.c b/drivers/ide/ide-taskfile.c
index 6308bb0dab5097bc6932caaf598a6307a93c82da..c034cd965831801273d1f88128a0ebfc5a4c64b4 100644 (file)
--- a/drivers/ide/ide-taskfile.c
+++ b/drivers/ide/ide-taskfile.c
@@ -442,7 +442,7 @@ int ide_raw_taskfile(ide_drive_t *drive, struct ide_cmd *cmd, u8 *buf,
         */
        if (nsect) {
                error = blk_rq_map_kern(drive->queue, rq, buf,
-                                       nsect * SECTOR_SIZE, __GFP_RECLAIM);
+                                       nsect * SECTOR_SIZE, GFP_NOIO);
                if (error)
                        goto put_req;
        }

diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
index 6b0f3ec487bd8fdaaad6199cba6fabd03b9e3ada..f125fd71c0f23e0a69d33d0353258a07946dd78f 100644 (file)
--- a/drivers/scsi/scsi_lib.c
+++ b/drivers/scsi/scsi_lib.c
@@ -273,7 +273,7 @@ int scsi_execute(struct scsi_device *sdev, const unsigned char *cmd,
        rq = scsi_req(req);
 
        if (bufflen &&  blk_rq_map_kern(sdev->request_queue, req,
-                                       buffer, bufflen, __GFP_RECLAIM))
+                                       buffer, bufflen, GFP_NOIO))
                goto out;
 
        rq->cmd_len = COMMAND_SIZE(cmd[0]);

diff --git a/fs/direct-io.c b/fs/direct-io.c
index 874607bb6e02134b8b185f333537a134d791d85d..093fb54cd3163d96b03ebf3b0bea4a83a26bb02a 100644 (file)
--- a/fs/direct-io.c
+++ b/fs/direct-io.c
@@ -432,8 +432,8 @@ dio_bio_alloc(struct dio *dio, struct dio_submit *sdio,
        struct bio *bio;
 
        /*
-        * bio_alloc() is guaranteed to return a bio when called with
-        * __GFP_RECLAIM and we request a valid number of vectors.
+        * bio_alloc() is guaranteed to return a bio when allowed to sleep and
+        * we request a valid number of vectors.
         */
        bio = bio_alloc(GFP_KERNEL, nr_vecs);
 

diff --git a/kernel/power/swap.c b/kernel/power/swap.c
index 11b4282c2d2031d8208d53917f78d3aec5323bf0..1efcb5b0c3ed422aa20b86f5980bdeb9ba26493b 100644 (file)
--- a/kernel/power/swap.c
+++ b/kernel/power/swap.c
@@ -269,7 +269,7 @@ static int hib_submit_io(int op, int op_flags, pgoff_t page_off, void *addr,
        struct bio *bio;
        int error = 0;
 
-       bio = bio_alloc(__GFP_RECLAIM | __GFP_HIGH, 1);
+       bio = bio_alloc(GFP_NOIO | __GFP_HIGH, 1);
        bio->bi_iter.bi_sector = page_off * (PAGE_SIZE >> 9);
        bio_set_dev(bio, hib_resume_bdev);
        bio_set_op_attrs(bio, op, op_flags);
@@ -376,7 +376,7 @@ static int write_page(void *buf, sector_t offset, struct hib_bio_batch *hb)
                return -ENOSPC;
 
        if (hb) {
-               src = (void *)__get_free_page(__GFP_RECLAIM | __GFP_NOWARN |
+               src = (void *)__get_free_page(GFP_NOIO | __GFP_NOWARN |
                                              __GFP_NORETRY);
                if (src) {
                        copy_page(src, buf);
@@ -384,7 +384,7 @@ static int write_page(void *buf, sector_t offset, struct hib_bio_batch *hb)
                        ret = hib_wait_io(hb); /* Free pages */
                        if (ret)
                                return ret;
-                       src = (void *)__get_free_page(__GFP_RECLAIM |
+                       src = (void *)__get_free_page(GFP_NOIO |
                                                      __GFP_NOWARN |
                                                      __GFP_NORETRY);
                        if (src) {
@@ -691,7 +691,7 @@ static int save_image_lzo(struct swap_map_handle *handle,
        nr_threads = num_online_cpus() - 1;
        nr_threads = clamp_val(nr_threads, 1, LZO_THREADS);
 
-       page = (void *)__get_free_page(__GFP_RECLAIM | __GFP_HIGH);
+       page = (void *)__get_free_page(GFP_NOIO | __GFP_HIGH);
        if (!page) {
                pr_err("Failed to allocate LZO page\n");
                ret = -ENOMEM;
@@ -989,7 +989,7 @@ static int get_swap_reader(struct swap_map_handle *handle,
                last = tmp;
 
                tmp->map = (struct swap_map_page *)
-                          __get_free_page(__GFP_RECLAIM | __GFP_HIGH);
+                          __get_free_page(GFP_NOIO | __GFP_HIGH);
                if (!tmp->map) {
                        release_swap_reader(handle);
                        return -ENOMEM;
@@ -1261,8 +1261,8 @@ static int load_image_lzo(struct swap_map_handle *handle,
 
        for (i = 0; i < read_pages; i++) {
                page[i] = (void *)__get_free_page(i < LZO_CMP_PAGES ?
-                                                 __GFP_RECLAIM | __GFP_HIGH :
-                                                 __GFP_RECLAIM | __GFP_NOWARN |
+                                                 GFP_NOIO | __GFP_HIGH :
+                                                 GFP_NOIO | __GFP_NOWARN |
                                                  __GFP_NORETRY);
 
                if (!page[i]) {