perf_counter: x86: Allow unpriviliged use of NMIs
authorPeter Zijlstra <a.p.zijlstra@chello.nl>
Wed, 13 May 2009 08:02:57 +0000 (10:02 +0200)
committerIngo Molnar <mingo@elte.hu>
Fri, 15 May 2009 07:46:57 +0000 (09:46 +0200)
Apply sysctl_perf_counter_priv to NMIs. Also, fail the counter
creation instead of silently down-grading to regular interrupts.

[ Impact: allow wider perf-counter usage ]

Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/cpu/perf_counter.c

index 88ae8cebf3c1cbda30cc5a90480b4f5727269057..c19e927b6979b8a0c3d273a106bdc5335682d35e 100644 (file)
@@ -280,8 +280,11 @@ static int __hw_perf_counter_init(struct perf_counter *counter)
         * If privileged enough, allow NMI events:
         */
        hwc->nmi = 0;
-       if (capable(CAP_SYS_ADMIN) && hw_event->nmi)
+       if (hw_event->nmi) {
+               if (sysctl_perf_counter_priv && !capable(CAP_SYS_ADMIN))
+                       return -EACCES;
                hwc->nmi = 1;
+       }
 
        hwc->irq_period = hw_event->irq_period;
        if ((s64)hwc->irq_period <= 0 || hwc->irq_period > x86_pmu.max_period)