On Thu, Apr 14, 2022 at 03:54:10PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > Most callers of xfs_rmap_lookup_le will retrieve the btree record > immediately if the lookup succeeds. The overlapped version of this > function (xfs_rmap_lookup_le_range) will return the record if the lookup > succeeds, so make the regular version do it too. Get rid of the useless > len argument, since it's not part of the lookup key. > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_rmap.c | 59 +++++++++++++++++----------------------------- > fs/xfs/libxfs/xfs_rmap.h | 4 ++- > fs/xfs/scrub/bmap.c | 24 +++---------------- > 3 files changed, 28 insertions(+), 59 deletions(-) Nice cleanup. Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx