On Mon, Jun 04, 2018 at 09:10:28AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Create a variant of xfs_dir2_data_freefind that is suitable for use in a > verifier. Because _freefind is called by the verifier, we simply > duplicate the _freefind function, convert the ASSERTs to return > __this_address, and modify the verifier to call our new function. Once > we've made it impossible for directory blocks with bad bestfree data to > make it into the filesystem we can remove the DEBUG code from the > regular _freefind function. > > Underlying argument: corruption of on-disk metadata should return > -EFSCORRUPTED instead of blowing ASSERTs. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> Looks good. 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
