[5.10,CANDIDATE,0/2] Two more xfs backports for 5.10.y (from v5.11)

Message

Amir Goldstein March 26, 2023, 5:06 p.m. UTC

Darrick,

These two backports were selected by Chandan for 5.4.y, but are
currently missing from 5.10.y.

I've put them through the usual kdevops testing routine.

This is the second time that I have considered patch #2 for 5.10.y.
The last time around, I observed increased the probablity of a known
buffer corruption assertion when running xfs/076, so I suspected
a regression, and dropped it from the submission [1].

At the time, the alleged regression happened only in the kdevops setup
and neither I nor Brain were able to understand why that happens [2].
This time around, the kdevops setup did not observe that odd regression.

Thanks,
Amir.

[1] https://lore.kernel.org/linux-xfs/[e-mail]/
[2] https://lore.kernel.org/linux-xfs/YpY6hUknor2S1iMd@bfoster/T/#mf1add66b8309a75a8984f28ea08718f22033bce7

Brian Foster (1):
  xfs: don't reuse busy extents on extent trim

Darrick J. Wong (1):
  xfs: shut down the filesystem if we screw up quota reservation

 fs/xfs/xfs_extent_busy.c | 14 --------------
 fs/xfs/xfs_trans_dquot.c | 13 ++++++++++---
 2 files changed, 10 insertions(+), 17 deletions(-)

Comments

Darrick J. Wong March 27, 2023, 4:07 p.m. UTC | #1

On Sun, Mar 26, 2023 at 08:06:21PM +0300, Amir Goldstein wrote:
> Darrick,
> 
> These two backports were selected by Chandan for 5.4.y, but are
> currently missing from 5.10.y.
> 
> I've put them through the usual kdevops testing routine.
> 
> This is the second time that I have considered patch #2 for 5.10.y.
> The last time around, I observed increased the probablity of a known
> buffer corruption assertion when running xfs/076, so I suspected
> a regression, and dropped it from the submission [1].
> 
> At the time, the alleged regression happened only in the kdevops setup
> and neither I nor Brain were able to understand why that happens [2].
> This time around, the kdevops setup did not observe that odd regression.

Heh, I had a roommate once whose name got misspelled to Brain on the
water bills, and I still chuckle every time I see that. ;)

Acked-by: Darrick J. Wong <[e-mail]>

--D

> 
> Thanks,
> Amir.
> 
> [1] https://lore.kernel.org/linux-xfs/[e-mail]/
> [2] https://lore.kernel.org/linux-xfs/YpY6hUknor2S1iMd@bfoster/T/#mf1add66b8309a75a8984f28ea08718f22033bce7
> 
> Brian Foster (1):
>   xfs: don't reuse busy extents on extent trim
> 
> Darrick J. Wong (1):
>   xfs: shut down the filesystem if we screw up quota reservation
> 
>  fs/xfs/xfs_extent_busy.c | 14 --------------
>  fs/xfs/xfs_trans_dquot.c | 13 ++++++++++---
>  2 files changed, 10 insertions(+), 17 deletions(-)
> 
> -- 
> 2.34.1
>