diff options
| author |   Filipe Manana <fdmanana@suse.com> | 2022-09-19 15:06:35 +0100 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2022-09-29 17:08:30 +0200 |
| commit | 2e0cdaa0288bbe3d6a05937bc7b61aa9da0cb2bf (patch) | |
| tree | 9747cea97640d4c0141a58d9f1dae37efe5521ad /fs/btrfs | |
| parent | btrfs: remove the refcount warning/check at free_extent_map() (diff) | |
| download | linux-dev-2e0cdaa0288bbe3d6a05937bc7b61aa9da0cb2bf.tar.xz linux-dev-2e0cdaa0288bbe3d6a05937bc7b61aa9da0cb2bf.zip | |
btrfs: remove unnecessary extent map initializations
When allocating an extent map, we use kmem_cache_zalloc() which guarantees
the returned memory is initialized to zeroes, therefore it's pointless
to initialize the generation and flags of the extent map to zero again.
Remove those initializations, as they are pointless and slightly increase
the object text size.
Before removing them:
$ size fs/btrfs/extent_map.o
text data bss dec hex filename
9241 274 24 9539 2543 fs/btrfs/extent_map.o
After removing them:
$ size fs/btrfs/extent_map.o
text data bss dec hex filename
9209 274 24 9507 2523 fs/btrfs/extent_map.o
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/extent_map.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c index 2e6dc5a772f4..6b7eee92d981 100644 --- a/fs/btrfs/extent_map.c +++ b/fs/btrfs/extent_map.c @@ -55,9 +55,7 @@ struct extent_map *alloc_extent_map(void) if (!em) return NULL; RB_CLEAR_NODE(&em->rb_node); - em->flags = 0; em->compress_type = BTRFS_COMPRESS_NONE; - em->generation = 0; refcount_set(&em->refs, 1); INIT_LIST_HEAD(&em->list); return em; |