Yes, of course it did.

file    dev/acpi/acpi_x86.c                     acpi & suspend & !small_kernel

The other ways of doing this will require a few #ifdef

>On Wed, Jul 05, 2023 at 12:51:56PM -0600, Tobias Heider wrote:
>> CVSROOT:     /cvs
>> Module name: src
>> Changes by:  to...@cvs.openbsd.org   2023/07/05 12:51:56
>> 
>> Modified files:
>>      sys/dev/acpi   : acpi.c acpi_x86.c 
>> 
>> Log message:
>> Move acpiioctl to acpi_x86.c, it is only used up on i386 and amd64.
>> 
>> ok kettenis@ deraadt@
>
>This broke the amd64 ramdisk.
>
>ld -T ld.script -X --warn-common -nopie -o bsd ${SYSTEM_HEAD} vers.o ${OBJS}
>ld: error: undefined symbol: acpiioctl
>>>> referenced by conf.c
>>>>               conf.o:(cdevsw)
>*** Error 1 in /home/src2/sys/arch/amd64/compile/RAMDISK_CD (Makefile:610 
>'bsd': @echo ld -T ld.script -X --warn-common -nopie -o bsd '${SYS...)
>*** Error 2 in /home/src2/distrib/amd64/ramdisk_cd (Makefile:71 'bsd')
>*** Error 2 in /home/src2/distrib/amd64 (<bsd.subdir.mk>:48 'all')
>*** Error 2 in /home/src2/distrib (<bsd.subdir.mk>:48 'all')
>*** Error 2 in . (Makefile:301 'distrib')
>*** Error 2 in . (Makefile:277 'do-release')
>*** Error 2 in /home/src2/etc (Makefile:260 'release')
>
>

Reply via email to