svn commit: r332972 - head/sys/x86/x86

[Konstantin Belousov]

Author: kib
Date: Tue Apr 24 20:33:08 2018
New Revision: 332972
URL: https://svnweb.freebsd.org/changeset/base/332972

Log:
  Extend ap_boot_mtx scope to also cover mca_init().
  
  Otherwise, under bootverbose, the lapic_enable_cmc() banner 'lapicX:
  CMCI unmasked' is printed by several CPUs in parallel, causing garbled
  output for the LAPIC dumps.
  
  Reported by:	royger
  Reviewed by:	jhb
  Sponsored by:	The FreeBSD Foundation
  MFC after:	1 week
  Differential revision:	https://reviews.freebsd.org/D15157

Modified:
  head/sys/x86/x86/mp_x86.c

Modified: head/sys/x86/x86/mp_x86.c
==============================================================================
--- head/sys/x86/x86/mp_x86.c	Tue Apr 24 20:29:40 2018	(r332971)
+++ head/sys/x86/x86/mp_x86.c	Tue Apr 24 20:33:08 2018	(r332972)
@@ -1007,9 +1007,9 @@ init_secondary_tail(void)
 	KASSERT(PCPU_GET(idlethread) != NULL, ("no idle thread"));
 	PCPU_SET(curthread, PCPU_GET(idlethread));
 
-	mca_init();
-
 	mtx_lock_spin(&ap_boot_mtx);
+
+	mca_init();
 
 	/* Init local apic for irq's */
 	lapic_setup(1);