Module Name:    src
Committed By:   jmcneill
Date:           Wed Jan  8 18:47:43 UTC 2020

Modified Files:
        src/sys/arch/arm/arm32: arm32_boot.c cpu.c

Log Message:
cpu_hatch is too late to report AP topology, do it at attach time instead


To generate a diff of this commit:
cvs rdiff -u -r1.35 -r1.36 src/sys/arch/arm/arm32/arm32_boot.c
cvs rdiff -u -r1.136 -r1.137 src/sys/arch/arm/arm32/cpu.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/arm/arm32/arm32_boot.c
diff -u src/sys/arch/arm/arm32/arm32_boot.c:1.35 src/sys/arch/arm/arm32/arm32_boot.c:1.36
--- src/sys/arch/arm/arm32/arm32_boot.c:1.35	Fri Dec 20 21:05:33 2019
+++ src/sys/arch/arm/arm32/arm32_boot.c	Wed Jan  8 18:47:43 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: arm32_boot.c,v 1.35 2019/12/20 21:05:33 ad Exp $	*/
+/*	$NetBSD: arm32_boot.c,v 1.36 2020/01/08 18:47:43 jmcneill Exp $	*/
 
 /*
  * Copyright (c) 2002, 2003, 2005  Genetec Corporation.  All rights reserved.
@@ -122,7 +122,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(1, "$NetBSD: arm32_boot.c,v 1.35 2019/12/20 21:05:33 ad Exp $");
+__KERNEL_RCSID(1, "$NetBSD: arm32_boot.c,v 1.36 2020/01/08 18:47:43 jmcneill Exp $");
 
 #include "opt_arm_debug.h"
 #include "opt_cputypes.h"
@@ -357,20 +357,6 @@ cpu_hatch(struct cpu_info *ci, u_int cpu
 	ci->ci_ctrl = armreg_sctlr_read();
 	uint32_t mpidr = armreg_mpidr_read();
 	ci->ci_mpidr = mpidr;
-	if (mpidr & MPIDR_MT) {
-		cpu_topology_set(ci,
-		    __SHIFTOUT(mpidr, MPIDR_AFF2),
-		    __SHIFTOUT(mpidr, MPIDR_AFF1),
-		    __SHIFTOUT(mpidr, MPIDR_AFF0),
-		    0);
-	} else {
-		cpu_topology_set(ci,
-		    __SHIFTOUT(mpidr, MPIDR_AFF1),
-	            __SHIFTOUT(mpidr, MPIDR_AFF0),
-	            0,
-	            0);
-	}
-
 	ci->ci_arm_cpuid = cpu_idnum();
 	ci->ci_arm_cputype = ci->ci_arm_cpuid & CPU_ID_CPU_MASK;
 	ci->ci_arm_cpurev = ci->ci_arm_cpuid & CPU_ID_REVISION_MASK;

Index: src/sys/arch/arm/arm32/cpu.c
diff -u src/sys/arch/arm/arm32/cpu.c:1.136 src/sys/arch/arm/arm32/cpu.c:1.137
--- src/sys/arch/arm/arm32/cpu.c:1.136	Fri Dec 20 21:05:33 2019
+++ src/sys/arch/arm/arm32/cpu.c	Wed Jan  8 18:47:43 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: cpu.c,v 1.136 2019/12/20 21:05:33 ad Exp $	*/
+/*	$NetBSD: cpu.c,v 1.137 2020/01/08 18:47:43 jmcneill Exp $	*/
 
 /*
  * Copyright (c) 1995 Mark Brinicombe.
@@ -46,7 +46,7 @@
 #include "opt_multiprocessor.h"
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.136 2019/12/20 21:05:33 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cpu.c,v 1.137 2020/01/08 18:47:43 jmcneill Exp $");
 
 #include <sys/param.h>
 #include <sys/conf.h>
@@ -109,20 +109,6 @@ cpu_attach(device_t dv, cpuid_t id)
 #ifdef MULTIPROCESSOR
 		uint32_t mpidr = armreg_mpidr_read();
 		ci->ci_mpidr = mpidr;
-
-		if (mpidr & MPIDR_MT) {
-			cpu_topology_set(ci,
-			    __SHIFTOUT(mpidr, MPIDR_AFF2),
-			    __SHIFTOUT(mpidr, MPIDR_AFF1),
-			    __SHIFTOUT(mpidr, MPIDR_AFF0),
-			    0);
-		} else {
-			cpu_topology_set(ci,
-			    __SHIFTOUT(mpidr, MPIDR_AFF1),
-			    __SHIFTOUT(mpidr, MPIDR_AFF0),
-			    0,
-			    0);
-		}
 #endif
 	} else {
 #ifdef MULTIPROCESSOR
@@ -152,6 +138,20 @@ cpu_attach(device_t dv, cpuid_t id)
 	ci->ci_dev = dv;
 	dv->dv_private = ci;
 
+	if (id & MPIDR_MT) {
+		cpu_topology_set(ci,
+		    __SHIFTOUT(id, MPIDR_AFF2),
+		    __SHIFTOUT(id, MPIDR_AFF1),
+		    __SHIFTOUT(id, MPIDR_AFF0),
+		    0);
+	} else {
+		cpu_topology_set(ci,
+		    __SHIFTOUT(id, MPIDR_AFF1),
+		    __SHIFTOUT(id, MPIDR_AFF0),
+		    0,
+		    0);
+	}
+
 	evcnt_attach_dynamic(&ci->ci_arm700bugcount, EVCNT_TYPE_MISC,
 	    NULL, xname, "arm700swibug");
 

Reply via email to