From: Filipe Manana Date: Tue, 20 Jul 2021 15:03:42 +0000 (+0100) Subject: btrfs: remove unnecessary list head initialization when syncing log X-Git-Tag: accepted/tizen/unified/20230118.172025~6109^2~116 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e68107e51f8466e1fae40d64b873d0a11398a628;p=platform%2Fkernel%2Flinux-rpi.git btrfs: remove unnecessary list head initialization when syncing log One of the last steps of syncing the log is to remove all log contexts from the root's list of contexts, done at btrfs_remove_all_log_ctxs(). There we iterate over all the contexts in the list and delete each one from the list, and after that we call INIT_LIST_HEAD() on the list. That is unnecessary since at that point the list is empty. So just remove the INIT_LIST_HEAD() call. It's not needed, increases code size (bloat-o-meter reported a delta of -122 for btrfs_sync_log() after this change) and increases two critical sections delimited by log mutexes. Reviewed-by: Josef Bacik Signed-off-by: Filipe Manana Signed-off-by: David Sterba --- diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index a7ce23b..3e6c8f8 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c @@ -3039,8 +3039,6 @@ static inline void btrfs_remove_all_log_ctxs(struct btrfs_root *root, list_del_init(&ctx->list); ctx->log_ret = error; } - - INIT_LIST_HEAD(&root->log_ctxs[index]); } /*