To make the helper function _unload_modules() more robust, reimplement it to call _patient_rmmod(). Another function with the similar name _unload_module() is left as it is. Signed-off-by: Shin'ichiro Kawasaki --- check | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/check b/check index 6a156b3..a8b7611 100755 --- a/check +++ b/check @@ -596,10 +596,13 @@ _unload_module() { } _unload_modules() { - local i + local i reason for ((i=${#MODULES_TO_UNLOAD[@]}; i > 0; i--)); do - _unload_module "${MODULES_TO_UNLOAD[i-1]}" 10 + if ! reason=$(_patient_rmmod "${MODULES_TO_UNLOAD[i-1]}" \ + 2>&1); then + echo "${reason}" >&2 + fi done unset MODULES_TO_UNLOAD -- 2.52.0