Re: [edk2] [PATCH v4 12/13] OvmfPkg/LockBoxLib: Update the comments for API UpdateLockBox()

2019-02-12 Thread Ni, Ray

Reviewed-by: Ray Ni 

--
Thanks,
Ray
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


Re: [edk2] [PATCH v4 12/13] OvmfPkg/LockBoxLib: Update the comments for API UpdateLockBox()

2019-02-11 Thread Laszlo Ersek
On 02/11/19 08:57, Hao Wu wrote:
> The previous commit:
> MdeModulePkg/SmmLockBoxLib: Support LockBox enlarge in UpdateLockBox()
> 
> adds the support to enlarge a LockBox when using the LockBoxLib API
> UpdateLockBox().
> 
> This commit is to sync the API description comment of UpdateLockBox() with
> its counterparts in MdeModulePkg.
> 
> Cc: Laszlo Ersek 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Hao Wu 
> ---
>  OvmfPkg/Library/LockBoxLib/LockBoxLib.c | 11 ---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/OvmfPkg/Library/LockBoxLib/LockBoxLib.c 
> b/OvmfPkg/Library/LockBoxLib/LockBoxLib.c
> index 45481b9230..fe9ea1b0d5 100644
> --- a/OvmfPkg/Library/LockBoxLib/LockBoxLib.c
> +++ b/OvmfPkg/Library/LockBoxLib/LockBoxLib.c
> @@ -3,7 +3,7 @@
>Library implementing the LockBox interface for OVMF
>  
>Copyright (C) 2013, Red Hat, Inc.
> -  Copyright (c) 2010 - 2014, Intel Corporation. All rights reserved.
> +  Copyright (c) 2010 - 2019, Intel Corporation. All rights reserved.
>  
>This program and the accompanying materials are licensed and made available
>under the terms and conditions of the BSD License which accompanies this
> @@ -224,8 +224,13 @@ SetLockBoxAttributes (
>@retval RETURN_INVALID_PARAMETER  the Guid is NULL, or Buffer is NULL, or
>  Length is 0.
>@retval RETURN_NOT_FOUND  the requested GUID not found.
> -  @retval RETURN_BUFFER_TOO_SMALL   the original buffer to too small to hold
> -new information.
> +  @retval RETURN_BUFFER_TOO_SMALL   for lockbox with attribute
> +LOCK_BOX_ATTRIBUTE_RESTORE_IN_PLACE, the
> +original buffer to too small to hold new
> +information.
> +  @retval RETURN_OUT_OF_RESOURCES   for lockbox with attribute
> +LOCK_BOX_ATTRIBUTE_RESTORE_IN_S3_ONLY, no
> +enough resource to save the information.
>@retval RETURN_ACCESS_DENIED  it is too late to invoke this interface
>@retval RETURN_NOT_STARTEDit is too early to invoke this interface
>@retval RETURN_UNSUPPORTEDthe service is not supported by
> 

Reviewed-by: Laszlo Ersek 

Thanks!
Laszlo
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


[edk2] [PATCH v4 12/13] OvmfPkg/LockBoxLib: Update the comments for API UpdateLockBox()

2019-02-10 Thread Hao Wu
The previous commit:
MdeModulePkg/SmmLockBoxLib: Support LockBox enlarge in UpdateLockBox()

adds the support to enlarge a LockBox when using the LockBoxLib API
UpdateLockBox().

This commit is to sync the API description comment of UpdateLockBox() with
its counterparts in MdeModulePkg.

Cc: Laszlo Ersek 
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Hao Wu 
---
 OvmfPkg/Library/LockBoxLib/LockBoxLib.c | 11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/OvmfPkg/Library/LockBoxLib/LockBoxLib.c 
b/OvmfPkg/Library/LockBoxLib/LockBoxLib.c
index 45481b9230..fe9ea1b0d5 100644
--- a/OvmfPkg/Library/LockBoxLib/LockBoxLib.c
+++ b/OvmfPkg/Library/LockBoxLib/LockBoxLib.c
@@ -3,7 +3,7 @@
   Library implementing the LockBox interface for OVMF
 
   Copyright (C) 2013, Red Hat, Inc.
-  Copyright (c) 2010 - 2014, Intel Corporation. All rights reserved.
+  Copyright (c) 2010 - 2019, Intel Corporation. All rights reserved.
 
   This program and the accompanying materials are licensed and made available
   under the terms and conditions of the BSD License which accompanies this
@@ -224,8 +224,13 @@ SetLockBoxAttributes (
   @retval RETURN_INVALID_PARAMETER  the Guid is NULL, or Buffer is NULL, or
 Length is 0.
   @retval RETURN_NOT_FOUND  the requested GUID not found.
-  @retval RETURN_BUFFER_TOO_SMALL   the original buffer to too small to hold
-new information.
+  @retval RETURN_BUFFER_TOO_SMALL   for lockbox with attribute
+LOCK_BOX_ATTRIBUTE_RESTORE_IN_PLACE, the
+original buffer to too small to hold new
+information.
+  @retval RETURN_OUT_OF_RESOURCES   for lockbox with attribute
+LOCK_BOX_ATTRIBUTE_RESTORE_IN_S3_ONLY, no
+enough resource to save the information.
   @retval RETURN_ACCESS_DENIED  it is too late to invoke this interface
   @retval RETURN_NOT_STARTEDit is too early to invoke this interface
   @retval RETURN_UNSUPPORTEDthe service is not supported by
-- 
2.12.0.windows.1

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel