On Aug 23, 2022 / 09:11, Shin'ichiro Kawasaki wrote: > Current blktests have unexpected test case failures caused by module > availability checks. This series addresses two issues related to module check > and fix the failures. > > The first issue is caused by module load by _have_driver(). When this helper > function checks the specified module (or driver) is available, it loads the > module using modprobe command. It leaves the module loaded and affects following > test cases. The first patch addresses this issue. The second patch avoids side > affects of the first patch on nbd test cases. > > The second issue is in _have_modules(). Recently, _have_driver() helper function > was introduced to provide similar but different feature from _have_modules(). > However, it turned out that _have_modules() is not working as expected and does > exactly same check as _have_driver(). The third patch fixes _have_modules() to > work as expected. This change makes block/001 and srp test group skipped. > Following two patches adjust skip conditions not to skip the test cases. > > The last patch is an additional clean up to change _have_modules() to > _have_module() to make its usage consistent with _have_driver(). I've applied this series to the upstream. Thanks for the reviews. -- Shin'ichiro Kawasaki
