Skip to content

generic: Add some tests around journal replay/recoveryloop

Ritesh Harjani fstests linux-ext4 linux-fsdevel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v3 Latest4 patches 17 repliesIn review

Changes in v3 from the cover letter

=========
1. Addressed review comments from Darrick.
2. Rebased to latest master.

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4generic/468: Add another falloc test entrytests/generic/468 tests/generic/468.out+10 -06 replies
2/4common/punch: Add block_size argument to _filter_fiemap_**common/punch+5 -40 replies
3/4generic/678: Add a new shutdown recovery testtests/generic/678 tests/generic/678.out+79 -02 replies
4/4generic/679: Add a test to check unwritten extents trackingtests/generic/679 tests/generic/679.out+71 -02 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.

Read the v3 thread

Cover letter [PATCHv3 0/4] generic: Add some tests around journal replay/recoveryloop

Hello,

The ext4 fast_commit kernel fix has landed into mainline tree [1]. In this v3, I have addressed review comments from Darrick. Does this looks good to be picked up?

I have tested ext4 1k, 4k (w & w/o fast_commit). Also tested other FS with default configs (like xfs, btrfs, f2fs). No surprises were seen.

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bfdc502a4a4c058bf4cbb1df0c297761d528f54d

-ritesh

Changelogs:
===========

v2 => v3
=========
1. Addressed review comments from Darrick.
2. Rebased to latest master.

v1 => v2
=========
Sending v2 with tests/ext4/ converted to tests/generic/
(although I had not received any review comments on v1).
It seems all of the tests which I had sent in v1 are not ext4 specific anyways. So in v2, I have made those into tests/generic/.

Summary
=========
These are some of the tests which when tested with ext4 fast_commit feature w/o kernel fixes, could cause tests failures and/or KASAN bug (generic/486).

I gave these tests a run with default xfs, btrfs and f2fs configs (along with ext4). No surprises observed.

[v2]: https://lore.kernel.org/all/[email protected]/
[v1]: https://lore.kernel.org/all/[email protected]/

Ritesh Harjani (4):
generic/468: Add another falloc test entry common/punch: Add block_size argument to _filter_fiemap_** generic/678: Add a new shutdown recovery test generic/679: Add a test to check unwritten extents tracking

 common/punch          |  9 +++---
 tests/generic/468     |  8 +++++
 tests/generic/468.out |  2 ++
 tests/generic/678     | 72 +++++++++++++++++++++++++++++++++++++++++++
 tests/generic/678.out |  7 +++++
 tests/generic/679     | 65 ++++++++++++++++++++++++++++++++++++++
 tests/generic/679.out |  6 ++++
 7 files changed, 165 insertions(+), 4 deletions(-)
 create mode 100755 tests/generic/678
 create mode 100644 tests/generic/678.out
 create mode 100755 tests/generic/679
 create mode 100644 tests/generic/679.out

--
2.31.1