Re: [PATCH] kunit: Mark filter_glob param as rw

[Krzysztofik, Janusz]

On Friday, 19 January 2024 00:29:33 CET Lucas De Marchi wrote:
> On Thu, Jan 18, 2024 at 05:23:33PM -0500, Rae Moar wrote:
> >On Thu, Jan 11, 2024 at 7:13 PM Lucas De Marchi
> ><lucas.demarchi@xxxxxxxxx> wrote:
> >>
> >> By allowing the filter_glob parameter to be written to, it's possible to
> >> tweak the testsuites that will be executed on new module loads. This
> >> makes it easier to run specific tests without having to reload kunit and
> >> provides a way to filter tests on real HW even if kunit is builtin.
> >> Example for xe driver:
> >>
> >> 1) Run just 1 test
> >>         # echo -n xe_bo > /sys/module/kunit/parameters/filter_glob
> >>         # modprobe -r xe_live_test
> >>         # modprobe xe_live_test
> >>         # ls /sys/kernel/debug/kunit/
> >>         xe_bo
> >>
> >> 2) Run all tests
> >>         # echo \* > /sys/module/kunit/parameters/filter_glob
> >>         # modprobe -r xe_live_test
> >>         # modprobe xe_live_test
> >>         # ls /sys/kernel/debug/kunit/
> >>         xe_bo  xe_dma_buf  xe_migrate  xe_mocs
> >>
> >> References: https://lore.kernel.org/intel-xe/
dzacvbdditbneiu3e3fmstjmttcbne44yspumpkd6sjn56jqpk@vxu7sksbqrp6/
> >> Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
> >
> >Hello!
> >
> >I have tested this and this looks good to me. I agree this is very
> >helpful and I wonder if we should do the same with the other module
> >parameters (filter, filter_action).
> 
> yeah, after I sent this I was wondering about the other parameters. I
> don't have a use for them right now, but I can try a few things and spin
> a new version if people find it useful.

Yes, please do.  I find it very useful for improving the current 
implementation of IGT kunit which now depends the ability to unload and reload 
the kunit base module with specific filter parameters in order to get a KTAP 
formatted list of test cases without executing them, then to run those test 
cases filtered one by one.

Thanks,
Janusz

> 
> >
> >It did worry me to make filter_glob writable due to the recent patch
> >that requires the output of filtering to be a valid virtual address
> >but I think there is a sufficient amount of checking of filter_glob.
> >
> >Thanks!
> >-Rae
> >
> >Reviewed-by: Rae Moar <rmoar@xxxxxxxxxx>
> 
> thanks
> Lucas De Marchi
> 
> 

---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 | Kapital zakladowy 200.000 PLN.
Spolka oswiadcza, ze posiada status duzego przedsiebiorcy w rozumieniu ustawy z dnia 8 marca 2013 r. o przeciwdzialaniu nadmiernym opoznieniom w transakcjach handlowych.

Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i moze zawierac informacje poufne. W razie przypadkowego otrzymania tej wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie; jakiekolwiek przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.