Hi Jiewen,
I will start providing the feedback for this series starting today.
I may need some help to understand the sequence of the various patch series
that enable this feature and would be grateful if you could point me to a
Github branch that I can refer.
Regards,
Sami Mujawar
On 18/10/2021, 14:06, "Yao, Jiewen" <jiewen....@intel.com> wrote:
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 (#82344): https://edk2.groups.io/g/devel/message/82344
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]
-=-=-=-=-=-=-=-=-=-=-=-
