Messages in this thread | | | Subject | Re: [PATCH 2/3] cpupower: Introduce a new unit test module for AMD P-State driver | From | Shuah Khan <> | Date | Wed, 23 Mar 2022 08:15:11 -0600 |
| |
On 3/23/22 1:15 AM, Meng Li wrote: > amd-pstate-ut is a kernel module for testing the functions of AMD P-State driver. > > It can verify the required conditions and basic functions of AMD P-State > driver before integration test. >
Can you elaborate on the need for a kernel module? It would be helpful to know tne value the mdoule adds and why it is necessary. Include details on why it can't be done as part of the user-space program.
I am not saying it isn't necssary, I would like to know the reasons before I review the patch.
Also if this is a driver test, why not use other test frameworks such as kunit or kselftest. cpupower is user-space utility and driver debug code would not belong in here.
thanks, -- Shuah
| |