On Fri, 1 Jul 2011, tip-bot for Vince Weaver wrote: > Commit-ID: 04bffb83d17c1b93578564f18e02b6cb1c385b40 > Gitweb: http://git.kernel.org/tip/04bffb83d17c1b93578564f18e02b6cb1c385b40 > Author: Vince Weaver <[email protected]> > AuthorDate: Fri, 10 Jun 2011 16:07:52 -0400 > Committer: Ingo Molnar <[email protected]> > CommitDate: Fri, 1 Jul 2011 10:27:09 +0200 > > perf_event: Return -EINVAL for Offcore Response Events
Wait! Please don't apply this one. It does indeed Return -EINVAL for Offcore Response Events but also for *all* RAW events. I missed that on initial testing, and then I caught the problem but couldn't find a simple way to return EINVAL only for config1 being set to nonzero. The code used to shutdown OFFCORE_RESPONSE here is very subtle and non-obvious. Vince > > Return a proper error code when denying raw access to Offcore Response > events. Right now we return "0" which gives tools the false impression > that opening the event worked. > > Signed-off-by: Peter Zijlstra <[email protected]> > Cc: <[email protected]> > Link: > http://lkml.kernel.org/r/[email protected] > Signed-off-by: Ingo Molnar <[email protected]> > --- > arch/x86/kernel/cpu/perf_event.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/x86/kernel/cpu/perf_event.c > b/arch/x86/kernel/cpu/perf_event.c > index 3a0338b..c6bfa6b 100644 > --- a/arch/x86/kernel/cpu/perf_event.c > +++ b/arch/x86/kernel/cpu/perf_event.c > @@ -598,7 +598,7 @@ static int x86_setup_perfctr(struct perf_event *event) > * there's no sane user-space generalization yet: > */ > if (attr->type == PERF_TYPE_RAW) > - return 0; > + return -EINVAL; > > if (attr->type == PERF_TYPE_HW_CACHE) > return set_ext_hw_attr(hwc, event); > > Vince [email protected] _______________________________________________ stable mailing list [email protected] http://linux.kernel.org/mailman/listinfo/stable
