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

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

 



On Fri, 21 Oct 2016, Petri Latvala <petri.latvala@xxxxxxxxx> 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@xxxxxxxxx>
>> 
>> ---
>> 
>> UNTESTED!!!
>
> This sounds scary but LGTM.
>
> Reviewed-by: Petri Latvala <petri.latvala@xxxxxxxxx>

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@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
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