diff options
| author | Sweet Tea Dorminy <sweettea-kernel@dorminy.me> | 2022-03-30 16:11:22 -0400 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2022-05-16 17:03:11 +0200 |
| commit | dd137dd1f2d719682b522d4eabe6dec461b7d6fa (patch) | |
| tree | bc566f7f1b070e24604cc790070ee1955c80572b /fs/btrfs/check-integrity.c | |
| parent | 0d031dc4aa05819beb8b9188f4306a3f2bc17f55 (diff) | |
btrfs: factor out allocating an array of pages
Several functions currently populate an array of page pointers one
allocated page at a time. Factor out the common code so as to allow
improvements to all of the sites at once.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Sweet Tea Dorminy <sweettea-kernel@dorminy.me>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/check-integrity.c')
| -rw-r--r-- | fs/btrfs/check-integrity.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/fs/btrfs/check-integrity.c b/fs/btrfs/check-integrity.c index 62eb149aac98..b8f9dfa32620 100644 --- a/fs/btrfs/check-integrity.c +++ b/fs/btrfs/check-integrity.c @@ -1552,11 +1552,9 @@ static int btrfsic_read_block(struct btrfsic_state *state, return -ENOMEM; block_ctx->datav = block_ctx->mem_to_free; block_ctx->pagev = (struct page **)(block_ctx->datav + num_pages); - for (i = 0; i < num_pages; i++) { - block_ctx->pagev[i] = alloc_page(GFP_NOFS); - if (!block_ctx->pagev[i]) - return -1; - } + ret = btrfs_alloc_page_array(num_pages, block_ctx->pagev); + if (ret) + return ret; dev_bytenr = block_ctx->dev_bytenr; for (i = 0; i < num_pages;) { |