Sure. It works fine. May I know *when* I can get your initial feedback ? I hope in days or 1 or 2 weeks.
I don’t want to wait for several months, as we have features depend upon it. Thank you Yao Jiewen > -----Original Message----- > From: Sami Mujawar <sami.muja...@arm.com> > Sent: Monday, October 18, 2021 9:00 PM > To: Yao, Jiewen <jiewen....@intel.com>; devel@edk2.groups.io; Xu, Min M > <min.m...@intel.com> > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Liming Gao > <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; Wang, > Jian J <jian.j.w...@intel.com>; Lu, Ken <ken...@intel.com>; nd <n...@arm.com> > Subject: Re: [edk2-devel] [PATCH V2 0/3] Introduce TdProtocol into EDK2 > > Hi Jiewen, > > We don't have publicly available documentation to share in this area, however > we strongly prefer an architecture agnostic solution, and my initial > suggestions > takes us in that direction. I am happy to work on mailing list with you to > make > these changes and review necessary code changes. Does that work for you? > > Regards, > > Sami Mujawar > > On 14/10/2021, 17:01, "Yao, Jiewen" <jiewen....@intel.com> wrote: > > Hi Sami > To clarify my description: > I am OK to define it in an architecture neutral protocol, such as > EFI_TEE_MEASUREMENT_PROTOCOL, or EFI_CCAM_PROTOCOL. I am happy to > do that. > > However, at current point of time, I am not sure how other arch supports > those feature, such as > AMD SEV (https://www.amd.com/system/files/TechDocs/56860.pdf), or ARM > Realm (https://developer.arm.com/documentation/ddi0615/latest/). I did not > find runtime measurement there. > > I hope SEV/Realm people to propose what interface change is required. I am > happy to discuss the solution here. > > Thank you > Yao Jiewen > > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao, > Jiewen > > Sent: Thursday, October 14, 2021 7:59 PM > > To: Xu, Min M <min.m...@intel.com>; Sami Mujawar > > <sami.muja...@arm.com>; devel@edk2.groups.io > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Liming Gao > > <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; > Wang, > > Jian J <jian.j.w...@intel.com>; Lu, Ken <ken...@intel.com>; nd > <n...@arm.com> > > Subject: Re: [edk2-devel] [PATCH V2 0/3] Introduce TdProtocol into EDK2 > > > > Hi Sami > > I am not sure if I can understand your comment - > > "Some interfaces may need to use an architecture specific library, and > some > > configuration options would need to be defined using PCDs." > > > > Would you please be more specific? > > > > Thank you > > Yao Jiewen > > > > > > > -----Original Message----- > > > From: Xu, Min M <min.m...@intel.com> > > > Sent: Thursday, October 14, 2021 1:41 PM > > > To: Sami Mujawar <sami.muja...@arm.com>; devel@edk2.groups.io; > Yao, > > > Jiewen <jiewen....@intel.com> > > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Liming Gao > > > <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; > Wang, > > > Jian J <jian.j.w...@intel.com>; Lu, Ken <ken...@intel.com>; nd > > <n...@arm.com> > > > Subject: RE: [edk2-devel] [PATCH V2 0/3] Introduce TdProtocol into > EDK2 > > > > > > On October 12, 2021 11:27 PM, Sami Mujawar wrote: > > > > Hi Min, > > > > > > > > Thank you for this patch. > > > > > > > > I think it would greatly help if the EFI_TD_PROTOCOL is changed to > > something > > > > more architecture neutral. As I understand, this patch series is > removing > the > > > > dependency on TPM for measurement and is instead providing a > lightweight > > > > interface for extending measurements for Confidential Compute > > Architecture > > > > (CCA) guests. > > > > > > > > Considering this, it would be good to generalise EFI_TD_PROTOCOL as > a > > > > Confidential Compute Architecture Measurement (CCAM) protocol. > > > > In fact, your v2 series demonstrates this need with the > introduction of > > > > MEASURE_BOOT_PROTOCOLS in "[PATCH V2 2/3] SecurityPkg: Support > > > > TdProtocol in DxeTpm2MeasureBootLib > > > > [https://edk2.groups.io/g/devel/message/81651]";. > > > > > > > > As it stands, I feel most of the code can be reused/common. Some > > interfaces > > > > may need to use an architecture specific library, and some > configuration > > > > options would need to be defined using PCDs. > > > > > > > > Kindly let me know your thoughts. > > > > > > > Thanks for your comments. Let me first discuss your feedback with our > > > architecture. We will reply to your proposal a bit later. > > > > > > Thanks. > > > Min > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#82221): https://edk2.groups.io/g/devel/message/82221 Mute This Topic: https://groups.io/mt/86163957/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
