Paths

Table of Contentst

Differential D56650

acpi: Remove userland bits on non-ACPI platforms
ClosedPublic
Actions

Authored by ivy on Apr 26 2026, 10:17 PM.

Tags

None

Referenced Files

	F175021007: D56650.id176588.diff
	Wed, Oct 7, 4:53 PM

	Unknown Object (File)
	Fri, Oct 2, 2:10 AM

	Unknown Object (File)
	Tue, Sep 29, 12:05 PM

	Unknown Object (File)
	Tue, Sep 29, 3:20 AM

	Unknown Object (File)
	Sun, Sep 27, 10:10 PM

	Unknown Object (File)
	Wed, Sep 23, 7:42 PM

	Unknown Object (File)
	Wed, Sep 23, 5:06 PM

	Unknown Object (File)
	Sep 4 2026, 7:47 AM

View All 68 Files

Subscribers

Details

Reviewers

jhb
kib
imp
andrew

Group Reviewers

Commits

rG58c08517ce5d: acpi: Remove userland bits on non-ACPI platforms
rGc4b244af42a1: acpi: Remove userland bits on non-ACPI platforms

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 created this revision.Apr 26 2026, 10:17 PM

Herald added a reviewer: andrew. · View Herald TranscriptApr 26 2026, 10:17 PM

Herald added a subscriber: asomers. · View Herald Transcript

ivy requested review of this revision.Apr 26 2026, 10:17 PM

Harbormaster completed remote builds in B72558: Diff 176588.Apr 26 2026, 10:17 PM

imp added inline comments.Apr 26 2026, 10:30 PM

libexec/rc/rc.d/Makefile
92	I thought armv7 also did ACPI. Am I mistaken?

ivy added inline comments.Apr 26 2026, 10:46 PM

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

Closed by commit rGc4b244af42a1: acpi: Remove userland bits on non-ACPI platforms (authored by ivy). · Explain WhyApr 28 2026, 7:51 PM

This revision was automatically updated to reflect the committed changes.

ivy added a commit: rGc4b244af42a1: acpi: Remove userland bits on non-ACPI platforms.

ivy added a commit: rG58c08517ce5d: acpi: Remove userland bits on non-ACPI platforms.Jul 25 2026, 1:08 PM

Revision Contents
Changeset List

Path

Size

libexec/

rc/

rc.d/

3 lines

packages/

Makefile.powerpc

Makefile.riscv64

1 line

acpi/

6 lines

sbin/

devd/

3 lines

tools/

build/

mk/

OptionalObsoleteFiles.inc

4 lines

Diff 176757

libexec/rc/rc.d/Makefile

Loading...

packages/Makefile.arm

Loading...

packages/Makefile.powerpc

Loading...

packages/Makefile.riscv64

Loading...

packages/acpi/Makefile

Loading...

sbin/devd/Makefile

Loading...

tools/build/mk/OptionalObsoleteFiles.inc

Loading...