check: reimplement _unload_modules() with _patient_rmmod()

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.

Reviewed-by: Bart Van Assche <[email protected]>
Reviewed-by: Daniel Wagner <[email protected]>
Signed-off-by: Shin'ichiro Kawasaki <[email protected]>

Author: kawasaki

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