On Tue, Apr 16, 2024 at 03:40:08PM +0000, Yao, Jiewen wrote: > Yeah, I also considered that before. But after look at current code > structure, I give up. > > Since following SEV component are NOT in AmdSev directory (especially the TCG > one), I do not see a strong reason to put them to IntelTdx dir. > https://github.com/tianocore/edk2/tree/master/OvmfPkg/AmdSevDxe > https://github.com/tianocore/edk2/tree/master/OvmfPkg/Tcg/TpmMmioSevDecryptPei > https://github.com/tianocore/edk2/tree/master/OvmfPkg/Library/BaseMemEncryptSevLib
Yes, existing placement is inconsistent. Some code is in AmdSev / IntelTdx subdirs, some is not. There are also some Tdx libraries in OvmfPkg/Library instead of OvmfPkg/IntelTdx > I think we can follow the existing code structure in this patch set. OK take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117900): https://edk2.groups.io/g/devel/message/117900 Mute This Topic: https://groups.io/mt/105531957/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
