btrfs: remove unnecessary extent map initializations
authorFilipe Manana <fdmanana@suse.com>
Mon, 19 Sep 2022 14:06:35 +0000 (15:06 +0100)
committerDavid Sterba <dsterba@suse.com>
Thu, 29 Sep 2022 15:08:30 +0000 (17:08 +0200)
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>
fs/btrfs/extent_map.c

index 2e6dc5a..6b7eee9 100644 (file)
@@ -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;