Commit 2b87965a authored Mar 28, 2017 by Steven Rostedt (VMware)

ftrace/x86: Do no run CPU sync when there is only one CPU online

Moving enabling of function tracing to early boot, even before scheduling is
enabled, means that it is not safe to enable interrupts. When function
tracing was enabled at boot up, it use to happen after scheduling and the
other CPUs were brought up. That required running a sync across all CPUs
when modifying the function hook locations in the code. To do the
synchronization, interrupts had to be enabled. Now function tracing can be
started before the other CPUs are brought up, and enabling interrupts in
that case is dangerous. As only tho boot CPU is active, there is no reason
to run the synchronization. If the online CPU count is one, do not bother
doing the synchronization. This removes the need to enable interrupts.

Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>

parent af0009fc

arch/x86/kernel/ftrace.c

+7 −1

Original line number	Diff line number	Diff line
		@@ -533,7 +533,13 @@ static void do_sync_core(void *data)

		static void run_sync(void)
		{
		int enable_irqs = irqs_disabled();
		int enable_irqs;

		/* No need to sync if there's only one CPU */
		if (num_online_cpus() == 1)
		return;

		enable_irqs = irqs_disabled();

		/* We may be called with interrupts disabled (on bootup). */
		if (enable_irqs)

Admin message