Page MenuHomeFreeBSD

Update acpi.4 to document additional options to debug.acpi.disabled tunable
Needs ReviewPublic

Authored by felix.the.red_gmail.com on Jun 27 2025, 10:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 15, 11:45 PM
Unknown Object (File)
Thu, Sep 25, 11:41 AM
Unknown Object (File)
Fri, Sep 19, 9:04 AM
Unknown Object (File)
Sep 6 2025, 8:45 AM
Unknown Object (File)
Sep 3 2025, 7:26 PM
Unknown Object (File)
Aug 18 2025, 12:23 AM
Unknown Object (File)
Aug 14 2025, 9:38 AM
Unknown Object (File)
Aug 14 2025, 4:51 AM

Details

Reviewers
jkim
wulf
takawata
vexeduxr
Group Reviewers
manpages
Summary

acpi(4): Add multiple debug.acpi.disabled options that are in
the source tree but lack a description in acpi(4).

The original request was to add APEI - the ACPI Platform Error Interface.
I searched through the source tree for acpi_disable() and found several
other options that had not been included.

PR:248767

Test Plan

grep -Rn acpi_disabled, compare output to entries in "DISABLING ACPI" section.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Diff now has full context (-U 10000).

pauamma_gundo.com added inline comments.
share/man/man4/acpi.4
350

Why not use ".Pq Vt device" as just above?

354

Ditto

358

Same, for "feature"

362

Ditto

366

Ditto

572

Why remove .Xr here?

Thanks for adding the context. Adding some reviewers who may be familiar with the topic.

Apologies for the late review, I got caught up with other things while looking at this and never got back to it.

share/man/man4/acpi.4
362–364

That's not what this driver is for. Near the top of acpi_asus.c:

Driver for extra ACPI-controlled gadgets (hotkeys, leds, etc) found on
recent Asus (and Medion) laptops.

376

This doesn't disable the driver, it only disables the handling of CMOS address space events.

538

This doesn't exist anymore.

545

The driver's only purpose seems to be to expose a sysctl that allows slim bootloader (SBL) to be notified of a firmware update. Nothing to do with Sanyo?