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

Borislav Petkov
2015-08-30T15:53:26Z

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.

--
Regards/Gruss,
Boris.

ECO tip #101: Trim your mails when you reply.
--