| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <ab36709d-a181-4621-a8e5-0ef38b80186b@arm.com> Date: Wed, 30 Oct 2024 09:55:38 +0000 From: Lukasz Luba <lukasz.luba@....com> To: Cristian Marussi <cristian.marussi@....com>, Sibi Sankar <quic_sibis@...cinc.com> Cc: sudeep.holla@....com, rafael@...nel.org, viresh.kumar@...aro.org, morten.rasmussen@....com, dietmar.eggemann@....com, pierre.gondois@....com, vincent.guittot@...aro.org, linux-arm-kernel@...ts.infradead.org, linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org, quic_mdtipton@...cinc.com, linux-arm-msm@...r.kernel.org Subject: Re: [PATCH V5 1/1] cpufreq: scmi: Register for limit change notifications On 10/30/24 09:32, Cristian Marussi wrote: > On Tue, Jun 04, 2024 at 12:56:54AM +0530, Sibi Sankar wrote: >> Register for limit change notifications if supported and use the throttled >> frequency from the notification to apply HW pressure. >> > > This sort of got lost in mbox...apologies. > > As discussed offlist, the issue with cpufreq and multiple SCMI instances > remains BUT it will be solved later on with a different approach, so > as of now this LGTM. > > Reviewed-by: Cristian Marussi <cristian.marussi@....com> Thank you Cristian for re-visiting this. That code also LGTM and fits into my other construction built on top of it [1] [2]. I had a talk with Rafael below that [1] about this subject patch. Feel free to add: Reviewed-by: Lukasz Luba <lukasz.luba@....com> [1] https://lore.kernel.org/lkml/20241029094452.495439-1-lukasz.luba@arm.com/ [2] https://lore.kernel.org/lkml/20240403162315.1458337-1-lukasz.luba@arm.com/
Powered by blists - more mailing lists