Re: [PATCH 09/15] x86, amd: add accessor for number of cores per compute unit

From: Guenter Roeck
Date: Mon Aug 31 2015 - 09:26:13 EST


On 08/31/2015 01:38 AM, Peter Zijlstra wrote:
On Sun, Aug 30, 2015 at 05:53:22PM +0200, Borislav Petkov wrote:
On Sat, Aug 29, 2015 at 11:19:14AM +0200, Ingo Molnar wrote:
So let me withdraw my ack: the much more important question that I
missed first time around, why is this reporting feature living in
hwmon, not in perf? We have energy reporting facilities in perf that
this should be synced to.

Because there's already fam15h_power driver which is exactly for that.
Making it part of perf is then a question of cat-ting the same sysfs
file twice, at the beginning and at the end of the trace, which is
trivial.

That don't make sense.

Looking at the BKDG Fam 15h 60h-6Fh these MSRs are per compute unit.
This means you can do much finer grained measurements than system wide
-- which is all hwmon seems capable of.


Is it ? Why ?

Guenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/