Re: [edk2] [PATCH] UefiCpuPkg/CpuMpPei: Correct BIST PPI logic.

2018-07-23 Thread Dong, Eric 
Reviewed-by: Eric Dong 

And pushed:

SHA-1: 005c855dc6be0f61f76de0d7ec4a62ee737518d6

* UefiCpuPkg/CpuMpPei: Correct BIST PPI logic.

Currently, the SecPlatformInformation2 PPI is installed when either
there is none present or the present one doesn't lack data.
Update the logic to only install the SecPlatformInformation2 PPI when
it's not already installed so that an up-to-date PPI remains the only
one and unchanged.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marvin Haeuser 
Reviewed-by: Laszlo Ersek 
Reviewed-by: Eric Dong 


Thanks,
Eric

> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Marvin Häuser
> Sent: Sunday, July 22, 2018 6:17 AM
> To: edk2-devel@lists.01.org
> Cc: ler...@redhat.com; Dong, Eric 
> Subject: [edk2] [PATCH] UefiCpuPkg/CpuMpPei: Correct BIST PPI logic.
> 
> Currently, the SecPlatformInformation2 PPI is installed when either there is
> none present or the present one doesn't lack data.
> Update the logic to only install the SecPlatformInformation2 PPI when it's not
> already installed so that an up-to-date PPI remains the only one and
> unchanged.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marvin Haeuser 
> ---
>  UefiCpuPkg/CpuMpPei/CpuBist.c | 22 +++-
>  1 file changed, 12 insertions(+), 10 deletions(-)
> 
> diff --git a/UefiCpuPkg/CpuMpPei/CpuBist.c
> b/UefiCpuPkg/CpuMpPei/CpuBist.c index 20728525e2d9..5312d9f01dc8
> 100644
> --- a/UefiCpuPkg/CpuMpPei/CpuBist.c
> +++ b/UefiCpuPkg/CpuMpPei/CpuBist.c
> @@ -275,18 +275,20 @@ CollectBistDataFromPpi (
>  (UINTN) BistInformationSize
>  );
> 
> -  if (SecPlatformInformation2 != NULL && NumberOfData <
> NumberOfProcessors) {
> -//
> -// Reinstall SecPlatformInformation2 PPI to include new BIST information
> -//
> -Status = PeiServicesReInstallPpi (
> -   SecInformationDescriptor,
> -   
> -   );
> -ASSERT_EFI_ERROR (Status);
> +  if (SecPlatformInformation2 != NULL) {
> +if (NumberOfData < NumberOfProcessors) {
> +  //
> +  // Reinstall SecPlatformInformation2 PPI to include new BIST 
> information
> +  //
> +  Status = PeiServicesReInstallPpi (
> + SecInformationDescriptor,
> + 
> + );
> +  ASSERT_EFI_ERROR (Status);
> +}
>} else {
>  //
> -// Install SecPlatformInformation2 PPI to include new BIST information
> +// Install SecPlatformInformation2 PPI
>  //
>  Status = PeiServicesInstallPpi ();
>  ASSERT_EFI_ERROR(Status);
> --
> 2.18.0.windows.1
> 
> ___
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH] UefiCpuPkg/CpuMpPei: Correct BIST PPI logic.

2018-07-23 Thread Laszlo Ersek 
On 07/22/18 00:17, Marvin Häuser wrote:
> Currently, the SecPlatformInformation2 PPI is installed when either
> there is none present or the present one doesn't lack data.
> Update the logic to only install the SecPlatformInformation2 PPI when
> it's not already installed so that an up-to-date PPI remains the only
> one and unchanged.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Marvin Haeuser 
> ---
>  UefiCpuPkg/CpuMpPei/CpuBist.c | 22 +++-
>  1 file changed, 12 insertions(+), 10 deletions(-)
> 
> diff --git a/UefiCpuPkg/CpuMpPei/CpuBist.c b/UefiCpuPkg/CpuMpPei/CpuBist.c
> index 20728525e2d9..5312d9f01dc8 100644
> --- a/UefiCpuPkg/CpuMpPei/CpuBist.c
> +++ b/UefiCpuPkg/CpuMpPei/CpuBist.c
> @@ -275,18 +275,20 @@ CollectBistDataFromPpi (
>  (UINTN) BistInformationSize
>  );
>  
> -  if (SecPlatformInformation2 != NULL && NumberOfData < NumberOfProcessors) {
> -//
> -// Reinstall SecPlatformInformation2 PPI to include new BIST information
> -//
> -Status = PeiServicesReInstallPpi (
> -   SecInformationDescriptor,
> -   
> -   );
> -ASSERT_EFI_ERROR (Status);
> +  if (SecPlatformInformation2 != NULL) {
> +if (NumberOfData < NumberOfProcessors) {
> +  //
> +  // Reinstall SecPlatformInformation2 PPI to include new BIST 
> information
> +  //
> +  Status = PeiServicesReInstallPpi (
> + SecInformationDescriptor,
> + 
> + );
> +  ASSERT_EFI_ERROR (Status);
> +}
>} else {
>  //
> -// Install SecPlatformInformation2 PPI to include new BIST information
> +// Install SecPlatformInformation2 PPI
>  //
>  Status = PeiServicesInstallPpi ();
>  ASSERT_EFI_ERROR(Status);
> 

Looks good to me:

Reviewed-by: Laszlo Ersek 

but Eric should decide.

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

[edk2] [PATCH] UefiCpuPkg/CpuMpPei: Correct BIST PPI logic.

2018-07-21 Thread Marvin Häuser 
Currently, the SecPlatformInformation2 PPI is installed when either
there is none present or the present one doesn't lack data.
Update the logic to only install the SecPlatformInformation2 PPI when
it's not already installed so that an up-to-date PPI remains the only
one and unchanged.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marvin Haeuser 
---
 UefiCpuPkg/CpuMpPei/CpuBist.c | 22 +++-
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/UefiCpuPkg/CpuMpPei/CpuBist.c b/UefiCpuPkg/CpuMpPei/CpuBist.c
index 20728525e2d9..5312d9f01dc8 100644
--- a/UefiCpuPkg/CpuMpPei/CpuBist.c
+++ b/UefiCpuPkg/CpuMpPei/CpuBist.c
@@ -275,18 +275,20 @@ CollectBistDataFromPpi (
 (UINTN) BistInformationSize
 );
 
-  if (SecPlatformInformation2 != NULL && NumberOfData < NumberOfProcessors) {
-//
-// Reinstall SecPlatformInformation2 PPI to include new BIST information
-//
-Status = PeiServicesReInstallPpi (
-   SecInformationDescriptor,
-   
-   );
-ASSERT_EFI_ERROR (Status);
+  if (SecPlatformInformation2 != NULL) {
+if (NumberOfData < NumberOfProcessors) {
+  //
+  // Reinstall SecPlatformInformation2 PPI to include new BIST information
+  //
+  Status = PeiServicesReInstallPpi (
+ SecInformationDescriptor,
+ 
+ );
+  ASSERT_EFI_ERROR (Status);
+}
   } else {
 //
-// Install SecPlatformInformation2 PPI to include new BIST information
+// Install SecPlatformInformation2 PPI
 //
 Status = PeiServicesInstallPpi ();
 ASSERT_EFI_ERROR(Status);
-- 
2.18.0.windows.1

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