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)
Sun, Nov 10, 7:54 PM
Unknown Object (File)
Thu, Nov 7, 2:59 PM
Unknown Object (File)
Tue, Nov 5, 9:34 AM
Unknown Object (File)
Sat, Oct 26, 1:31 AM
Unknown Object (File)
Fri, Oct 25, 4:49 AM
Unknown Object (File)
Fri, Oct 25, 2:03 AM
Unknown Object (File)
Wed, Oct 23, 9:13 AM
Unknown Object (File)
Tue, Oct 22, 10:29 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
242

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
271

You might print the value itself in this message.

This revision was automatically updated to reflect the committed changes.