Hi, > +_Bfv: > + DD TDX_BFV_RAW_DATA_OFFSET > + DD TDX_BFV_RAW_DATA_SIZE > + DQ TDX_BFV_MEMORY_BASE > + DQ TDX_BFV_MEMORY_SIZE > + DD TDX_METADATA_SECTION_TYPE_BFV > + DD TDX_METADATA_ATTRIBUTES_EXTENDMR
Size is still added twice, doesn't make sense given that they are either equal or RAW_DATA_SIZE is zero. One size field being 32bit and the other being 64bit is pointless too (see also my mail to Jiewen). > + DD TDX_METADATA_SECTION_TYPE_TEMP_MEM There are a bunch of TEMP_MEM entries, some of them are next to each other in MEMFD, so you can squash them into one entry. Can you move the metadata changes to a separate patch please? take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#81158): https://edk2.groups.io/g/devel/message/81158 Mute This Topic: https://groups.io/mt/85891884/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
