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 (#82220): https://edk2.groups.io/g/devel/message/82220
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]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to