PR/57878 CVS commit: src/sys/dev/pci

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

The following reply was made to PR kern/57878; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57878 CVS commit: src/sys/dev/pci
Date: Mon, 29 Jan 2024 01:05:56 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Mon Jan 29 01:05:55 UTC 2024
 
 Modified Files:
 	src/sys/dev/pci: agp_i810.c
 
 Log Message:
 agp_i810(4): Use ipi(9) for chipset flush on all CPUs, not xcall(9).
 
 i915 now calls into this with a spin lock held, so we have to use
 ipi(9), which spin-waits for the other CPUs to complete, rather than
 xcall(9), which may sleep-wait.
 
 Fortunately, this is just to execute WBINVD on x86 (and if this code
 ever runs on other architectures, which it probably doesn't, it'll be
 a similar barrier instruction), so spinning to wait for that on all
 CPUs isn't too costly.
 
 PR kern/57878
 
 XXX pullup-10
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.125 -r1.126 src/sys/dev/pci/agp_i810.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.