Messages in this thread |  | | Subject | Re: [PATCH v5 2/2] cpupower: Fix to return negative value if no permisssion for read_msr | From | Shuah Khan <> | Date | Mon, 26 Jul 2021 14:04:38 -0600 |
| |
On 7/23/21 12:26 AM, 徐福海 wrote: > From: xufuhai<xufuhai1992@gmail.com> <mailto:xufuhai@kuaishou.com> > > If the read_msr function is executed by a non-root user, the function returns > -1, which means that there is no permission to access//dev/cpu//%d/msr, but > cpufreq_has_boost_support should also return -1 immediately, and should not > follow the original logic to return 0, which will cause amd The cpupower tool > returns the boost active state as 0. > > Reproduce procedure: > cpupower frequency-info > > Reported-by: yangrui<yangrui@kuaishou.com> <mailto:yangrui@kuaishou.com> > Signed-off-by: xufuhai<xufuhai1992@gmail.com> <mailto:xufuhai@kuaishou.com> > Signed-off-by: chenguanqiao<chenguanqiao@kuaishou.com> <mailto:chenguanqiao@kuaishou.com> > Signed-off-by: lishujin<lishujin@kuaishou.com> <mailto:lishujin@kuaishou.com>
Please use your full names for the Signed-off-by - similar to the way you would sign a legal document.
thanks, -- Shuah
|  |