diff options
| author | Filipe Manana <[email protected]> | 2025-05-29 15:12:45 +0000 |
|---|---|---|
| committer | David Sterba <[email protected]> | 2025-07-21 21:53:29 +0000 |
| commit | 1ed0cfc89e992112b5e5f5677c17081e0eee688d (patch) | |
| tree | 17ffbe5d07f176b0d010bfd1fb38908ca41cc77c | |
| parent | btrfs: free path sooner at __btrfs_unlink_inode() (diff) | |
| download | kernel-1ed0cfc89e992112b5e5f5677c17081e0eee688d.tar.gz kernel-1ed0cfc89e992112b5e5f5677c17081e0eee688d.zip | |
btrfs: use btrfs_del_item() at del_logged_dentry()
There's no need to use btrfs_delete_one_dir_name() at del_logged_dentry()
because we are processing a dir index key which can contain only a single
name, unlike dir item keys which can encode multiple names in case of name
hash collisions. We have explicitly looked up for a dir index key by
calling btrfs_lookup_dir_index_item() and we don't log dir item keys
anymore (since commit 339d03542484 ("btrfs: only copy dir index keys when
logging a directory")). So simplify and use btrfs_del_item() directly
instead of btrfs_delete_one_dir_name().
Reviewed-by: Boris Burkov <[email protected]>
Signed-off-by: Filipe Manana <[email protected]>
Reviewed-by: David Sterba <[email protected]>
Signed-off-by: David Sterba <[email protected]>
| -rw-r--r-- | fs/btrfs/tree-log.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 9420568131ab..cb20f124812a 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c @@ -3432,7 +3432,7 @@ static int del_logged_dentry(struct btrfs_trans_handle *trans, * inode item because on log replay we update the field to reflect * all existing entries in the directory (see overwrite_item()). */ - return btrfs_delete_one_dir_name(trans, log, path, di); + return btrfs_del_item(trans, log, path); } /* |