Re: [PATCH 2/2] perf: SNB exclusive PMU access for INST_RETIRED:PREC_DIST

[Stephane Eranian]

On Fri, Oct 19, 2012 at 5:49 PM, Andi Kleen <ak@xxxxxxxxxxxxxxx> wrote:
>> +     /*
>> +      * for INST_RETIRED.PREC_DIST to work correctly with PEBS, it must
>> +      * be measured alone on SNB (exclusive PMU access) as per Intel SDM.
>> +      */
>> +     if ((cfg & INTEL_ARCH_EVENT_MASK) == 0x01c0 && !event->attr.exclusive) {
>> +             pr_info("perf: INST_RETIRED.PREC_DIST only works in exclusive mode\n");
>> +             return -EINVAL;
>> +     }
>
>
> Strictly you have to check for precise too, right?
>
Yes, the restriction is enforced only when PEBS is active and
this is what I do based on where pebs_aliases() is called.


> -Andi
--
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/