On Wed, Dec 13, 2017 at 03:58:25PM -0800, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Since __xfs_dir3_data_check verifies on-disk metadata, we can't have it > noisily blowing asserts and hanging the system on corrupt data coming in > off the disk. Instead, have it return a boolean like all the other > checker functions, and only have it noisily fail if we fail in debug > mode. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Looks fine, though we do lose fine-grained debug information here. I guess we get that back from the conversion to _this_address as the return value, though... Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
