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

2024-06-03 Thread Gerd Hoffmann
Hi, > > I have a draft PR open with this fixed: > > https://github.com/tianocore/edk2/pull/5696 > > > > Can you check this works for bhyve? > > Works. Thanks! Thanks for testing, flipped PR to 'Ready'. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all

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

2024-06-03 Thread Corvin Köhne
On Thu, 2024-05-30 at 11:03 +0200, Gerd Hoffmann wrote: >   Hi, > > > > -    SetMem (, sizeof MtrrSettings.Fixed, > > > 0x06); > > > +    SetMem (, sizeof MtrrSettings.Fixed, > > > MTRR_CACHE_WRITE_BACK); > > > ZeroMem (, sizeof > > > MtrrSettings.Variables); > > > -   

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

2024-05-30 Thread Gerd Hoffmann
Hi, > > -    SetMem (, sizeof MtrrSettings.Fixed, 0x06); > > +    SetMem (, sizeof MtrrSettings.Fixed, > > MTRR_CACHE_WRITE_BACK); > > ZeroMem (, sizeof > > MtrrSettings.Variables); > > -    MtrrSettings.MtrrDefType |= BIT11 | BIT10 | 6; > > +    MtrrSettings.MtrrDefType |= BIT10; > >

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

2024-05-22 Thread Corvin Köhne
On Tue, 2024-01-30 at 14:04 +0100, Gerd Hoffmann wrote: > Specifically before running lzma uncompress of the main firmware > volume. > This is needed to make sure caching is enabled, otherwise the > uncompress > can be extremely slow. > > Adapt the ASSERTs and MTRR setup in PlatformInitLib to the

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

2024-04-14 Thread Min Xu
On Thursday, April 11, 2024 4:13 PM, Gerd Hoffmann wrote: > > On my end the state of affairs is unchanged. Our builds have the patch > included and there are zero problems so far, the issue reported by Min > doesn't reproduce and it is still unclear what is going on. > > Min, any update? > Hi,

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

2024-04-11 Thread Gerd Hoffmann
Hi, > > > @Gerd, what's the qemu command and test environment your QE > > > run the case? We'd like run it in our side. > > > > > > > > Tested edk2-ovmf-20231122-1.el9.rhel21704.20240202.1130.noarch with > > TDX guest, no issue found > > > > Version: > > > >

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

2024-04-11 Thread Corvin Köhne
On Tue, 2024-02-20 at 09:15 +0100, Gerd Hoffmann wrote: > On Tue, Feb 20, 2024 at 06:27:21AM +, Min Xu wrote: > > On Monday, February 12, 2024 11:22 PM, Gerd Hoffmann wrote: > > > On Thu, Feb 01, 2024 at 10:38:43AM +0100, Gerd Hoffmann wrote: > > > >   Hi, > > > > > > > > > > Can you confirm

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

2024-02-20 Thread Gerd Hoffmann
On Tue, Feb 20, 2024 at 06:27:21AM +, Min Xu wrote: > On Monday, February 12, 2024 11:22 PM, Gerd Hoffmann wrote: > > 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",

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

2024-02-19 Thread Min Xu
On Monday, February 12, 2024 11:22 PM, Gerd Hoffmann wrote: > 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? > > >