On Sun, Oct 02, 2022 at 11:20:08AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > sparse complains that we can return an uninitialized error from this > function and that pag could be uninitialized. We know that there are no > zero-AG filesystems and hence we had to call xchk_bmap_check_ag_rmaps at > least once, so this is not actually possible, but I'm too worn out from > automated complaints from unsophisticated AIs so let's just fix this and > move on to more interesting problems, eh? > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > --- > fs/xfs/scrub/bmap.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) Looks fine. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx