Skip to content

Commit

Permalink
x86: move setup_secondary_clock a little bit down in the function
Browse files Browse the repository at this point in the history
This is done so we call setup_secondary_clock() in the same place x86_64
does. A separate patch for this is appearantly not needed. But clock
initialization is such a delicate thing, that it's safer to do this way

Signed-off-by: Glauber Costa <gcosta@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
  • Loading branch information
Glauber de Oliveira Costa authored and Ingo Molnar committed Apr 17, 2008
1 parent 3ff8171 commit 8f15b82
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion arch/x86/kernel/smpboot_32.c
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@ static void __cpuinit start_secondary(void *unused)
*/
check_tsc_sync_target();

setup_secondary_clock();
if (nmi_watchdog == NMI_IO_APIC) {
disable_8259A_irq(0);
enable_NMI_through_LVT0();
Expand Down Expand Up @@ -214,6 +213,8 @@ static void __cpuinit start_secondary(void *unused)
unlock_ipi_call_lock();
per_cpu(cpu_state, smp_processor_id()) = CPU_ONLINE;

setup_secondary_clock();

wmb();
cpu_idle();
}
Expand Down

0 comments on commit 8f15b82

Please sign in to comment.