diff options
author | Paul Mackerras <paulus@samba.org> | 2009-03-16 06:00:00 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-04-06 03:30:14 -0400 |
commit | b6c5a71da1477d261bc36254fe1f20d32b57598d (patch) | |
tree | 02815fa0a0016eacd6203b1f496109f07bbd7be5 /kernel | |
parent | 7bb497bd885eedd0f56dfe3cc1b5ff20710d33b9 (diff) |
perf_counter: abstract wakeup flag setting in core to fix powerpc build
Impact: build fix for powerpc
Commit bd753921015e7905 ("perf_counter: software counter event
infrastructure") introduced a use of TIF_PERF_COUNTERS into the core
perfcounter code. This breaks the build on powerpc because we use
a flag in a per-cpu area to signal wakeups on powerpc rather than
a thread_info flag, because the thread_info flags have to be
manipulated with atomic operations and are thus slower than per-cpu
flags.
This fixes the by changing the core to use an abstracted
set_perf_counter_pending() function, which is defined on x86 to set
the TIF_PERF_COUNTERS flag and on powerpc to set the per-cpu flag
(paca->perf_counter_pending). It changes the previous powerpc
definition of set_perf_counter_pending to not take an argument and
adds a clear_perf_counter_pending, so as to simplify the definition
on x86.
On x86, set_perf_counter_pending() is defined as a macro. Defining
it as a static inline in arch/x86/include/asm/perf_counters.h causes
compile failures because <asm/perf_counters.h> gets included early in
<linux/sched.h>, and the definitions of set_tsk_thread_flag etc. are
therefore not available in <asm/perf_counters.h>. (On powerpc this
problem is avoided by defining set_perf_counter_pending etc. in
<asm/hw_irq.h>.)
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/perf_counter.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/perf_counter.c b/kernel/perf_counter.c index 0018c5e81249..b39456ad74a1 100644 --- a/kernel/perf_counter.c +++ b/kernel/perf_counter.c | |||
@@ -1433,7 +1433,7 @@ static void perf_swcounter_interrupt(struct perf_counter *counter, | |||
1433 | 1433 | ||
1434 | if (nmi) { | 1434 | if (nmi) { |
1435 | counter->wakeup_pending = 1; | 1435 | counter->wakeup_pending = 1; |
1436 | set_tsk_thread_flag(current, TIF_PERF_COUNTERS); | 1436 | set_perf_counter_pending(); |
1437 | } else | 1437 | } else |
1438 | wake_up(&counter->waitq); | 1438 | wake_up(&counter->waitq); |
1439 | } | 1439 | } |