[igt-dev] [PATCH i-g-t 7/8] lib/debugfs: use regular module param functions for prefault_disable

Juha-Pekka Heikkila juhapekka.heikkila at gmail.com
Tue Apr 21 18:30:00 UTC 2020


On 21.4.2020 21.10, Chris Wilson wrote:
> Quoting Juha-Pekka Heikkila (2020-04-21 17:17:24)
>> From: Jani Nikula <jani.nikula at intel.com>
>>
>> Functional change switches from igt_require to igt_assert on errors.
> 
> Adding entirely unused functions for fun?

That probably is from legacy, Jani's igt branch that I took was not very 
new.

Chris, you have opinion about patch 6/8 on what's going on there? 
There's in __igt_params_open(..) those two blocks of file checking which 
on first block is checking if wanted parameter file is in selected 
device directory and second block is just about 'guessing' default 
device name. I'm somehow thinking there would be easier way for those 
but I didn't find other way so I ended up reading and parsing files. It 
seems to work but not feeling comfortable with it.

/Juha-Pekka



More information about the igt-dev mailing list