]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
btrfs: remove pointless empty log context list check when syncing log
authorFilipe Manana <fdmanana@suse.com>
Thu, 5 Oct 2023 11:11:09 +0000 (12:11 +0100)
committerDavid Sterba <dsterba@suse.com>
Thu, 12 Oct 2023 14:44:18 +0000 (16:44 +0200)
When syncing the log, if we get an error when updating the log root, we
check first if the log root tree context is in a log context list, and if
so it deletes from the log root tree context from the list. This check
however is pointless because at this moment the context is always in a
list, he have just added it to a context list. The check became pointless
after commit a93e01682e28 ("btrfs: remove no longer needed use of
log_writers for the log root tree"). So remove this now pointless empty
list check.

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/tree-log.c

index 958bb23d3d992014c3d45f1c885cac1911772681..8b3893c01734bb291ca9004c99af6415890cd6d7 100644 (file)
@@ -2996,9 +2996,7 @@ int btrfs_sync_log(struct btrfs_trans_handle *trans,
         */
        ret = update_log_root(trans, log, &new_root_item);
        if (ret) {
-               if (!list_empty(&root_log_ctx.list))
-                       list_del_init(&root_log_ctx.list);
-
+               list_del_init(&root_log_ctx.list);
                blk_finish_plug(&plug);
                btrfs_set_log_full_commit(trans);
                if (ret != -ENOSPC)