[PATCH igt] module_reload: remove snd_hda_intel

Paulo Zanoni
2013-10-21T13:56:31Z

From: Paulo Zanoni <paulo.r.zanoni@intel.com>

The audio driver uses the power well provided by our driver, so on
Haswell we can't "rmmod i915" if we don't "rmmod snd_hda_intel" first.

The problem with removing snd_hda_intel is that we also need to kill
its users. On the specific machine I tested, the only user seem to be
alsactl, but on other machines this may change. IMHO we should leave
the "kill user space" step to whoever is running the script, but
Daniel asked me to put it here so we have a better chance of Just
Working on QA's machines.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
---
tests/module_reload | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/tests/module_reload b/tests/module_reload
index 3d8cb85..c1fd395 100755
--- a/tests/module_reload
+++ b/tests/module_reload
@@ -18,6 +18,10 @@ if ! echo 0 > /sys/class/vtconsole/vtcon1/bind ; then
exit 77
fi

+# The sound driver uses our power well
+pkill alsactl
+rmmod snd_hda_intel &> /dev/null
+
#ignore errors in ips - gen5 only
rmmod intel_ips &> /dev/null
rmmod i915
@@ -38,6 +42,8 @@ fi
modprobe i915
echo 1 > /sys/class/vtconsole/vtcon1/bind

+modprobe snd_hda_intel
+
# try to run something
$SOURCE_DIR/gem_exec_nop > /dev/null && echo "module successfully loaded again"

--
1.8.3.1