Re: [PATCH V2 00/13] selftests: ublk: test cleanup & add more tests

[Ming Lei <ming.lei@redhat.com>]

On Sat, Apr 12, 2025 at 10:30:16AM +0800, Ming Lei wrote:
> Hello Jens,
> 
> This patchset cleans up ublk selftests and add more tests:
> 
> - two bug fixes(1, 2)
> 
> - cleanup (3, 4)
> 
> - allow to run tests in parallel(5), also big simplification on
> test script
> 
> - add two stress tests for zero copy(6)
> 
> - kublk misc change(7, 8, 9), helps for evaluating performance
> 
> - support target specific command line, so help to add new
> target(Uday is working on fault-inject target) (10)
> 
> - add two tests for covering recovery features(11)
> 
> - add one heavy io & remove test over recovery enabled device(12),
> which can catch io hang triggered by several recent patches.
> 
> - the last patch is for making sure ublk temp file is cleaned up
> if test is skipped
> 
> With this change, kernel built-in ublk selftests can :
> 
> - cover almost all tests done by ublksrv 'make test T=generic', which has
> been effective to capture driver issue early, so it will make ublk driver
> development more efficiently
> 
> - add more stress tests for covering ublk zc feature, which has found one
> kernel panic issue introduced recently, fix merged already
> 
> - help to add new tests, such as per-target command line, which
> will help to write fault-inject target
> 
> 
> Thanks,
> 
> V2:
> 	- use ARRAY_SIZE() (Johannes Thumshirn)
> 	- drop one driver bug fix
> 	- fix ublk temp file cleanup
> 	- improve document

Hello Jens,

Can you consider to merge this patchset to v6.15?

- ublk selftest is just added to 6.15

- it includes bug fixes and nice cleanup/simplification

- test code is always tested fully

Then we can speedup to make the test code mature/stable in this cycle,
and later it can backported to liburing/blktest project.



thanks,
Ming