forked from Minki/linux
perf_counter: Optimize disable of time based sw counters
Currently we call hrtimer_cancel() unconditionally on disable of time based software counters. Avoid when possible. [ Impact: micro-optimize the code ] Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Paul Mackerras <paulus@samba.org> Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Cc: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Marcelo Tosatti <mtosatti@redhat.com> Cc: John Kacur <jkacur@redhat.com> LKML-Reference: <20090520102553.388185031@chello.nl> Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
parent
26b119bc81
commit
b986d7ec0f
@ -2716,7 +2716,8 @@ static int cpu_clock_perf_counter_enable(struct perf_counter *counter)
|
|||||||
|
|
||||||
static void cpu_clock_perf_counter_disable(struct perf_counter *counter)
|
static void cpu_clock_perf_counter_disable(struct perf_counter *counter)
|
||||||
{
|
{
|
||||||
hrtimer_cancel(&counter->hw.hrtimer);
|
if (counter->hw.irq_period)
|
||||||
|
hrtimer_cancel(&counter->hw.hrtimer);
|
||||||
cpu_clock_perf_counter_update(counter);
|
cpu_clock_perf_counter_update(counter);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -2767,7 +2768,8 @@ static int task_clock_perf_counter_enable(struct perf_counter *counter)
|
|||||||
|
|
||||||
static void task_clock_perf_counter_disable(struct perf_counter *counter)
|
static void task_clock_perf_counter_disable(struct perf_counter *counter)
|
||||||
{
|
{
|
||||||
hrtimer_cancel(&counter->hw.hrtimer);
|
if (counter->hw.irq_period)
|
||||||
|
hrtimer_cancel(&counter->hw.hrtimer);
|
||||||
task_clock_perf_counter_update(counter, counter->ctx->time);
|
task_clock_perf_counter_update(counter, counter->ctx->time);
|
||||||
|
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user