diff options
author | Ingo Molnar <mingo@elte.hu> | 2009-05-13 23:16:59 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-05-15 03:47:07 -0400 |
commit | 251e8e3c7235f5944805a64f24c79fc4696793f1 (patch) | |
tree | 8e171cd075aa20432f7c4a03e88deaf2f3f32b49 /drivers | |
parent | 9029a5e3801f1cc7cdaab80169d82427acf928d8 (diff) |
perf_counter: Remove ACPI quirk
We had a disable/enable around acpi_idle_do_entry() due to an erratum
in an early prototype CPU i had access to. That erratum has been fixed
in the BIOS so remove the quirk.
The quirk also kept us from profiling interrupts that hit the ACPI idle
instruction - so this is an improvement as well, beyond a cleanup and
a micro-optimization.
[ Impact: improve profiling scope, cleanup, micro-optimization ]
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>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/processor_idle.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c index 9645758c0472..f7ca8c55956b 100644 --- a/drivers/acpi/processor_idle.c +++ b/drivers/acpi/processor_idle.c | |||
@@ -765,7 +765,6 @@ static inline void acpi_idle_do_entry(struct acpi_processor_cx *cx) | |||
765 | { | 765 | { |
766 | /* Don't trace irqs off for idle */ | 766 | /* Don't trace irqs off for idle */ |
767 | stop_critical_timings(); | 767 | stop_critical_timings(); |
768 | perf_disable(); | ||
769 | if (cx->entry_method == ACPI_CSTATE_FFH) { | 768 | if (cx->entry_method == ACPI_CSTATE_FFH) { |
770 | /* Call into architectural FFH based C-state */ | 769 | /* Call into architectural FFH based C-state */ |
771 | acpi_processor_ffh_cstate_enter(cx); | 770 | acpi_processor_ffh_cstate_enter(cx); |
@@ -780,7 +779,6 @@ static inline void acpi_idle_do_entry(struct acpi_processor_cx *cx) | |||
780 | gets asserted in time to freeze execution properly. */ | 779 | gets asserted in time to freeze execution properly. */ |
781 | unused = inl(acpi_gbl_FADT.xpm_timer_block.address); | 780 | unused = inl(acpi_gbl_FADT.xpm_timer_block.address); |
782 | } | 781 | } |
783 | perf_enable(); | ||
784 | start_critical_timings(); | 782 | start_critical_timings(); |
785 | } | 783 | } |
786 | 784 | ||