diff options
author | Jens Axboe <axboe@kernel.dk> | 2017-09-27 05:40:16 -0600 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2017-10-03 08:38:17 -0600 |
commit | 640ab98fb3629c0f8417b9b2532eca596495f3bb (patch) | |
tree | 59f6d84e07327e6b94473bcd66e7366582b7f85b /include | |
parent | 7beb2f845b715cb98584cf630e9a9d5b05501166 (diff) | |
download | linux-640ab98fb3629c0f8417b9b2532eca596495f3bb.tar.gz linux-640ab98fb3629c0f8417b9b2532eca596495f3bb.tar.bz2 linux-640ab98fb3629c0f8417b9b2532eca596495f3bb.zip |
buffer: have alloc_page_buffers() use __GFP_NOFAIL
Instead of adding weird retry logic in that function, utilize
__GFP_NOFAIL to ensure that the vm takes care of handling any
potential retries appropriately. This means we don't have to
call free_more_memory() from here.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/buffer_head.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/buffer_head.h b/include/linux/buffer_head.h index c8dae555eccf..ae2d25f01b98 100644 --- a/include/linux/buffer_head.h +++ b/include/linux/buffer_head.h @@ -156,7 +156,7 @@ void set_bh_page(struct buffer_head *bh, struct page *page, unsigned long offset); int try_to_free_buffers(struct page *); struct buffer_head *alloc_page_buffers(struct page *page, unsigned long size, - int retry); + bool retry); void create_empty_buffers(struct page *, unsigned long, unsigned long b_state); void end_buffer_read_sync(struct buffer_head *bh, int uptodate); |