mbox series

[PATCHSET,0/4] fstests: fix broken fuzzing xfs_mdrestore calls

Message ID 167096070957.1750373.5715692265711468248.stgit@magnolia (mailing list archive)
Headers show
Series fstests: fix broken fuzzing xfs_mdrestore calls | expand

Message

Darrick J. Wong Dec. 13, 2022, 7:45 p.m. UTC
Hi all,

Back in 2021, I amended the xfs create-a-filesystem helper code to
support the generation of compressed metadumps.  Unfortunately, I
neglected to submit the other half of that code which detects a
*compressed* metadump file and uses it, which means that the metadump
caching used by all the xfs fuzz tests have been broken ever since, but
only if DUMP_COMPRESSOR is set by the user.

Fast-forward to 2022, and I noticed that fuzz tests runs would
occasionally complain about POPULATE_METADUMP not pointing to a valid
file if DUMP_COMPRESSOR is set, and no dump file exists, compressed or
uncompressed.  After some curious looks from Zorro, I finally realized
that POPULATE_METADUMP points to the uncompressed file, so checking for
its existence is not correct.

Hence port all the code that fixed mdrestore of compressed metadumps
further down the patch stack, and fix the broken conditional to get
things running again.

This is an extraordinary way to destroy everything.  Enjoy!
Comments and questions are, as always, welcome.

--D
---
 common/ext4     |   21 +++++++++++++++++++++
 common/fuzzy    |    5 ++---
 common/populate |   15 ++-------------
 common/xfs      |   29 +++++++++++++++++++++++++++++
 tests/xfs/129   |    2 +-
 tests/xfs/234   |    2 +-
 tests/xfs/253   |    2 +-
 tests/xfs/284   |    2 +-
 tests/xfs/291   |    2 +-
 tests/xfs/336   |    2 +-
 tests/xfs/432   |    2 +-
 tests/xfs/503   |    8 ++++----
 12 files changed, 65 insertions(+), 27 deletions(-)