Page MenuHomeFreeBSD
Differential D55118

acpi: Ask SPMC for D-state in s2idle instead of _S255D
Needs ReviewPublic
Actions

Authored by obiwac on Feb 5 2026, 12:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 19, 6:49 AM
Unknown Object (File)
Thu, Mar 19, 5:12 AM
Unknown Object (File)
Mon, Mar 16, 2:22 AM
Unknown Object (File)
Wed, Mar 11, 10:59 AM
Unknown Object (File)
Wed, Mar 11, 10:05 AM
Unknown Object (File)
Mon, Mar 9, 5:36 PM
Unknown Object (File)
Sun, Mar 8, 1:24 PM
Unknown Object (File)
Sun, Mar 8, 12:42 PM
View All 31 Files
Subscribers
emaste
imp

Details

Reviewers
olce
jkim
Summary

Currently, when entering s2idle, we are parsing the _S255D object to get the shallowest D-state supported by device, as acpi_stype_to_sstate() returns -1 for s2idle.

Instead, we should really be asking the SPMC device for the minimum D-state constraints for the given device, or falling back to _S3D.

To do this, a new acpi_spmc interface was added to ask for an ACPI handle's min D-state constraint, and acpi_device_pwr_for_sleep() has been refactored to attempt the SPMC path on s2idle or the _SxD path if that fails or not doing s2idle.

Relevant document: https://uefi.org/htmlspecs/ACPI_Spec_6_4_html/07_Power_and_Performance_Mgmt/device-power-management-objects.html#s3d-s3-device-state

Sponsored by: The FreeBSD Foundation

Test Plan

I need someone with working S3 on their laptop to make sure I didn't break this (again)!

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70732
Build 67615: arc lint + arc unit

Event Timeline

obiwac created this revision.Feb 5 2026, 12:13 AM
Herald added a subscriber: imp. · View Herald TranscriptFeb 5 2026, 12:13 AM
obiwac requested review of this revision.Feb 5 2026, 12:13 AM
Harbormaster completed remote builds in B70422: Diff 171184.Feb 5 2026, 12:13 AM
obiwac edited the summary of this revision. (Show Details)Feb 5 2026, 12:15 AM
obiwac added reviewers: olce, jkim.
obiwac retitled this revision from acpi: _S225D fix to acpi: Evaluate _S3D instead of _S225D when entering s2idle.
obiwac edited the summary of this revision. (Show Details)
obiwac updated this revision to Diff 171353.Feb 6 2026, 5:28 PM

Ask SPMC for D-state in s2idle instead of _S255D

Harbormaster completed remote builds in B70473: Diff 171353.Feb 6 2026, 5:28 PM
obiwac retitled this revision from acpi: Evaluate _S3D instead of _S225D when entering s2idle to acpi: Ask SPMC for D-state in s2idle instead of _S255D.Feb 6 2026, 5:32 PM
obiwac edited the summary of this revision. (Show Details)
obiwac edited the test plan for this revision. (Show Details)
emaste added a subscriber: emaste.Feb 10 2026, 3:04 PM

I need someone with working S3 on their laptop to make sure I didn't break this (again)!

I can give this a try on my WIP branch in a little while, on my Framework 13" 11th gen Intel. Currently running wipbsd-n283918-87f5b3153f15 which is a bunch of WIP from main at c59a47dc6c016dff74466cecb160459980a5d782, and S3 is working.

emaste added a comment.Feb 11 2026, 1:55 PM

Now running wipbsd-n284325-ae4a668b7590 which has been rebased on main as of 89aa8a94053fdd22ed716fdf424a2d10e70b3188 and includes this patch. CPU is 11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (2803.20-MHz K8-class CPU. S3 works as before.

obiwac updated this revision to Diff 172077.Feb 17 2026, 4:56 PM

Fix passing dev instead of handle to acpi_device_pwr_for_sleep_sxd

Harbormaster completed remote builds in B70732: Diff 172077.Feb 17 2026, 4:56 PM
obiwac updated this revision to Diff 173102.Wed, Mar 4, 1:10 PM

rebase

Harbormaster completed remote builds in B71174: Diff 173102.Wed, Mar 4, 1:10 PM
obiwac updated this revision to Diff 173103.Wed, Mar 4, 1:13 PM

Remove comment about overriding next D-state, as it was confusing and was anyway explained elsewhere.

Harbormaster completed remote builds in B71175: Diff 173103.Wed, Mar 4, 1:13 PM

Revision Contents
Changeset List

PathSize
sys/
conf/
1 line
dev/
acpica/
115 lines
20 lines
39 lines

Diff 172077

View Options

sys/conf/files

Loading...
View Options

sys/dev/acpica/acpi.c

Loading...
View Options

sys/dev/acpica/acpi_spmc.c

Loading...
View Options

sys/dev/acpica/acpi_spmc_if.m

Loading...