mbox series

[0/6] blktests: char device tests with iouring-cmd fio

Message ID 20221221103441.3216600-1-mcgrof@kernel.org (mailing list archive)
Headers show
Series blktests: char device tests with iouring-cmd fio | expand

Message

Luis Chamberlain Dec. 21, 2022, 10:34 a.m. UTC
As io-uring cmd grows there's a desire to do a bit more funky things
with it. Add basic support with fio and add a few simple tests to
tests the NVMe conventional drives character device as well as the
ZNS character device.

These tests are perhaps a bit *too* basic to merge, not sure, let
me know. But I figured that this would provide example to let us
grow this with more complex things later as folks add support for
more features.

Luis Chamberlain (6):
  common/fio: add helpers using io-uring cmd engine
  tests/nvme: add new test for rand-read on the nvme character device
  tests/nvme: add new test for rand-write on the nvme character device
  tests/nvme: add new test for optimal write on the nvme character
    device
  tests/zbd: add new basic test for reading zone character device
  tests/zbd: add new basic test for reading zone character device

 common/fio         | 47 +++++++++++++++++++++++++++++++++++++++++
 tests/nvme/046     | 42 +++++++++++++++++++++++++++++++++++++
 tests/nvme/046.out |  2 ++
 tests/nvme/047     | 41 ++++++++++++++++++++++++++++++++++++
 tests/nvme/047.out |  2 ++
 tests/nvme/048     | 41 ++++++++++++++++++++++++++++++++++++
 tests/nvme/048.out |  2 ++
 tests/zbd/011      | 52 ++++++++++++++++++++++++++++++++++++++++++++++
 tests/zbd/011.out  |  2 ++
 tests/zbd/012      | 50 ++++++++++++++++++++++++++++++++++++++++++++
 tests/zbd/012.out  |  2 ++
 11 files changed, 283 insertions(+)
 create mode 100755 tests/nvme/046
 create mode 100644 tests/nvme/046.out
 create mode 100755 tests/nvme/047
 create mode 100644 tests/nvme/047.out
 create mode 100755 tests/nvme/048
 create mode 100644 tests/nvme/048.out
 create mode 100755 tests/zbd/011
 create mode 100644 tests/zbd/011.out
 create mode 100755 tests/zbd/012
 create mode 100644 tests/zbd/012.out

Comments

Shinichiro Kawasaki Dec. 28, 2022, 3:25 a.m. UTC | #1
On Dec 21, 2022 / 02:34, Luis Chamberlain wrote:
> As io-uring cmd grows there's a desire to do a bit more funky things
> with it. Add basic support with fio and add a few simple tests to
> tests the NVMe conventional drives character device as well as the
> ZNS character device.
> 
> These tests are perhaps a bit *too* basic to merge, not sure, let
> me know. But I figured that this would provide example to let us
> grow this with more complex things later as folks add support for
> more features.

It is good to have new test cases to test new features and their new code paths.
I agree to have new test cases for the NVMe character device with io-uring.

Having said that, I'm not sure if we should have all of the five test cases in
the series. The test cases nvme/046, 047, 048 are similar. They do random read,
random write, or sequential write respectively. I'm not sure how the workload
difference expands the code coverage of the code paths in the NVMe driver. Same
for zbd/011 and 012. They are intended for ZNS devices, but I do not see ZNS
unique part in the NVMe character device code paths. Then the variation of the
test cases do not look useful to find bugs in the driver. As the first step, I
think single test case will be enough which does basic read and/or write to
exercises the NVMe character device and io-uring.