On August 31, 2021 1:35 PM, Gerd Hoffmann wrote:
> On Tue, Aug 31, 2021 at 03:09:08AM +0000, Xu, Min M wrote:
> > On Monday, August 30, 2021 3:41 PM, Gerd Hoffmann wrote:
> > >   Hi,
> > >
> > > >  _TdxPageTable:
> > > >     If 5-level page table is supported (GPAW is 52), a top level page
> > > >     directory pointers (1 * 256TB entry) is generated in this page.
> > > >  _OvmfPageTable:
> > > >     Initial page table for standard Ovmf.
> > >
> > > Hmm, isn't 5-level paging independent from TDX?  Why mix the two?
> > >
> > > I think a top level page directory should be added to the standard
> > > ovmf initial page tables instead, and setting up 5-level paging
> > > should not happen in tdx- specific code.
> > In current Ovmf implementation
> > (OvmfPkg/ResetVector/Ia32/PageTables64.asm)
> > there are 6 pages reserved for initial page tables. It doesn't support 
> > 5-level
> paging.
> 
> Sure.  And I think we should add proper 5-level paging support to the current
> ovmf implementation instead of adding hacks to the tdx code.
My understanding is that we should first add 5-level paging support in OVMF, 
right?
I am planning to add 5-level paging in OvmfPkgX64.dsc. Any comments?
> 
> take care,
>   Gerd
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#80142): https://edk2.groups.io/g/devel/message/80142
Mute This Topic: https://groups.io/mt/85242569/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to