PR/58561 CVS commit: src/sys/arch

["Manuel Bouyer" <bouyer%netbsd.org@localhost>]

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

From: "Manuel Bouyer" <bouyer%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/58561 CVS commit: src/sys/arch
Date: Fri, 20 Jun 2025 17:02:18 +0000

 Module Name:	src
 Committed By:	bouyer
 Date:		Fri Jun 20 17:02:18 UTC 2025
 
 Modified Files:
 	src/sys/arch/amd64/amd64: trap.c
 	src/sys/arch/i386/i386: trap.c
 
 Log Message:
 Remove #ifndef XENPV around x86_disable_intr()/enable_intr() in trap().
 This was added by ad@ in trap.c 1.46 (in Apr 2008) as part of the
 kernel preemption work. AFAIK trap() is always called with events
 enabled on Xen, so I can't see why Xen wouldn't need x86_disable_intr()
 when bare metal needs it.
 
 Pointed out by riastradh@ in PR port-xen/58561, but it may be completely
 irrelevant to the panic reported here.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.129 -r1.130 src/sys/arch/amd64/amd64/trap.c
 cvs rdiff -u -r1.309 -r1.310 src/sys/arch/i386/i386/trap.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.