Hello build bot (Jenkins), Marc Jones, Anjaneya "Reddy" Chagam, Johnny Lin, Rocky Phagura, Aaron Durbin, Tim Chu,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/46435
to look at the new patch set (#3).
Change subject: cpu/x86/mtrr: add support for address space higher than 16TiB ......................................................................
cpu/x86/mtrr: add support for address space higher than 16TiB
On DeltaLake server, there are following entry in MTRR address space: 0x0000201000000000 - 0x0000201000400000 size 0x00400000 type 0
In this case, the base address (with 4k granularity) cannot be held in uint32_t. This results incorrect MTRR register setup. As the consequence UEFI forum FWTS reports following critical error: Memory range 0x100000000 to 0x183fffffff (System RAM) has incorrect attribute Uncached.
Change appropriate variables' data type from uint32_t to uint64_t. Add fls64() to find least significant bit set in a 64-bit word. Add fms64() to find most significant bit set in a 64-bit word.
Signed-off-by: Jonathan Zhang jonzhang@fb.com Signed-off-by: Marc Jones marcjones@sysproconsulting.com Change-Id: I41bc5befcc1374c838c91b9f7c5279ea76dd67c7 --- M src/cpu/x86/mtrr/mtrr.c 1 file changed, 38 insertions(+), 15 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/46435/3