Re: [edk2-devel] [PATCH V5 3/3] SecurityPkg: Support CcMeasurementProtocol in DxeTpmMeasurementLib

2021-11-09 Thread Min Xu
On November 8, 2021 10:18 PM, Sami Mujawar wrote: > Hi Min, > > Thank you for the updated patch. > > I have a minor suggestion marked inline as [SAMI]. Otherwise, these changes > look good to me. > > Reviewed-by: Sami Mujawar > > > + @retval EFI_SUCCESS Operation completed

Re: [edk2-devel] [PATCH V5 3/3] SecurityPkg: Support CcMeasurementProtocol in DxeTpmMeasurementLib

2021-11-08 Thread Sami Mujawar
Hi Min, Thank you for the updated patch. I have a minor suggestion marked inline as [SAMI]. Otherwise, these changes look good to me. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 07/11/2021 12:35 PM, Min Xu wrote: BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3625

[edk2-devel] [PATCH V5 3/3] SecurityPkg: Support CcMeasurementProtocol in DxeTpmMeasurementLib

2021-11-07 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3625 DxeTpmMeasurementLib supports TPM based measurement in DXE phase. After CcMeasurementProtocol is introduced, CC based measurement needs to be supported in DxeTpmMeasurementLib as well. A platform should have only one RTS/RTR. Only one of