fwts mtrr should test IA32_MTTR_DEF_TYPE before running the test

[Colin Ian King]

Reported on the fwts-devel list by Eric Dong:

https://lists.ubuntu.com/archives/fwts-devel/2016-July/008176.html

Hi,

I use Fwts to test the uefi bios, it reported below errors:

mtrr ;HED mtrr: MTRR tests.
mtrr ;SEP ------------------------------------------------------------
mtrr ;INF MTRR overview
mtrr ;INF -------------
mtrr ;INF Reg 0: 0x00000000c0000000 - 0x0000000100000000 ( 1024 MB) Uncached
mtrr ;INF Reg 1: 0x00000000bf800000 - 0x00000000c0000000 ( 8 MB) Uncached
mtrr ;NLN
mtrr ;INF Test 1 of 3: Validate the kernel MTRR IOMEM setup.
mtrr ;FAL FAILED [MEDIUM] MTRRIncorrectAttr: Test 1, Memory range
mtrr ;FAL 0x1000 to 0x5efff (System RAM) has incorrect attribute
mtrr ;FAL Default (Most probably Uncached).
mtrr ;FAL FAILED [MEDIUM] MTRRLackingAttr: Test 1, Memory range 0x1000
mtrr ;FAL to 0x5efff (System RAM) is lacking attribute Write-Back.

But we already update the IA32_MTRR_DEF_TYPE (0x2FF) to set the default type to Write-Back. If update the IA32_MTRR_DEF_TYPE to set the default type to Uncached. These errors will not occur. I think this is Fwts tools issue. It should check the default type from IA32_MTRR_DEF_TYPE before run this test. Any feedback about it?

Thanks,
Eric

[Ivan Hu (ivan.hu)]

Fix patches have been included to V17.06.00.