diff options
| author | David Sterba <dsterba@suse.com> | 2017-02-13 13:42:29 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2017-02-17 12:03:49 +0100 |
| commit | 53d3235995b3f87754a42de24d707f158c1b3b40 (patch) | |
| tree | a98d862301bdf55fe271c4ebea2fb4825d36b915 /fs/btrfs/extent_io.h | |
| parent | 9d037933861805799db4edfd075412a39e2549f4 (diff) | |
| download | linux-53d3235995b3f87754a42de24d707f158c1b3b40.tar.bz2 | |
btrfs: embed extent_changeset::range_changed to the structure
We can embed range_changed to the extent changeset to address following
problems:
- no need to allocate ulist dynamically, we also get rid of the GFP_NOFS
for free
- fix lack of allocation failure checking in btrfs_qgroup_reserve_data
The stack consuption where extent_changeset is used slightly increases:
before: 16
after: 16 - 8 (for pointer) + 32 (sizeof ulist) = 40
Which is bearable.
Reviewed-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/extent_io.h')
| -rw-r--r-- | fs/btrfs/extent_io.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h index 4551a5b4b8f5..270d03be290e 100644 --- a/fs/btrfs/extent_io.h +++ b/fs/btrfs/extent_io.h @@ -193,7 +193,7 @@ struct extent_changeset { u64 bytes_changed; /* Changed ranges */ - struct ulist *range_changed; + struct ulist range_changed; }; static inline void extent_set_compress_type(unsigned long *bio_flags, |