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: <1554687347.2178.1.camel@intel.com> Date: Mon, 08 Apr 2019 09:35:47 +0800 From: Zhang Rui <rui.zhang@...el.com> To: Thomas Gleixner <tglx@...utronix.de>, Len Brown <lenb@...nel.org> Cc: x86@...nel.org, linux-kernel@...r.kernel.org, Len Brown <len.brown@...el.com>, linux-pm@...r.kernel.org Subject: Re: [PATCH 08/14] powercap/intel_rapl: Support multi-die/package On 五, 2019-04-05 at 20:27 +0200, Thomas Gleixner wrote: > On Tue, 26 Feb 2019, Len Brown wrote: > > > > > From: Zhang Rui <rui.zhang@...el.com> > > > > On the new dual-die/package systems, the RAPL MSR becomes die- > > scope. > > Thus instead of one powercap device per physical package, now there > > should be one powercap device for each unique die on these systems. > > > > This patch introduces intel_rapl driver support for new > > dual-die/package systems. > This patch .... See Documentation/processs/ > > and this sentence is not really helpful either. > > > > > On the hardwares that do not have multi-die, > > topology_logical_die_id() > > equals topology_physical_package_id(), thus there is no functional > > change. > Something like this: > > To support this the RAPL package domain has to be identified by the > die id > instead of the package id. On single die CPUs the die id is the same > as the > physical package id. > > Hmm? Yeah, sounds much better. Len, will you help me update the changelog or do you want me to send an updated version to you? thanks, rui > > > > > Signed-off-by: Zhang Rui <rui.zhang@...el.com> > > Signed-off-by: Len Brown <len.brown@...el.com> > > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com> > > Cc: linux-pm@...r.kernel.org > > --- > > drivers/powercap/intel_rapl.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/powercap/intel_rapl.c > > b/drivers/powercap/intel_rapl.c > > index 6057d9695fed..8723e9ae7436 100644 > > --- a/drivers/powercap/intel_rapl.c > > +++ b/drivers/powercap/intel_rapl.c > > @@ -266,7 +266,7 @@ static struct rapl_domain > > *platform_rapl_domain; /* Platform (PSys) domain */ > > /* caller to ensure CPU hotplug lock is held */ > > static struct rapl_package *rapl_find_package(int cpu) > > { > > - int id = topology_physical_package_id(cpu); > > + int id = topology_logical_die_id(cpu); > > struct rapl_package *rp; > > > > list_for_each_entry(rp, &rapl_packages, plist) { > > @@ -1457,7 +1457,7 @@ static void rapl_remove_package(struct > > rapl_package *rp) > > /* called from CPU hotplug notifier, hotplug lock held */ > > static struct rapl_package *rapl_add_package(int cpu) > > { > > - int id = topology_physical_package_id(cpu); > > + int id = topology_logical_die_id(cpu); > > struct rapl_package *rp; > > int ret; > >
Powered by blists - more mailing lists