diff options
| author | Qu Wenruo <wqu@suse.com> | 2021-08-17 17:38:50 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2021-10-25 21:17:16 +0200 |
| commit | 651fb419273300ebad459630becd839d79439bbc (patch) | |
| tree | 818992dcadc16205f9c203565609ecaead1917d7 /fs/btrfs/subpage.h | |
| parent | fdf250db89b65167891a73327f8cd4f58a323612 (diff) | |
btrfs: subpage: make btrfs_alloc_subpage() return btrfs_subpage directly
The existing calling convention of btrfs_alloc_subpage() is pretty
awful. Change it to a more common pattern by returning struct
btrfs_subpage directly and let the caller to determine if the call
succeeded.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/subpage.h')
| -rw-r--r-- | fs/btrfs/subpage.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/btrfs/subpage.h b/fs/btrfs/subpage.h index 0120948f37a1..fdcadc5193c1 100644 --- a/fs/btrfs/subpage.h +++ b/fs/btrfs/subpage.h @@ -59,9 +59,8 @@ void btrfs_detach_subpage(const struct btrfs_fs_info *fs_info, struct page *page); /* Allocate additional data where page represents more than one sector */ -int btrfs_alloc_subpage(const struct btrfs_fs_info *fs_info, - struct btrfs_subpage **ret, - enum btrfs_subpage_type type); +struct btrfs_subpage *btrfs_alloc_subpage(const struct btrfs_fs_info *fs_info, + enum btrfs_subpage_type type); void btrfs_free_subpage(struct btrfs_subpage *subpage); void btrfs_page_inc_eb_refs(const struct btrfs_fs_info *fs_info, |