On Fri, Sep 29, 2023 at 01:22:59PM +0800, Zorro Lang wrote: > On Mon, Sep 25, 2023 at 02:42:56PM -0700, Darrick J. Wong wrote: > > From: Darrick J. Wong <djwong@xxxxxxxxxx> > > > > Update the commit id in the _fixed_by tag now that we've merged the > > kernel fix. > > > > Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx> > > --- > > tests/xfs/270 | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/tests/xfs/270 b/tests/xfs/270 > > index 7d4e1f6a87..4e4f767dc1 100755 > > --- a/tests/xfs/270 > > +++ b/tests/xfs/270 > > @@ -17,7 +17,7 @@ _begin_fstest auto quick mount > > > > # real QA test starts here > > _supported_fs xfs > > -_fixed_by_kernel_commit xxxxxxxxxxxx \ > > +_fixed_by_kernel_commit 74ad4693b647 \ > > "xfs: fix log recovery when unknown rocompat bits are set" > > This patch is good to me, but we have more xfs cases have fixed commit which > have been merged: > > $ grep -rsni xxxxxxxx tests/xfs|grep _fixed > tests/xfs/600:23:_fixed_by_git_commit kernel XXXXXXXXXXXXX \ > tests/xfs/557:21:_fixed_by_kernel_commit XXXXXXXXXXXX \ > tests/xfs/270:20:_fixed_by_kernel_commit xxxxxxxxxxxx \ > > xfs/600: cfa2df68b7ce xfs: fix an agbno overflow in __xfs_getfsmap_datadev > xfs/557: 817644fa4525 xfs: get root inode correctly at bulkstat > > Do you want to fix them in one patch, or you hope to merge this > patch at first? Eh, I'll just resubmit with all three. --D > Thanks, > Zorro > > > > # skip fs check because superblock contains unknown ro-compat features > > _require_scratch_nocheck > > >
