x86: Add a 'pause' instruction in __udelay() for QEMU target
authorMiao Yan <yanmiaobest@gmail.com>
Mon, 27 Jul 2015 11:16:07 +0000 (19:16 +0800)
committerSimon Glass <sjg@chromium.org>
Wed, 5 Aug 2015 14:42:38 +0000 (08:42 -0600)
When running SMP configuration on QEMU (tcg mode, no kvm), there is
a busy loop in start_aps(), calling udelay(), that waits for APs to
show up online. However, there is a chance that VCPU1 will be timeout
waiting, IOW the secondary VCPUs haven't started their execution yet.

This patch adds a 'pause' instruction in __udelay() only for QEMU
target, to give other VCPUs a chance to run. When QEMU sees the
'pause' instruction, it will yeild the execution to other CPUs.

Signed-off-by: Miao Yan <yanmiaobest@gmail.com>
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
arch/x86/lib/tsc_timer.c

index 7f5ba2ca6f1dd69f2d461723fec4536fc92a4523..0df1af238c1d654e59d9b09c555c363cb2196943 100644 (file)
@@ -355,7 +355,15 @@ void __udelay(unsigned long usec)
        stop = now + usec * get_tbclk_mhz();
 
        while ((int64_t)(stop - get_ticks()) > 0)
+#if defined(CONFIG_QEMU) && defined(CONFIG_SMP)
+               /*
+                * Add a 'pause' instruction on qemu target,
+                * to give other VCPUs a chance to run.
+                */
+               asm volatile("pause");
+#else
                ;
+#endif
 }
 
 int timer_init(void)