Messages in this thread | | | Date | Thu, 17 Oct 2013 11:07:30 +0200 | From | Peter Zijlstra <> | Subject | Re: [PATCH v2 0/3] perf,x86: add Intel RAPL PMU support |
| |
On Thu, Oct 17, 2013 at 10:14:20AM +0200, Ingo Molnar wrote: > > > I.e. you would be encoding the counter unit as the suffix, might as well > > > call it "power/cores.joules" and use the dot as the separator for the > > > unit, but would be just a compact form to encode the counter->unit > > > table. > > > > May be easier to add a sysfs entry with the unit to display. > > Yes - with no entry meaning a raw 'count' or such.
The downside to such a sysfs entry will be the scope. It would either be pmu wide (unwieldy for many PMUs) or be only per listed event; and we really don't want exhaustive event lists in the kernel.
| |