Re: [xfs-linux:xfs-7.3-merge 37/54] fs/xfs/libxfs/xfs_sb.c:1473:34: warning: variable 'bp' set but not used

On 7/22/26 04:35, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git xfs-7.3-merge
> head:   97f34168769238b1fe4412229c99f1a78364f6f9
> commit: b9ad0b9891dd0ddbfd8d9bfc7a55b0b70e9871f2 [37/54] xfs: don't hold buffer locks across sync transaction commit in xfs_sync_sb_buf
> config: x86_64-rhel-9.4-bpf (https://download.01.org/0day-ci/archive/20260721/[email protected]/config)
> compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260721/[email protected]/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <[email protected]>
> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/
> 
> All warnings (new ones prefixed by >>):
> 
>     In file included from fs/xfs/libxfs/xfs_rtbitmap.h:9,
>                      from fs/xfs/libxfs/xfs_sb.c:28:
>     fs/xfs/libxfs/xfs_sb.c: In function 'xfs_sync_sb_buf':
>>> fs/xfs/libxfs/xfs_rtgroup.h:362:41: warning: statement with no effect [-Wunused-value]
>       362 | # define xfs_log_rtsb(tp, sb_bp)        (NULL)
>           |                                         ^
>     fs/xfs/libxfs/xfs_sb.c:1483:17: note: in expansion of macro 'xfs_log_rtsb'
>      1483 |                 xfs_log_rtsb(tp, bp);
>           |                 ^~~~~~~~~~~~
>>> fs/xfs/libxfs/xfs_sb.c:1473:34: warning: variable 'bp' set but not used [-Wunused-but-set-variable]
>      1473 |         struct xfs_buf          *bp;
>           |                                  ^~
> 
> 
My apologies for missing the CONFIG_XFS_RT=n case. A fix is ready and 
currently being tested. I will send it out shortly.