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
F172573266: D59551.id186629.diff
Sat, Sep 19, 9:03 AM
F172549710: D59551.id186630.diff
Sat, Sep 19, 4:55 AM
F172527024: D59551.id186629.diff
Sat, Sep 19, 1:32 AM
Unknown Object (File)
Fri, Sep 18, 8:51 PM
Unknown Object (File)
Fri, Sep 18, 3:27 PM
Unknown Object (File)
Thu, Sep 17, 6:11 PM
Unknown Object (File)
Thu, Sep 17, 6:01 PM
Unknown Object (File)
Wed, Sep 16, 9:23 PM
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