From: Long Li <leo.lilong@huawei.com>
Date: Mon, 30 Sep 2024 10:42:17 +0000 (+0800)
Subject: xfs: remove the redundant xfs_alloc_log_agf
X-Git-Tag: v6.13-rc1~133^2~16
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=8b9b261594d8ef218ef4d0e732dad153f82aab49;p=thirdparty%2Fkernel%2Flinux.git

xfs: remove the redundant xfs_alloc_log_agf

There are two invocations of xfs_alloc_log_agf in xfs_alloc_put_freelist.
The AGF does not change between the two calls. Although this does not pose
any practical problems, it seems like a small mistake. Therefore, fix it
by removing the first xfs_alloc_log_agf invocation.

Signed-off-by: Long Li <leo.lilong@huawei.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
---

diff --git a/fs/xfs/libxfs/xfs_alloc.c b/fs/xfs/libxfs/xfs_alloc.c
index 22bdbb3e9980c..68643a3592ac0 100644
--- a/fs/xfs/libxfs/xfs_alloc.c
+++ b/fs/xfs/libxfs/xfs_alloc.c
@@ -3156,8 +3156,6 @@ xfs_alloc_put_freelist(
 		logflags |= XFS_AGF_BTREEBLKS;
 	}
 
-	xfs_alloc_log_agf(tp, agbp, logflags);
-
 	ASSERT(be32_to_cpu(agf->agf_flcount) <= xfs_agfl_size(mp));
 
 	agfl_bno = xfs_buf_to_agfl_bno(agflbp);