Re: [PATCH i-g-t] igt/drv_module_reload_basic: Do not require snd-hda-intel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 15, 2016 at 10:19:32AM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> 
> Recent refactoring has made absence of snd-hda-intel a test
> failure do the respective modprobe being at the end of a
> reload function now.
> 
> Only fail in this case if module was previously unloaded.
> 
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
> Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> ---
>  tests/drv_module_reload_basic | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic
> index fc508d7204b2..3bba796f0306 100755
> --- a/tests/drv_module_reload_basic
> +++ b/tests/drv_module_reload_basic
> @@ -11,6 +11,8 @@ SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"
>  # no other drm service should be running, so we can just unbind
>  
>  function reload() {
> +	local snd_hda_intel_unloaded
> +
>  	echo Reloading i915.ko with $*
>  
>  	# we must kick away fbcon (but only fbcon)
> @@ -23,7 +25,8 @@ function reload() {
>  
>  	# The sound driver uses our power well
>  	pkill alsactl
> -	rmmod snd_hda_intel &> /dev/null
> +	snd_hda_intel_unloaded=0
> +	rmmod snd_hda_intel &> /dev/null && snd_hda_intel_unloaded=1
>  
>  	#ignore errors in ips - gen5 only
>  	rmmod intel_ips &> /dev/null
> @@ -47,7 +50,7 @@ function reload() {
>  		echo 1 > /sys/class/vtconsole/vtcon1/bind
>  	fi
>  
> -	modprobe snd_hda_intel
> +	modprobe -q snd_hda_intel || return $snd_hda_intel_unloaded

I'm sure that there is a better way to write this to allow easier
extension of the bash function. (Writing it in C comes to mind!)

The code seems to dtrt since we must try reloading the snd_hda_intel even
if wasn't loaded on the previous run.

So Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux