If we really are going to ban raw access to Offcore Response events for a
while, then we should return a proper error code. Right now we return "0"
which gives tools the false impression that opening the event worked.
Vince
[email protected]
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);