Re: [LTP] [PATCH] thermal: add new test group

Cyril Hrubis
2025-11-10T10:33:20Z

Hi!
> > First of all, the tst_test.sh test library is in maitenance mode and new
> > shell tests should use the tst_loader.sh instead. Have a look at the
> > testcases/kernel/mem/vma/vma05.sh test on how does it look like.
>
> Piotr, first thank you for submitting a test.
>
> My wish would be no more shell based test. shell loader is kind of compromise to
> improve existing shell based tests, but I'm not happy to see yet new more tests
> in shell (we in SUSE will be then actively rewriting it into C :( to avoid
> various problems with shell tests).

Honestly for a test that is as simple as this one shell may be fine.
Anything more complex that this should be probably written in C.

--
Cyril Hrubis
chrubis@suse.cz