Page MenuHomeFreeBSD

arm64/acpi: Give the real PA limit to ACPI
ClosedPublic

Authored by scottph on Aug 20 2020, 6:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 24, 11:04 AM
Unknown Object (File)
Sun, Sep 22, 3:59 PM
Unknown Object (File)
Sat, Sep 21, 6:31 PM
Unknown Object (File)
Sat, Sep 21, 12:54 AM
Unknown Object (File)
Thu, Sep 19, 4:46 PM
Unknown Object (File)
Thu, Sep 19, 7:24 AM
Unknown Object (File)
Thu, Sep 19, 7:24 AM
Unknown Object (File)
Wed, Sep 18, 2:34 PM
Subscribers

Details

Summary

Read PA bits from ID_AA64MMFR0_EL1.PARange.

MFC after: 1 week
Sponsored by: Ampere Computing, Inc.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

You should use get_kernel_reg to read the ID registers when we need a common version over all CPUs. It's not going to affect it here, but when use later in the boot it can as on a big.LITTLE system these ID registers can sometimes be different across the different core types.

sys/arm64/acpica/acpi_machdep.c
252

This should be ID_AA64MMFR0_PARange_VAL(mmfr0) Maybe with >> ID_AA64MMFR0_PARange_SHIFT, although the shift is 0 so it might be easier to add a comment to make this knowledge explicit.

scottph marked an inline comment as done.
This revision is now accepted and ready to land.Aug 21 2020, 9:55 AM
markj added inline comments.
sys/arm64/acpica/acpi_machdep.c
281

You might print the value itself in this message.

This revision was automatically updated to reflect the committed changes.