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_module() to work as expected. This change makes block/001 and srp test group skipped. Following three patches adjust skip conditions not to skip the test cases. Changes from v2: * 1st patch: kept single bracket style * 3rd patch: replaced find -or option with -o * 4th patch: adjusted to the 1st patch change * 6th patch: simplified _have_drivers() arguments Changes from v1: * 1st patch: added Reviewed-by tag * 4th patch: added changes in new test cases Shin'ichiro Kawasaki (6): common/rc: avoid module load in _have_driver() nbd/rc: load nbd module explicitly common/rc: ensure modules are loadable in _have_modules() common,tests: replace _have_driver() with _have_drivers() block/001: use _have_drivers() in place of _have_modules() srp/rc: allow test with built-in sd_mod and sg drivers common/null_blk | 2 +- common/rc | 43 ++++++++++++++++++++++++++++++++++++------- tests/block/001 | 3 ++- tests/nbd/rc | 14 +++++++++++--- tests/nvme/rc | 12 +++++------- tests/scsi/rc | 2 +- tests/srp/rc | 4 +--- tests/zbd/009 | 2 +- tests/zbd/010 | 2 +- 9 files changed, 59 insertions(+), 25 deletions(-) -- 2.37.1
