Re: [PATCH 08/11] powercap/intel_rapl: Support multi-die/package

2019-02-25 Thread Zhang Rui
On 一, 2019-02-25 at 23:41 -0500, Len Brown wrote: > On Thu, Feb 21, 2019 at 12:44 AM Len Brown wrote: > > > > > > On Wed, Feb 20, 2019 at 6:02 AM Peter Zijlstra > g> wrote: > > > > > > > > > > > > >   list_for_each_entry(rp, &rapl_packages, plist) { > > > > @@ -1457,7 +1457,7 @@ static v

Re: [PATCH 08/11] powercap/intel_rapl: Support multi-die/package

2019-02-25 Thread Len Brown
On Thu, Feb 21, 2019 at 12:44 AM Len Brown wrote: > > On Wed, Feb 20, 2019 at 6:02 AM Peter Zijlstra wrote: > > > > 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

Re: [PATCH 08/11] powercap/intel_rapl: Support multi-die/package

2019-02-20 Thread Len Brown
On Wed, Feb 20, 2019 at 6:02 AM Peter Zijlstra wrote: > > 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 *

Re: [PATCH 08/11] powercap/intel_rapl: Support multi-die/package

2019-02-20 Thread Peter Zijlstra
On Mon, Feb 18, 2019 at 10:40:10PM -0500, Len Brown wrote: > From: Zhang Rui > > 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. > > Th

Re: [PATCH 08/11] powercap/intel_rapl: Support multi-die/package

2019-02-19 Thread Rafael J. Wysocki
On Tue, Feb 19, 2019 at 4:40 AM Len Brown wrote: > > From: Zhang Rui > > 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