Question about mtrr test

[Dong, Eric]

Colin,

Thanks for your quick response.

> -----Original Message-----
> From: Colin Ian King [mailto:colin.king at canonical.com]
> Sent: Thursday, July 14, 2016 6:27 PM
> To: Dong, Eric; fwts-devel at lists.ubuntu.com
> Subject: Re: Question about mtrr test
> 
> On 14/07/16 11:08, Dong, Eric wrote:
> > I use LUV v2.1-rc1 image to do the test, not sure the Fwts version. I think it is Fwts V16.06.00.
> >
> >
> >> -----Original Message-----
> >> From: Colin Ian King [mailto:colin.king at canonical.com]
> >> Sent: Thursday, July 14, 2016 6:00 PM
> >> To: Dong, Eric; fwts-devel at lists.ubuntu.com
> >> Subject: Re: Question about mtrr test
> >>
> >> On 14/07/16 10:58, Dong, Eric wrote:
> >>> 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?
> 
> OK, I think thats a bug, thanks for reporting that. I've filed it under:
> https://bugs.launchpad.net/ubuntu/+source/fwts/+bug/1603026
> 
> Any ideas Ivan/Alex?
> 
> Colin
> >>>
> >>>
> >>>
> >>> Thanks,
> >>>
> >>> Eric
> >>>
> >>>
> >>>
> >>> This body part will be downloaded on demand.
> >>>
> >>
> >> Eric, which version of fwts are you using?
> >>
> >