Re: [edk2-devel] [PATCH v3 1/4] OvmfPkg/Sec: Setup MTRR early in the boot process.

["Gerd Hoffmann" <kraxel@redhat.com>]

On Thu, Feb 01, 2024 at 10:38:43AM +0100, Gerd Hoffmann wrote:
>   Hi,
> 
> > > Can you confirm (a) this patch is OK for "OvmfPkg/IntelTdx/Sec/SecMain.c",
> > > and (b) this series fixes the slowdown you had encountered?
> > > 
> > > (that's what's left before we can merge this series)
> > > 
> > We test the patch in TDX and find EXIT_REASON_CR_ACCESS is triggered in DXE phase.
> 
> Hmm.  Sure this caused by this patch series?  For the PEI-less TDX
> build this series moves the MTRR setup to a different place in SEC.
> Once the DXE phase started the MTRR configuration should be identical
> with and without this patch series, and the series also doesn't touch
> any control register.

Ping.  Can you double-check please?  Our QE ran a test build with this
series applied through regression testing (including TDX) and has not
found any issues.

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#115352): https://edk2.groups.io/g/devel/message/115352
Mute This Topic: https://groups.io/mt/104052591/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-