[blktests,v4,0/2] md: add regression test for "md/md-bitmap: fix writing non bitmap pages"

Message

Ofir Gal July 21, 2024, 7:11 a.m. UTC

This patchset adds a regression test for "md/md-bitmap: fix writing non
bitmap pages".

The regression test requires a network layer as the underlying layer of
md, it use nvme-tcp as the network layer.

Changelog:
v2 - applied Shinichiro's comments, use common/nvme instead of
   tests/nvme/rc, disconnecting nvme controller on cleanup_nvme_over_tcp

v3 - applied Shinichiro's comments, fixed shellcheck, moved
   _nvme_disconnect_ctrl() to common/nvme. applied Daniel's comments,
   using ${def_subsysnqn}, moved _find_nvme_ns() to common/nvme.

v4 - applied Yu's comments, add requires() for md-mod and raid1

Ofir Gal (2):
  nvme: move helper functions to common/nvme
  md: add regression test for "md/md-bitmap: fix writing non bitmap
    pages"

 common/brd       |  28 +++
 common/nvme      | 636 +++++++++++++++++++++++++++++++++++++++++++++++
 tests/md/001     |  86 +++++++
 tests/md/001.out |   3 +
 tests/md/rc      |  13 +
 tests/nvme/rc    | 629 +---------------------------------------------
 6 files changed, 767 insertions(+), 628 deletions(-)
 create mode 100644 common/brd
 create mode 100644 common/nvme
 create mode 100755 tests/md/001
 create mode 100644 tests/md/001.out
 create mode 100644 tests/md/rc

Comments

Shin'ichiro Kawasaki July 29, 2024, 12:19 p.m. UTC | #1

On Jul 21, 2024 / 10:11, Ofir Gal wrote:
> This patchset adds a regression test for "md/md-bitmap: fix writing non
> bitmap pages".
> 
> The regression test requires a network layer as the underlying layer of
> md, it use nvme-tcp as the network layer.
> 
> Changelog:
> v2 - applied Shinichiro's comments, use common/nvme instead of
>    tests/nvme/rc, disconnecting nvme controller on cleanup_nvme_over_tcp
> 
> v3 - applied Shinichiro's comments, fixed shellcheck, moved
>    _nvme_disconnect_ctrl() to common/nvme. applied Daniel's comments,
>    using ${def_subsysnqn}, moved _find_nvme_ns() to common/nvme.
> 
> v4 - applied Yu's comments, add requires() for md-mod and raid1

I applied this series with the some comment improvements in the second patch.
Thanks!