Re: [PATCH] xfs: fix extent busy updating

["Darrick J. Wong" <djwong@kernel.org>]

On Tue, Jan 03, 2023 at 11:32:17AM -0800, Wengang Wang wrote:
> In xfs_extent_busy_update_extent() case 6 and 7, whenever bno is modified on
> extent busy, the relavent length has to be modified accordingly.
> 
> Signed-off-by: Wengang Wang <wen.gang.wang@oracle.com>
> ---
>  fs/xfs/xfs_extent_busy.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/fs/xfs/xfs_extent_busy.c b/fs/xfs/xfs_extent_busy.c
> index ad22a003f959..f3d328e4a440 100644
> --- a/fs/xfs/xfs_extent_busy.c
> +++ b/fs/xfs/xfs_extent_busy.c
> @@ -236,6 +236,7 @@ xfs_extent_busy_update_extent(
>  		 *
>  		 */
>  		busyp->bno = fend;
> +		busyp->length = bend - fend;

Looks correct to me, but how did you find this?  Is there some sort of
test case we could attach to this?

--D

>  	} else if (bbno < fbno) {
>  		/*
>  		 * Case 8:
> -- 
> 2.21.0 (Apple Git-122.2)
>