On Sun, Jun 05, 2022 at 09:40:03AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > The variable @args is fed to a tracepoint, and that's the only place > it's used. This is fine for the kernel, but for userspace, tracepoints > are #define'd out of existence, which results in this warning on gcc > 11.2: > > xfs_attr.c: In function ‘xfs_attr_node_try_addname’: > xfs_attr.c:1440:42: warning: unused variable ‘args’ [-Wunused-variable] > 1440 | struct xfs_da_args *args = attr->xattri_da_args; > | ^~~~ > > Clean this up. > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_attr.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Yeah, I noticed this in the xfsprogs libxfs port and fixed it there on the port by converting the xfs_attr3_leaf_add(... state->args) parameter to use args. This way works too, and the xfsprogs libxfs can easily be cleaned up in the next xfsprogs libxfs sync. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx
