Either we return $IGT_EXIT_FAILURE or remove it entirely (like in this
patch). If rmmod returns non-zero (i.e., Module: i915 is still in use), reload
will bail with $IGT_EXIT_SKIP, making the check with lsmod useless.
Also use the return value in the fault-injection loop.
Signed-off-by: Marius Vlad <marius.c.vlad@xxxxxxxxx>
---
tests/drv_module_reload_basic | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic
index 3bba796..3a8df33 100755
--- a/tests/drv_module_reload_basic
+++ b/tests/drv_module_reload_basic
@@ -30,7 +30,7 @@ function reload() {
#ignore errors in ips - gen5 only
rmmod intel_ips &> /dev/null
- rmmod i915 || return $IGT_EXIT_SKIP
+ rmmod i915
#ignore errors in intel-gtt, often built-in
rmmod intel-gtt &> /dev/null
# drm may be used by other devices (nouveau, radeon, udl, etc)
@@ -76,7 +76,7 @@ finish_load || exit $?
# Repeat the module reload trying to to generate faults
for i in $(seq 1 4); do
- reload inject_load_failure=$i
+ reload inject_load_failure=$i || exit $?
done
reload || exit $?
--
2.5.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
