Module Name:    src
Committed By:   jmcneill
Date:           Mon Jul 20 14:59:57 UTC 2020

Modified Files:
        src/sys/arch/mips/mips: cpu_subr.c

Log Message:
No need for cpu_hatch_lock after all since we hatch secondaries one at a time


To generate a diff of this commit:
cvs rdiff -u -r1.54 -r1.55 src/sys/arch/mips/mips/cpu_subr.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/arch/mips/mips/cpu_subr.c
diff -u src/sys/arch/mips/mips/cpu_subr.c:1.54 src/sys/arch/mips/mips/cpu_subr.c:1.55
--- src/sys/arch/mips/mips/cpu_subr.c:1.54	Mon Jul 20 14:38:38 2020
+++ src/sys/arch/mips/mips/cpu_subr.c	Mon Jul 20 14:59:57 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: cpu_subr.c,v 1.54 2020/07/20 14:38:38 skrll Exp $	*/
+/*	$NetBSD: cpu_subr.c,v 1.55 2020/07/20 14:59:57 jmcneill Exp $	*/
 
 /*-
  * Copyright (c) 2010, 2019 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cpu_subr.c,v 1.54 2020/07/20 14:38:38 skrll Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cpu_subr.c,v 1.55 2020/07/20 14:59:57 jmcneill Exp $");
 
 #include "opt_cputype.h"
 #include "opt_ddb.h"
@@ -106,8 +106,6 @@ const pcu_ops_t * const pcu_ops_md_defs[
 };
 
 #ifdef MULTIPROCESSOR
-static kmutex_t cpu_hatch_lock;
-
 struct cpu_info * cpuid_infos[MAXCPUS] = {
 	[0] = &cpu_info_store,
 };
@@ -951,9 +949,7 @@ cpu_hatch(struct cpu_info *ci)
 	 * Let this CPU do its own post-running initialization
 	 * (for things that have to be done on the local CPU).
 	 */
-	mutex_enter(&cpu_hatch_lock);
 	(*mips_locoresw.lsw_cpu_run)(ci);
-	mutex_exit(&cpu_hatch_lock);
 
 	/*
 	 * Now turn on interrupts (and verify they are on).
@@ -977,8 +973,6 @@ cpu_boot_secondary_processors(void)
 	CPU_INFO_ITERATOR cii;
 	struct cpu_info *ci;
 
-	mutex_init(&cpu_hatch_lock, MUTEX_DEFAULT, IPL_HIGH);
-
 	for (CPU_INFO_FOREACH(cii, ci)) {
 		if (CPU_IS_PRIMARY(ci))
 			continue;

Reply via email to