PR/56264 CVS commit: src/sys/arch/arm/broadcom

["Rin Okuyama" <rin%netbsd.org@localhost>]

The following reply was made to PR port-arm/56264; it has been noted by GNATS.

From: "Rin Okuyama" <rin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/56264 CVS commit: src/sys/arch/arm/broadcom
Date: Wed, 1 Sep 2021 03:08:08 +0000

 Module Name:	src
 Committed By:	rin
 Date:		Wed Sep  1 03:08:08 UTC 2021
 
 Modified Files:
 	src/sys/arch/arm/broadcom: bcm2835_intr.c
 
 Log Message:
 PR port-arm/56264
 
 Register all PICs when bcmicu1 is attached, in order to avoid calling
 pic_add() from cpu_hatch(), which blocks for aarch64 kernel on RPI3.
 This prevented MP kernel to boot due to KASSERT failure as described
 in the PR.
 
 This is a kind of a workaround; the real fix should be to
 
 (a) reorganize cpu_hatch() for aarch64 and arm:
 http://mail-index.netbsd.org/port-arm/2021/06/21/msg007320.html
 
 (b) or change MI abstraction of ``MP ready'':
 http://mail-index.netbsd.org/port-arm/2021/06/22/msg007327.html
 
 However, still, this fix does not bring about any penalty, and it is
 not good to leave RPI3 broken for months...
 
 Tested on RPI3 (aarch64 MP, armv7hf MP) as well as RPI1 (armv6hf UP).
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.38 -r1.39 src/sys/arch/arm/broadcom/bcm2835_intr.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.