From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web08.21689.1605853593364097362 for ; Thu, 19 Nov 2020 22:26:34 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Fri, 20 Nov 2020 14:26:27 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , , "'Mistry, Nishant C'" Cc: References: <661e44cf628ae7315fc738b64a52736f6b5b5285.1605047447.git.nishant.c.mistry@intel.com> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIXSBTZWN1cml0eVBrZzogQWRkIFJQTUMgSW5kZXggdG8gdGhlIFJwbWNMaWI=?= Date: Fri, 20 Nov 2020 14:26:30 +0800 Message-ID: <017b01d6bf06$1558f9f0$400aedd0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQLDCQW5h9LemGh3lNy4fUTd8kAVyQGGQFvOp+vGnuA= Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Jian: This change is like a feature instead of bug fix. Now, we are in soft feature freeze phase.=20 According to SFF definition https://github.com/tianocore/tianocore.github.io/wiki/SoftFeatureFreeze,= =20 this feature should be deferred to next stable tag.=20 So, I suggest to revert this change, and merge it after the stable tag 202011.=20 Thanks Liming > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: bounce+27952+67669+4905953+8761045@groups.io > =B4=FA=B1=ED Wang, Jian J > =B7=A2=CB=CD=CA=B1=BC=E4: 2020=C4=EA11=D4=C218=C8=D5 11:35 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io; Mistry, Nishant C > > =D6=F7=CC=E2: Re: [edk2-devel] [PATCH] SecurityPkg: Add RPMC Index to th= e RpmcLib >=20 >=20 > Reviewed-by: Jian J Wang >=20 > Regards, > Jian >=20 > > -----Original Message----- > > From: devel@edk2.groups.io On Behalf Of Nishant > > Mistry > > Sent: Thursday, November 12, 2020 2:49 AM > > To: devel@edk2.groups.io > > Subject: [edk2-devel] [PATCH] SecurityPkg: Add RPMC Index to the RpmcL= ib > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2594 > > > > The re-design requires multiple RPMC counter usages. > > The consumer will be capable of selecting amongst multiple counters. > > > > Signed-off-by: Nishant C Mistry > > --- > > SecurityPkg/Include/Library/RpmcLib.h | 6 +++++- > > SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c | 6 +++++- > > 2 files changed, 10 insertions(+), 2 deletions(-) > > > > diff --git a/SecurityPkg/Include/Library/RpmcLib.h > > b/SecurityPkg/Include/Library/RpmcLib.h > > index 5882bfae2f..3c15bce1ce 100644 > > --- a/SecurityPkg/Include/Library/RpmcLib.h > > +++ b/SecurityPkg/Include/Library/RpmcLib.h > > @@ -14,6 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent > > /** > > Requests the monotonic counter from the designated RPMC counter. > > > > + @param[in] CounterIndex The RPMC index > > @param[out] CounterValue A pointer to a buffer to > store the RPMC > > value. > > > > @retval EFI_SUCCESS The operation completed > successfully. > > @@ -23,12 +24,15 @@ SPDX-License-Identifier: BSD-2-Clause-Patent > > EFI_STATUS > > EFIAPI > > RequestMonotonicCounter ( > > + IN UINT8 CounterIndex, > > OUT UINT32 *CounterValue > > ); > > > > /** > > Increments the monotonic counter in the SPI flash device by 1. > > > > + @param[in] CounterIndex The RPMC index > > + > > @retval EFI_SUCCESS The operation completed > successfully. > > @retval EFI_DEVICE_ERROR A device error occurred > while attempting > > to update the counter. > > @retval EFI_UNSUPPORTED The operation is > un-supported. > > @@ -36,7 +40,7 @@ RequestMonotonicCounter ( > > EFI_STATUS > > EFIAPI > > IncrementMonotonicCounter ( > > - VOID > > + IN UINT8 CounterIndex > > ); > > > > #endif > > diff --git a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c > > b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c > > index e1dd09eb10..697e493a7c 100644 > > --- a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c > > +++ b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c > > @@ -12,6 +12,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent > > /** > > Requests the monotonic counter from the designated RPMC counter. > > > > + @param[in] CounterIndex The RPMC index > > @param[out] CounterValue A pointer to a buffer to > store the RPMC > > value. > > > > @retval EFI_SUCCESS The operation completed > successfully. > > @@ -21,6 +22,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent > > EFI_STATUS > > EFIAPI > > RequestMonotonicCounter ( > > + IN UINT8 CounterIndex, > > OUT UINT32 *CounterValue > > ) > > { > > @@ -31,6 +33,8 @@ RequestMonotonicCounter ( > > /** > > Increments the monotonic counter in the SPI flash device by 1. > > > > + @param[in] CounterIndex The RPMC index > > + > > @retval EFI_SUCCESS The operation completed > successfully. > > @retval EFI_DEVICE_ERROR A device error occurred > while attempting > > to update the counter. > > @retval EFI_UNSUPPORTED The operation is > un-supported. > > @@ -38,7 +42,7 @@ RequestMonotonicCounter ( > > EFI_STATUS > > EFIAPI > > IncrementMonotonicCounter ( > > - VOID > > + IN UINT8 CounterIndex > > ) > > { > > ASSERT (FALSE); > > -- > > 2.16.2.windows.1 > > > > > > > > > > >=20 >=20 >=20 >=20 >=20