Re: ACPI changes in -current, -10 vs. kernels w/o "genfb"

[Manuel Bouyer <bouyer%antioche.eu.org@localhost>]

On Fri, Oct 20, 2023 at 06:47:54PM -0500, John D. Baker wrote:
> On Thu, 19 Oct 2023, Manuel Bouyer wrote:
> 
> > On Thu, Oct 19, 2023 at 08:46:27AM -0500, John D. Baker wrote:
> >
> > > [...]
> > > #      link  VERTHANDI/netbsd
> > > /r0/build/netbsd-10/tools/amd64/bin/i486--netbsdelf-ld -Map netbsd.map --cref -T netbsd.ldscript -Ttext c0100000 -e start -X -o netbsd ${SYSTEM_OBJ:[@]:Nswapnetbsd.o} ${EXTRA_OBJ} vers.o swapnetbsd.o
> > > /r0/build/netbsd-10/tools/amd64/bin/i486--netbsdelf-ld: acpi_wakeup.o: in function `acpi_md_sleep_patch':
> > > /x/netbsd-10/src/sys/arch/x86/acpi/acpi_wakeup.c:145: undefined reference to `acpi_md_vesa_modenum'
> > > [...]
> > 
> > Hello,
> > should be fixed on HEAD, will request a pullup to netbsd-10
> 
> Thanks!  HEAD built just fine, but with -10/i386, my custom kernels build
> OK, but the stock XEN3PAE_DOM0 build fails with:
> 
> [...]
> #      link  XEN3PAE_DOM0/netbsd
> /r0/build/netbsd-10/tools/amd64/bin/i486--netbsdelf-ld -Map netbsd.map --cref -T netbsd.ldscript -Ttext 0xc0100000 -e start -X -o netbsd ${SYSTEM_OBJ:[@]:Nswapnetbsd.o} ${EXTRA_OBJ} vers.o swapnetbsd.o
> /r0/build/netbsd-10/tools/amd64/bin/i486--netbsdelf-ld: genfb_machdep.o: in function `x86_genfb_init':
> /x/netbsd-10/src/sys/arch/x86/x86/genfb_machdep.c:141: undefined reference to `acpi_md_vesa_modenum'
> [...]
> 
> I'm guessing the issue is that XEN3PAE_DOM0 has "genfb", but no ACPI
> support, so is missing the symbol.

Actually it has ACPI (which is why genfb tries to use the symbol) but
not acpi_wakeup

What's strange is that I did a full build on HEAD and didn't notice the issue.

Will look at it

-- 
Manuel Bouyer <bouyer%antioche.eu.org@localhost>
     NetBSD: 26 ans d'experience feront toujours la difference
--