Re: [PATCH 2/4] xfs: simplify xfs_rmap_lookup_le call sites

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux