Re: [ANNOUNCE] fstests: for-next branch updated to v2024.02.04

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



On Mon, Feb 05, 2024 at 01:45:02PM +0800, Zorro Lang wrote:
> On Sun, Feb 04, 2024 at 10:48:47PM +0800, Zorro Lang wrote:
> > Hi all,
> > 
> > The for-next branch of the xfstests repository at:
> > 
> > 	git://git.kernel.org/pub/scm/fs/xfs/xfstests-dev.git
> > 
> > has just been updated and tagged as v2024.02.04 release.
> > 
> > Release Notes:
> > 1) There's 4 new test cases in this release -- btrfs/310, generic/740, xfs/601,
> >    and xfs/606.
> > 2) btrfs/303 is removed.
> > 3) Supports XFS metadump V1 and V2 testing, refer to common/xfs_metadump_tests
> >    new file (might be renamed to common/metadump if more other fs need).
> > 4) Some fixes and updates for UBIFS, BCACHEFS and NFS testing, welcome more
> >    updates for more filesystems.
> > 5) Some updates/fixes for source building, if anyone find building issues in
> >    this release, please report to us.
> > 
> > More details refer to below, thanks for all your contributions!
> > 
> > Thanks,
> > Zorro
> > 
> > The new head of the for-next branch is commit:
> > 
> > 9b63adb69add t_snapshot_deleted_subvolume: add check for BTRFS_IOC_SNAP_DESTROY_V2
> > 
> > New commits ([N]ew, [U]pdate, [D]elete):
> > 
> > Alexander Aring (1):
> > [N]      [56625c897fd9] generic: add fcntl lock corner cases tests
> > 
> > Anthony Iliopoulos (2):
> > [U]      [42f3cb7e56e3] src/af_unix: drop unused offsetof macro definition
> > [U]      [79d98d94e58c] build: remove unused m4 macros
> > 
> > Boris Burkov (1):
> > [D]      [a6e196af46cc] btrfs: Remove btrfs/303
> > 
> > Darrick J. Wong (10):
> > [U]      [e7eadc3ee835] generic/256: constrain runtime with TIME_FACTOR
> > [U]      [8c8be04c1065] common/xfs: simplify maximum metadump format detection
> > [U]      [42f797643529] common/populate: always metadump full metadata blocks
> > [U]      [56ccc8cc69d1] xfs/336: fix omitted -a and -o in metadump call
> > [N]      [85c1e0f518ff] common: refactor metadump v1 and v2 tests
> > [U]      [1bdc3a5f1c9a] xfs/{129,234,253,605}: disable metadump v1 testing with external devices
> > [U]      [329671426287] xfs/503: test metadump obfuscation, not progressbars
> > [N]      [c31a500edc04] xfs/503: split copy and metadump into two tests
> > [U]      [7e2bf5955c2d] common/xfs: only pass -l in _xfs_mdrestore for v2 metadumps
> > [U]      [6f4223b6865f] xfs/122: fix for xfs_attr_shortform removal in 6.8
> 
> Note: A regression problem on this release sometimes after running xfs/605 or
> xfs/129, it only affect XFS testing, it might kill all later testing, I'm
> looking into it, and fix ASAP, please linux-xfs don't not update to this
> release temporarily, wait for a fix. Sorry for this trouble.

!!! It's a critical dangerous bug! Please don't use this release on xfs testing, it might
kill your system!

The problem comes from common/xfs_metadump_tests:_cleanup_verify_metadump():

   rm -f "$XFS_METADUMP_FILE" "$XFS_METADUMP_IMG"*

The XFS_METADUMP_FILE and XFS_METADUMP_IMG are set in _setup_verify_metadump(), but
if the _setup_verify_metadump isn't called due to someone case _notrun before calling
it, its _cleanup will still do _cleanup_verify_metadump, likes this:

  _cleanup()
  {
        cd /
        rm -r -f $tmp.*
        _cleanup_verify_metadump

Then boooomb, it does "rm -f *" in "/" .... The "*" after "rm" must be written
carefully. I'll send a patch to fix this problem soon, please don't update
fstests until the fix be merged.

Thanks,
Zorro

> 
> Thanks,
> Zorro
> 
> > 
> > Jeff Layton (2):
> > [U]      [ab217141c2fb] common/rc: NFSv2/3 do not support negative timestamps
> > [U]      [6c7f9b29e71d] src/t_mtab.c: increase the lock timeout to 120s
> > 
> > Li Zhijian (1):
> > [U]      [0160ab05979d] gitignore: Add missing /src/t_reflink_read_race
> > 
> > Pavel Reichl (1):
> > [U]      [f681ebf32b24] xfs/598: Add missing "fixed_by" hints
> > 
> > Qu Wenruo (1):
> > [N]      [dfdd248c648a] btrfs: verify the read behavior of compressed inline extent
> > 
> > Richard Weinberger (2):
> > [U]      [98dcd8c239d8] generic/020: Compute correct max_attrs for UBIFS
> > [U]      [1cf74a17a74e] generic/597,598: Require groups
> > 
> > Ritesh Harjani (IBM) (1):
> > [U]      [4cc6188cd89f] xfs/604: Make test as _notrun for higher blocksizes filesystem
> > 
> > Su Yue (2):
> > [U]      [eca4b512935a] fstests: introduce MKFS_BCACHEFS_PROG for bcachefs
> > [U]      [cbde2e1269dc] common/rc: improve block_size support for bcachefs
> > 
> > Yang Xu (1):
> > [U]      [9b63adb69add] t_snapshot_deleted_subvolume: add check for BTRFS_IOC_SNAP_DESTROY_V2
> > 
> > Zorro Lang (1):
> > [N]      [fb013bd945b8] xfs: test xfs_growfs with too-small size expansion
> > 
> > 
> > Code Diffstat:
> > 
> >  .gitignore                         |   1 +
> >  acinclude.m4                       |   5 -
> >  common/config                      |   4 +-
> >  common/populate                    |   2 +-
> >  common/rc                          |  63 ++++++--
> >  common/ubifs                       |   9 ++
> >  common/xfs                         |  51 +++++-
> >  common/xfs_metadump_tests          | 138 ++++++++++++++++
> >  configure.ac                       |   2 +-
> >  m4/manual_format.m4                |  17 --
> >  m4/package_libcdev.m4              |  95 -----------
> >  m4/package_ncurses.m4              |  27 ----
> >  m4/package_pthread.m4              |  19 ---
> >  m4/package_types.m4                |  50 ------
> >  m4/package_xfslibs.m4              |  28 ----
> >  m4/visibility_hidden.m4            |  22 ---
> >  src/Makefile                       |   2 +-
> >  src/af_unix.c                      |   2 -
> >  src/fcntl_lock_corner_tests.c      | 322 +++++++++++++++++++++++++++++++++++++
> >  src/t_mtab.c                       |   2 +-
> >  src/t_snapshot_deleted_subvolume.c |  10 +-
> >  tests/btrfs/303                    |  77 ---------
> >  tests/btrfs/303.out                |   2 -
> >  tests/btrfs/310                    |  83 ++++++++++
> >  tests/btrfs/310.out                |   2 +
> >  tests/generic/020                  |   6 +
> >  tests/generic/256                  |   7 +
> >  tests/generic/597                  |   2 +
> >  tests/generic/598                  |   2 +
> >  tests/generic/740                  |  27 ++++
> >  tests/generic/740.out              |   2 +
> >  tests/xfs/122.out                  |   2 +
> >  tests/xfs/129                      |  91 +----------
> >  tests/xfs/234                      |  92 +----------
> >  tests/xfs/253                      |  90 +----------
> >  tests/xfs/284                      |   4 +-
> >  tests/xfs/291                      |  32 ++--
> >  tests/xfs/336                      |   2 +-
> >  tests/xfs/432                      |  31 +---
> >  tests/xfs/503                      |  82 ++--------
> >  tests/xfs/503.out                  |   6 +-
> >  tests/xfs/598                      |   5 +
> >  tests/xfs/601                      |  54 +++++++
> >  tests/xfs/601.out                  |   4 +
> >  tests/xfs/604                      |   4 +-
> >  tests/xfs/605                      |  92 +----------
> >  tests/xfs/606                      |  56 +++++++
> >  tests/xfs/606.out                  |   2 +
> >  48 files changed, 887 insertions(+), 843 deletions(-)
> > --
> > Zorro Lang
> > zlang@xxxxxxxxxx
> 
> 





[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux