From 29d9c5e0370593cec8298601ab3e9ce29799fb6f Mon Sep 17 00:00:00 2001 From: Filipe Manana Date: Mon, 1 Sep 2025 12:43:42 +0100 Subject: [PATCH] btrfs: avoid unnecessary path allocation at fixup_inode_link_count() There's no need to allocate a path as our single caller already has a path that we can use. So pass the caller's path and use it. Signed-off-by: Filipe Manana Reviewed-by: David Sterba Signed-off-by: David Sterba --- fs/btrfs/tree-log.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index b4e901da9e8ba..5754333ae732d 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c @@ -1728,19 +1728,15 @@ process_slot: * will free the inode. */ static noinline int fixup_inode_link_count(struct walk_control *wc, + struct btrfs_path *path, struct btrfs_inode *inode) { struct btrfs_trans_handle *trans = wc->trans; struct btrfs_root *root = inode->root; - struct btrfs_path *path; int ret; u64 nlink = 0; const u64 ino = btrfs_ino(inode); - path = btrfs_alloc_path(); - if (!path) - return -ENOMEM; - ret = count_inode_refs(inode, path); if (ret < 0) goto out; @@ -1776,7 +1772,7 @@ static noinline int fixup_inode_link_count(struct walk_control *wc, } out: - btrfs_free_path(path); + btrfs_release_path(path); return ret; } @@ -1821,7 +1817,7 @@ static noinline int fixup_inode_link_counts(struct walk_control *wc, break; } - ret = fixup_inode_link_count(wc, inode); + ret = fixup_inode_link_count(wc, path, inode); iput(&inode->vfs_inode); if (ret) break; -- 2.47.3