[i-g-t PATCH] tests: debug log lsmod on module unload failure

[i-g-t PATCH] tests: debug log lsmod on module unload failure

[Jani Nikula]

It's a bit verbose, but should only happen on errors. This should help
debug the snd_hda_intel unload issues.

Signed-off-by: Jani Nikula <jani.nikula@intel.com>

---

UNTESTED!!!
---
 tests/drv_module_reload_basic | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic
index a8d628dd96bd..b8cad88133e9 100755
--- a/tests/drv_module_reload_basic
+++ b/tests/drv_module_reload_basic
@@ -33,14 +33,22 @@ function reload() {
 	pkill alsactl
 	snd_hda_intel_unloaded=0
 	if mod_loaded snd_hda_intel; then
-		rmmod snd_hda_intel && snd_hda_intel_unloaded=1
+		if rmmod snd_hda_intel; then
+			snd_hda_intel_unloaded=1
+		else
+			lsmod >&2
+		fi
 	fi
 
 	# gen5 only
 	if mod_loaded intel_ips; then
 		rmmod intel_ips
 	fi
-	rmmod i915 || return $IGT_EXIT_SKIP
+
+	if ! rmmod i915; then
+		lsmod >&2
+		return $IGT_EXIT_SKIP
+	fi
 	#ignore errors in intel-gtt, often built-in
 	rmmod intel-gtt &> /dev/null
 	# drm may be used by other devices (nouveau, radeon, udl, etc)
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [i-g-t PATCH] tests: debug log lsmod on module unload failure

[Petri Latvala]

On Tue, Oct 18, 2016 at 10:07:04AM +0300, Jani Nikula wrote:
> It's a bit verbose, but should only happen on errors. This should help
> debug the snd_hda_intel unload issues.
> 
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> 
> ---
> 
> UNTESTED!!!

This sounds scary but LGTM.

Reviewed-by: Petri Latvala <petri.latvala@intel.com>


> ---
>  tests/drv_module_reload_basic | 12 ++++++++++--
>  1 file changed, 10 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic
> index a8d628dd96bd..b8cad88133e9 100755
> --- a/tests/drv_module_reload_basic
> +++ b/tests/drv_module_reload_basic
> @@ -33,14 +33,22 @@ function reload() {
>  	pkill alsactl
>  	snd_hda_intel_unloaded=0
>  	if mod_loaded snd_hda_intel; then
> -		rmmod snd_hda_intel && snd_hda_intel_unloaded=1
> +		if rmmod snd_hda_intel; then
> +			snd_hda_intel_unloaded=1
> +		else
> +			lsmod >&2
> +		fi
>  	fi
>  
>  	# gen5 only
>  	if mod_loaded intel_ips; then
>  		rmmod intel_ips
>  	fi
> -	rmmod i915 || return $IGT_EXIT_SKIP
> +
> +	if ! rmmod i915; then
> +		lsmod >&2
> +		return $IGT_EXIT_SKIP
> +	fi
>  	#ignore errors in intel-gtt, often built-in
>  	rmmod intel-gtt &> /dev/null
>  	# drm may be used by other devices (nouveau, radeon, udl, etc)
> -- 
> 2.1.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [i-g-t PATCH] tests: debug log lsmod on module unload failure

[Jani Nikula]

On Fri, 21 Oct 2016, Petri Latvala <petri.latvala@intel.com> wrote:
> On Tue, Oct 18, 2016 at 10:07:04AM +0300, Jani Nikula wrote:
>> It's a bit verbose, but should only happen on errors. This should help
>> debug the snd_hda_intel unload issues.
>> 
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>> 
>> ---
>> 
>> UNTESTED!!!
>
> This sounds scary but LGTM.
>
> Reviewed-by: Petri Latvala <petri.latvala@intel.com>

Thanks, I gave this a bit of testing, and pushed.

BR,
Jani.

>
>
>> ---
>>  tests/drv_module_reload_basic | 12 ++++++++++--
>>  1 file changed, 10 insertions(+), 2 deletions(-)
>> 
>> diff --git a/tests/drv_module_reload_basic b/tests/drv_module_reload_basic
>> index a8d628dd96bd..b8cad88133e9 100755
>> --- a/tests/drv_module_reload_basic
>> +++ b/tests/drv_module_reload_basic
>> @@ -33,14 +33,22 @@ function reload() {
>>  	pkill alsactl
>>  	snd_hda_intel_unloaded=0
>>  	if mod_loaded snd_hda_intel; then
>> -		rmmod snd_hda_intel && snd_hda_intel_unloaded=1
>> +		if rmmod snd_hda_intel; then
>> +			snd_hda_intel_unloaded=1
>> +		else
>> +			lsmod >&2
>> +		fi
>>  	fi
>>  
>>  	# gen5 only
>>  	if mod_loaded intel_ips; then
>>  		rmmod intel_ips
>>  	fi
>> -	rmmod i915 || return $IGT_EXIT_SKIP
>> +
>> +	if ! rmmod i915; then
>> +		lsmod >&2
>> +		return $IGT_EXIT_SKIP
>> +	fi
>>  	#ignore errors in intel-gtt, often built-in
>>  	rmmod intel-gtt &> /dev/null
>>  	# drm may be used by other devices (nouveau, radeon, udl, etc)
>> -- 
>> 2.1.4
>> 
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx