Page MenuHomeFreeBSD

acpi_cpu: only report unmapped processor objects enabled in the MADT
ClosedPublic

Authored by seuros on Wed, Sep 9, 10:13 PM.
Referenced Files
F173248985: D59551.id186629.diff
Thu, Sep 24, 4:54 PM
F173236958: D59551.id186630.diff
Thu, Sep 24, 2:09 PM
F173214173: D59551.id186630.diff
Thu, Sep 24, 8:50 AM
F173194847: D59551.id186310.diff
Thu, Sep 24, 5:39 AM
F173189510: D59551.id.diff
Thu, Sep 24, 4:43 AM
F173186533: D59551.diff
Thu, Sep 24, 4:26 AM
Unknown Object (File)
Wed, Sep 23, 4:21 PM
Unknown Object (File)
Wed, Sep 23, 9:47 AM
Subscribers
None

Details

Summary

Firmware expose a DSDT sized for the largest SKU of the platform,
so a verbose boot prints an "ignored" line for every vacant
processor slot. A vacant slot has no enabled MADT entry; a CPU that
failed to come online does.

Diff Detail

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

Event Timeline

seuros created this revision.
adrian added a reviewer: jhb.

what platform did you see this on?

AMD64 , Raptor Lake+

Oldest machine are offline, so i didn't check.

I'd suggest some function renames, but otherwise looks fine. I might do some followup changes.

sys/dev/acpica/acpi_cpu.c
166

Suggested better name.

Also, I'd remove the p in the name, and since it's an internal function, possible the full acpi_cpu prefix as well. Applicable as well to the other added function.

167

See previous inline comment.

This revision is now accepted and ready to land.Fri, Sep 11, 10:15 AM
This revision now requires review to proceed.Sun, Sep 13, 8:46 PM
This revision is now accepted and ready to land.Sun, Sep 13, 8:47 PM