Re: [PATCH 1/1] perf, Add support for Xeon-Phi PMU

From: stephane eranian
Date: Tue Sep 25 2012 - 08:05:50 EST


On Tue, Sep 25, 2012 at 2:01 PM, Peter Zijlstra <a.p.zijlstra@xxxxxxxxx> wrote:
> On Tue, 2012-09-25 at 15:42 +0400, Cyrill Gorcunov wrote:
>
>> Guys, letme re-read this whole mail thread first since I have no clue
>> what this remapping about ;)
>
> x86_setup_perfctr() / set_ext_hw_attr() have special purposed 0 and -1
> config values to mean -ENOENT and -EINVAL resp.
>
> This means neither config value can be a 'real' event. Now it turns out
> Xeon-Phi has an actual event 0, which is masked by these special case
> thingies.

Then how about using -1 or -2 for ENOENT and EINVAL?
--
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/