Page MenuHomeFreeBSD

acpi: Remove userland bits on non-ACPI platforms
ClosedPublic

Authored by ivy on Apr 26 2026, 10:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 16, 4:08 AM
Unknown Object (File)
Sat, Jun 13, 3:24 PM
Unknown Object (File)
Thu, Jun 11, 8:26 AM
Unknown Object (File)
Thu, Jun 4, 3:36 PM
Unknown Object (File)
Wed, May 27, 5:49 AM
Unknown Object (File)
May 17 2026, 9:10 PM
Unknown Object (File)
May 17 2026, 9:05 PM
Unknown Object (File)
May 17 2026, 4:04 PM
Subscribers

Details

Summary

ACPI is only supported on amd64, arm64 and i386. Don't install the
power_profile rc script or devd configuration on other platforms.

This avoids creating a useless FreeBSD-acpi package on those platforms.

MFC after: 2 weeks
Sponsored by: https://www.patreon.com/bsdivy

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ivy requested review of this revision.Apr 26 2026, 10:17 PM
libexec/rc/rc.d/Makefile
92

I thought armv7 also did ACPI. Am I mistaken?

libexec/rc/rc.d/Makefile
92

we don't ship the other ACPI userland bits on armv7 (see usr.sbin/Makefile.*), and none of the kernel configs in sys/arm/conf include device acpi, so it doesn't currently seem to be supported.

This looks great!

libexec/rc/rc.d/Makefile
92

Ah! We share some files between armv7 and aarch64 that have ACPI in them. And while we boot with UEFI, we don't use ACPI on armv7. So I am mistaken on this.

This revision is now accepted and ready to land.Apr 27 2026, 12:33 AM
This revision was automatically updated to reflect the committed changes.