* [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to the RpmcLib"
@ 2020-11-24 1:22 gaoliming
2020-11-24 1:28 ` [edk2-devel] " Yao, Jiewen
0 siblings, 1 reply; 3+ messages in thread
From: gaoliming @ 2020-11-24 1:22 UTC (permalink / raw)
To: devel; +Cc: Jiewen Yao, Jian J Wang, Nishant C Mistry
This reverts commit 6c8dd15c4ae42501438a525ec41299f365f223cb.
Based on the discussion https://edk2.groups.io/g/devel/message/67764,
this change is regarded as the feature request. But, it doesn't pass
reviewed before 202011 stable tag soft feature freeze. So, it should
not be merged into 202011 stable tag.
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Nishant C Mistry <nishant.c.mistry@intel.com>
Signed-off-by: Liming Gao <gaoliming@byosoft.com.cn>
---
This patch targets for 202011 stable tag.
SecurityPkg/Include/Library/RpmcLib.h | 6 +-----
SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c | 6 +-----
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/SecurityPkg/Include/Library/RpmcLib.h b/SecurityPkg/Include/Library/RpmcLib.h
index 3c15bce1ce85..5882bfae2f24 100644
--- a/SecurityPkg/Include/Library/RpmcLib.h
+++ b/SecurityPkg/Include/Library/RpmcLib.h
@@ -14,7 +14,6 @@ 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.
@@ -24,15 +23,12 @@ 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.
@@ -40,7 +36,7 @@ RequestMonotonicCounter (
EFI_STATUS
EFIAPI
IncrementMonotonicCounter (
- IN UINT8 CounterIndex
+ VOID
);
#endif
diff --git a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
index 697e493a7cea..e1dd09eb10a0 100644
--- a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
+++ b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
@@ -12,7 +12,6 @@ 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.
@@ -22,7 +21,6 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
EFI_STATUS
EFIAPI
RequestMonotonicCounter (
- IN UINT8 CounterIndex,
OUT UINT32 *CounterValue
)
{
@@ -33,8 +31,6 @@ 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.
@@ -42,7 +38,7 @@ RequestMonotonicCounter (
EFI_STATUS
EFIAPI
IncrementMonotonicCounter (
- IN UINT8 CounterIndex
+ VOID
)
{
ASSERT (FALSE);
--
2.27.0.windows.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [edk2-devel] [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to the RpmcLib"
2020-11-24 1:22 [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to the RpmcLib" gaoliming
@ 2020-11-24 1:28 ` Yao, Jiewen
2020-11-26 1:11 ` 回复: " gaoliming
0 siblings, 1 reply; 3+ messages in thread
From: Yao, Jiewen @ 2020-11-24 1:28 UTC (permalink / raw)
To: devel@edk2.groups.io, gaoliming@byosoft.com.cn
Cc: Wang, Jian J, Mistry, Nishant C
Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> gaoliming
> Sent: Tuesday, November 24, 2020 9:23 AM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Mistry, Nishant C <nishant.c.mistry@intel.com>
> Subject: [edk2-devel] [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to
> the RpmcLib"
>
> This reverts commit 6c8dd15c4ae42501438a525ec41299f365f223cb.
>
> Based on the discussion https://edk2.groups.io/g/devel/message/67764,
> this change is regarded as the feature request. But, it doesn't pass
> reviewed before 202011 stable tag soft feature freeze. So, it should
> not be merged into 202011 stable tag.
>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Nishant C Mistry <nishant.c.mistry@intel.com>
> Signed-off-by: Liming Gao <gaoliming@byosoft.com.cn>
> ---
> This patch targets for 202011 stable tag.
>
> SecurityPkg/Include/Library/RpmcLib.h | 6 +-----
> SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c | 6 +-----
> 2 files changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/SecurityPkg/Include/Library/RpmcLib.h
> b/SecurityPkg/Include/Library/RpmcLib.h
> index 3c15bce1ce85..5882bfae2f24 100644
> --- a/SecurityPkg/Include/Library/RpmcLib.h
> +++ b/SecurityPkg/Include/Library/RpmcLib.h
> @@ -14,7 +14,6 @@ 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.
> @@ -24,15 +23,12 @@ 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.
> @@ -40,7 +36,7 @@ RequestMonotonicCounter (
> EFI_STATUS
> EFIAPI
> IncrementMonotonicCounter (
> - IN UINT8 CounterIndex
> + VOID
> );
>
> #endif
> diff --git a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> index 697e493a7cea..e1dd09eb10a0 100644
> --- a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> +++ b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> @@ -12,7 +12,6 @@ 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.
> @@ -22,7 +21,6 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> EFI_STATUS
> EFIAPI
> RequestMonotonicCounter (
> - IN UINT8 CounterIndex,
> OUT UINT32 *CounterValue
> )
> {
> @@ -33,8 +31,6 @@ 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.
> @@ -42,7 +38,7 @@ RequestMonotonicCounter (
> EFI_STATUS
> EFIAPI
> IncrementMonotonicCounter (
> - IN UINT8 CounterIndex
> + VOID
> )
> {
> ASSERT (FALSE);
> --
> 2.27.0.windows.1
>
>
>
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* 回复: [edk2-devel] [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to the RpmcLib"
2020-11-24 1:28 ` [edk2-devel] " Yao, Jiewen
@ 2020-11-26 1:11 ` gaoliming
0 siblings, 0 replies; 3+ messages in thread
From: gaoliming @ 2020-11-26 1:11 UTC (permalink / raw)
To: devel, jiewen.yao; +Cc: 'Wang, Jian J', 'Mistry, Nishant C'
Create PR https://github.com/tianocore/edk2/pull/1146
Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+67855+4905953+8761045@groups.io
> <bounce+27952+67855+4905953+8761045@groups.io> 代表 Yao, Jiewen
> 发送时间: 2020年11月24日 9:29
> 收件人: devel@edk2.groups.io; gaoliming@byosoft.com.cn
> 抄送: Wang, Jian J <jian.j.wang@intel.com>; Mistry, Nishant C
> <nishant.c.mistry@intel.com>
> 主题: Re: [edk2-devel] [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to
> the RpmcLib"
>
> Acked-by: Jiewen Yao <Jiewen.yao@intel.com>
> Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > gaoliming
> > Sent: Tuesday, November 24, 2020 9:23 AM
> > To: devel@edk2.groups.io
> > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> > <jian.j.wang@intel.com>; Mistry, Nishant C <nishant.c.mistry@intel.com>
> > Subject: [edk2-devel] [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to
> > the RpmcLib"
> >
> > This reverts commit 6c8dd15c4ae42501438a525ec41299f365f223cb.
> >
> > Based on the discussion https://edk2.groups.io/g/devel/message/67764,
> > this change is regarded as the feature request. But, it doesn't pass
> > reviewed before 202011 stable tag soft feature freeze. So, it should
> > not be merged into 202011 stable tag.
> >
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > Cc: Nishant C Mistry <nishant.c.mistry@intel.com>
> > Signed-off-by: Liming Gao <gaoliming@byosoft.com.cn>
> > ---
> > This patch targets for 202011 stable tag.
> >
> > SecurityPkg/Include/Library/RpmcLib.h | 6 +-----
> > SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c | 6 +-----
> > 2 files changed, 2 insertions(+), 10 deletions(-)
> >
> > diff --git a/SecurityPkg/Include/Library/RpmcLib.h
> > b/SecurityPkg/Include/Library/RpmcLib.h
> > index 3c15bce1ce85..5882bfae2f24 100644
> > --- a/SecurityPkg/Include/Library/RpmcLib.h
> > +++ b/SecurityPkg/Include/Library/RpmcLib.h
> > @@ -14,7 +14,6 @@ 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.
> > @@ -24,15 +23,12 @@ 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.
> > @@ -40,7 +36,7 @@ RequestMonotonicCounter (
> > EFI_STATUS
> > EFIAPI
> > IncrementMonotonicCounter (
> > - IN UINT8 CounterIndex
> > + VOID
> > );
> >
> > #endif
> > diff --git a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> > b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> > index 697e493a7cea..e1dd09eb10a0 100644
> > --- a/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> > +++ b/SecurityPkg/Library/RpmcLibNull/RpmcLibNull.c
> > @@ -12,7 +12,6 @@ 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.
> > @@ -22,7 +21,6 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> > EFI_STATUS
> > EFIAPI
> > RequestMonotonicCounter (
> > - IN UINT8 CounterIndex,
> > OUT UINT32 *CounterValue
> > )
> > {
> > @@ -33,8 +31,6 @@ 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.
> > @@ -42,7 +38,7 @@ RequestMonotonicCounter (
> > EFI_STATUS
> > EFIAPI
> > IncrementMonotonicCounter (
> > - IN UINT8 CounterIndex
> > + VOID
> > )
> > {
> > ASSERT (FALSE);
> > --
> > 2.27.0.windows.1
> >
> >
> >
> >
> >
> >
>
>
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-11-26 1:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-24 1:22 [PATCH 1/1] Revert "SecurityPkg: Add RPMC Index to the RpmcLib" gaoliming
2020-11-24 1:28 ` [edk2-devel] " Yao, Jiewen
2020-11-26 1:11 ` 回复: " gaoliming
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox