On Wed, Jun 22, 2022 at 09:35:19AM -0700, Darrick J. Wong wrote: > On Wed, Jun 22, 2022 at 09:23:07AM -0700, Darrick J. Wong wrote: > > On Thu, Jun 16, 2022 at 11:27:41AM -0700, Leah Rumancik wrote: > > > The patch testing has been increased to 100 runs per test on each > > > config. A baseline without the patches was established with 100 runs > > > to help detect hard failures / tests with a high fail rate. Any > > > failures seen in the backports branch but not in the baseline branch > > > were then run 1000+ times on both the baseline and backport branches > > > and the failure rates compared. The failures seen on the 5.15 > > > baseline are listed at > > > https://gist.github.com/lrumancik/5a9d85d2637f878220224578e173fc23. > > > No regressions were seen with these patches. > > > > > > To make the review process easier, I have been coordinating with Amir > > > who has been testing this same set of patches on 5.10. He will be > > > sending out the corresponding 5.10 series shortly. > > > > > > Change log from v1 > > > (https://lore.kernel.org/all/20220603184701.3117780-1-leah.rumancik@xxxxxxxxx/): > > > - Increased testing > > > - Reduced patch set to overlap with 5.10 patches > > > > > > Thanks, > > > Leah > > > > > > Brian Foster (1): > > > xfs: punch out data fork delalloc blocks on COW writeback failure > > > > > > Darrick J. Wong (4): > > > xfs: remove all COW fork extents when remounting readonly > > > xfs: prevent UAF in xfs_log_item_in_current_chkpt > > > xfs: only bother with sync_filesystem during readonly remount > > > > 5.15 already has the vfs fixes to make sync_fs/sync_filesystem actually > > return error codes, right? Confirmed "vfs: make sync_filesystem return errors from ->sync_fs" made it into 5.15.y (935745abcf4c695a18b9af3fbe295e322547a114). > > > > > xfs: use setattr_copy to set vfs inode attributes > > > > > > Dave Chinner (1): > > > xfs: check sb_meta_uuid for dabuf buffer recovery > > > > > > Rustam Kovhaev (1): > > > xfs: use kmem_cache_free() for kmem_cache objects > > > > > > Yang Xu (1): > > > xfs: Fix the free logic of state in xfs_attr_node_hasname > > > > This one trips me up every time I look at it, but this looks correct. > > > > If the answer to the above question is yes, then: > > Acked-by: Darrick J. Wong <djwong@xxxxxxxxxx> > > I should've mentioned that this is acked-by for patches 1-7, since Amir > posted a question about patch 8 that seems not to have been answered(?) > > (Do all the new setgid inheritance tests actually pass with patch 8 > applied? The VFS fixes were thorny enough that I'm not as confident > that they've all been applied to 5.15.y...) The setgid tests (ex generic/314, generic/444, generic/673, generic/68[3-7], xfs/019) ran without issues. The dax config did have failures on both the baseline and backports branch but the other configs all passed cleanly. There were some changes to these tests since the last time I updated fstests though so I'll go ahead and rerun this set to be sure. Best, Leah > > --D > > > --D > > > > > > > > fs/xfs/libxfs/xfs_attr.c | 17 +++++------ > > > fs/xfs/xfs_aops.c | 15 ++++++++-- > > > fs/xfs/xfs_buf_item_recover.c | 2 +- > > > fs/xfs/xfs_extfree_item.c | 6 ++-- > > > fs/xfs/xfs_iops.c | 56 ++--------------------------------- > > > fs/xfs/xfs_log_cil.c | 6 ++-- > > > fs/xfs/xfs_pnfs.c | 3 +- > > > fs/xfs/xfs_super.c | 21 +++++++++---- > > > 8 files changed, 47 insertions(+), 79 deletions(-) > > > > > > -- > > > 2.36.1.476.g0c4daa206d-goog > > >
