From: David Gwynne <david@gwynne.id.au>
Subject: Re: cpu_xcall for amd64, was: rewrite wbinvd_on_all_cpus_acked() using cpu_xcall(9)
To: hshoexer <hshoexer@yerbouti.franken.de>
Cc: OpenBSD Tech <tech@openbsd.org>
Date: Mon, 10 Nov 2025 21:51:29 +1000



> On 10 Nov 2025, at 19:56, hshoexer <hshoexer@yerbouti.franken.de> wrote:
> 
> On Sun, Nov 09, 2025 at 09:54:23AM +1000, David Gwynne wrote:
>> ...
>> so ok on this?
> 
> I think the diff is missing a change for amd64/conf/files.amd64:
> To actually compile don't we need device cpu to depend on xcall?
> Like this:

i think whatever code depends on it should cause it to get compiled.

> 
> diff --git a/sys/arch/amd64/conf/files.amd64 b/sys/arch/amd64/conf/files.amd64
> index 630de36a01d..38d5796f650 100644
> --- a/sys/arch/amd64/conf/files.amd64
> +++ b/sys/arch/amd64/conf/files.amd64
> @@ -95,7 +95,7 @@ file arch/amd64/amd64/mpbios.c mpbios needs-flag
> file arch/amd64/amd64/mpbios_intr_fixup.c mpbios & pci
> 
> define cpu {[apid = -1]}
> -device cpu
> +device cpu: xcall
> attach cpu at mainbus
> file arch/amd64/amd64/cpu.c cpu
> 
>