[coreboot-gerrit] Change in coreboot[master]: cpu/x86/mtrr: add support for address space higher than 16TiB

[Jonathan Zhang (Code Review)]

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46435 )

Change subject: cpu/x86/mtrr: add support for address space higher than 16TiB
......................................................................

Patch Set 7:

(2 comments)

https://review.coreboot.org/c/coreboot/+/46435/7/src/cpu/x86/mtrr/mtrr.c 
File src/cpu/x86/mtrr/mtrr.c:

https://review.coreboot.org/c/coreboot/+/46435/7/src/cpu/x86/mtrr/mtrr.c@487 
PS7, Line 487: 			mtrr_size = 1 << size_msb;
> We need to make these 1ULL as well […]

Sorry for missing this. I submitted [CB:46711].

https://review.coreboot.org/c/coreboot/+/46435/7/src/cpu/x86/mtrr/mtrr.c@489 
PS7, Line 489: 			mtrr_size = 1 << addr_lsb;
> ** CID 1435825:  Integer handling issues  (OVERFLOW_BEFORE_WIDEN) […]

Sorry for missing this. I submitted [CB:46711].