OSDN Git Service

btrfs: remove redundant log root assignment from log_dir_items()
authorFilipe Manana <fdmanana@suse.com>
Thu, 16 Sep 2021 10:32:11 +0000 (11:32 +0100)
committerDavid Sterba <dsterba@suse.com>
Tue, 26 Oct 2021 17:08:02 +0000 (19:08 +0200)
At log_dir_items() we are assigning the exact same value to the local
variable 'log', once when it's declared and once again shortly after.
Remove the later assignment as it's pointless.

This patch is part of a patchset comprised of the following 5 patches:

  btrfs: remove root argument from btrfs_log_inode() and its callees
  btrfs: remove redundant log root assignment from log_dir_items()
  btrfs: factor out the copying loop of dir items from log_dir_items()
  btrfs: insert items in batches when logging a directory when possible
  btrfs: keep track of the last logged keys when logging a directory

This is patch 2/5. The change log of the last patch (5/5) has performance
results.

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 50f6b84..a7018eb 100644 (file)
@@ -3656,8 +3656,6 @@ static noinline int log_dir_items(struct btrfs_trans_handle *trans,
        u64 last_offset = (u64)-1;
        u64 ino = btrfs_ino(inode);
 
-       log = root->log_root;
-
        min_key.objectid = ino;
        min_key.type = key_type;
        min_key.offset = min_offset;