PR/57661 CVS commit: src/sys/arch/amd64/include

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

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

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57661 CVS commit: src/sys/arch/amd64/include
Date: Sun, 27 Apr 2025 01:32:09 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Sun Apr 27 01:32:09 UTC 2025
 
 Modified Files:
 	src/sys/arch/amd64/include: param.h
 
 Log Message:
 amd64/param.h: Fix KASAN/KMSAN build (and ALL build by extension).
 
 Make UPAGES match what it was before my recent changes, as verified
 by the __CTASSERT below, justifying the existence of the __CTASSERT.
 
 As I recall, SVS is incompatible with KASAN/KMSAN, so it doesn't
 contribute to the sum; presumably KASAN/KMSAN requires three pages,
 though I'm not sure where this is documented.  If it turns out this
 accounting is wrong, we should fix it and cross-reference any relevant
 constraints affecting the accounting.  (But for now I'm just making
 sure to restore the status quo of these definitions, which was my
 intent all along with adding the __CTASSERT.)
 
 Issue noted by hannken@.
 
 PR port-amd64/57661: Crash when booting on Xeon Silver 4416+ in
 KVM/Qemu
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.41 -r1.42 src/sys/arch/amd64/include/param.h
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.