On 9/16/21 3:26 AM, Gerd Hoffmann via groups.io wrote: > Hi, > >> +; GHCB page table >> +GhcbPageTable: >> + DD GHCB_PT_ADDR >> + DD GHCB_PT_SIZE >> + DD OVMF_SECTION_TYPE_SEC_MEM > Hmm, TDX will re-use those pages for something else. So as long as TDX > is happy with OVMF_SECTION_TYPE_SEC_MEM everything should work fine and > just mentioning that in the comments will be ok. > > If not we might need a SEV_SECTION_TYPE_SEC_MEM to indicate the entry is > valid for SEV only and Intel can add a separate > TDX_SECTION_TYPE_SOMETHING_ELSE entry for the same address range.
Yep, looking at the current TDX patches we see that GHCB is used for the mailbox, to make integration easy its good idea that we define SEV_SECTION_TYPE_SEC_MEM and use it for those GHCB memory pages. I will add in next version. thanks -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#80777): https://edk2.groups.io/g/devel/message/80777 Mute This Topic: https://groups.io/mt/85582693/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-