Re: [edk2] [PATCH] ArmPkg/ArmLib: remove pointless sync barriers after each cache op

Hi Ard, On Mon, Nov 02, 2015 at 02:24:18PM +0100, Ard Biesheuvel wrote: > There is no need to issue a full data synchronization barrier and an > instruction synchronization barrier after each and every set/way or > MVA cache maintenance operation. So remove them. So long as the loops calling

Re: [edk2] [PATCH] ArmPkg/ArmLib: remove pointless sync barriers after each cache op

On Mon, Nov 02, 2015 at 02:24:18PM +0100, Ard Biesheuvel wrote: > There is no need to issue a full data synchronization barrier and an > instruction synchronization barrier after each and every set/way or > MVA cache maintenance operation. So remove them. You don't need all of these, but it is

Re: [edk2] [PATCH 1/2] ShellPkg/UefiDpLib: Fix a DP cumulative data issue

Series Reviewed-by: Samer El-Haj-Mahmoud -Original Message- From: Shia, Cinnamon Sent: Monday, November 02, 2015 3:12 AM To: edk2-devel@lists.01.org Cc: El-Haj-Mahmoud, Samer ; Shia, Cinnamon Subject: [PATCH 1/2]

Re: [edk2] [PATCH] MdeModulePkg: Add a BDS platform hook function PlatformBootManagerBeforeDefaultBoot to PlatformBootManagerLib

Thanks Ray for the series. Sunny and myself will review -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ni, Ruiyu Sent: Monday, November 02, 2015 2:33 AM To: El-Haj-Mahmoud, Samer ; Wang, Sunny (HPS SW)

Re: [edk2] [Patch 2/2] UefiCpuPkg/CpuDxe: Fix one dead lock issue in ProcessorToIdleState()

On 11/02/15 04:00, Fan, Jeff wrote: > Laszlo, > > Your updating patch looks better. But when I tried it on my platform, > Sometime, it hang at syncing MTRRs from BSP to Aps. I am > investigating it and will feedback you later. Thanks. Actually, it has crossed my mind that *maybe* the recent KVM

Re: [edk2] SecurityPkg: PeiRsa2048Sha256GuidedSectionExtractLib error handling

> What kind of FV does the RSA2048Sha256 section exist in? BFV, Other FV must > be processed in PEI (containing required Peim or DxeCore) or FV can be > processed in DXE This is the compressed FV containing the DXE phase modules so we FVMAIN inside FVMAIN_COMPRESSED inside

Re: [edk2] [PATCH 1/2] ShellPkg/UefiDpLib: Fix a DP cumulative data issue

Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Monday, November 02, 2015 1:12 AM > To: edk2-devel@lists.01.org > Subject: [edk2] [PATCH 1/2] ShellPkg/UefiDpLib: Fix a

Re: [edk2] [PATCH 6/6] OvmfPkg/PlatformPei: Set PcdCpuMaxLogicalProcessorNumber using QEMU fw_cfg

On 10/31/15 18:50, Laszlo Ersek wrote: > I'm very sorry, but I don't think I can spend time on this, unless > someone gives me ssh and/or console access to a host that readily > reproduces the bug, with the latest kvm/master, qemu, and ekd2 > builds. We just got lucky, the problem reproduces on

Re: [edk2] [PATCH 2/2] ShellPkg/UefiDpLib: Support dumping cumulative data

Cinnamon, Can you send the UNI file as an attachment please? Jaben, Can you help commit the code after reviewing the UNI file? Thanks, --Samer -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Carsey, Jaben Sent: Monday, November 02, 2015

[edk2] [PATCH 2/5] SecurityPkg: Enable Customized Secure Boot feature

Enable Secure Boot feature defined in UEFI2.5 ECR1263. Add AuditMode/DeployedMode value definition. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang --- MdePkg/Include/Guid/GlobalVariable.h | 14 ++

[edk2] [PATCH 3/5] SecurityPkg: Enable Customized Secure Boot feature

Enable Secure Boot feature defined in UEFI2.5 ECR1263. Add gEfiSecureBootModeGuid definition Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang --- SecurityPkg/Include/Guid/AuthenticatedVariableFormat.h | 1 + SecurityPkg/SecurityPkg.dec

[edk2] [PATCH 1/5] SecurityPkg: Enable Customized Secure Boot feature

Enable Secure Boot feature defined in UEFI2.5 ECR1263. Add VarCheck for AuditMode/DeployedMode Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang --- .../VarCheckUefiLib/VarCheckUefiLibNullClass.c | 22 ++ 1 file

[edk2] [PATCH 4/5] SecurityPkg: Enable Customized Secure Boot feature

Enable Secure Boot feature defined in UEFI2.5 ECR1263. Add AuthVariable implementation logic. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang --- SecurityPkg/Library/AuthVariableLib/AuthService.c | 1221 +---

[edk2] [PATCH 5/5] SecurityPkg: Enable Customized Secure Boot feature

Enable Secure Boot feature defined in UEFI2.5 ECR1263. Add SecureBoot UI update. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang --- .../SecureBootConfigDxe/SecureBootConfig.vfr | 77 +++-

Re: [edk2] [PATCH 6/6] OvmfPkg/PlatformPei: Set PcdCpuMaxLogicalProcessorNumber using QEMU fw_cfg

On 2015-11-02 17:14:12, Laszlo Ersek wrote: > On 11/02/15 23:49, Laszlo Ersek wrote: > > On 11/02/15 23:29, Jordan Justen wrote: > >> On 2015-11-02 12:16:12, Laszlo Ersek wrote: > >>> On 11/02/15 19:53, Alex Williamson wrote: > On Mon, 2015-11-02 at 19:31 +0100, Laszlo Ersek wrote: > > On

Re: [edk2] [PATCH] UefiCpuPkg: PiSmmCpuDxeSmm: Remove Framework compatibility

It's good to me! Reviewed-by: Jeff Fan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Michael Kinney Sent: Saturday, October 31, 2015 2:03 AM To: edk2-devel@lists.01.org Subject: [edk2] [PATCH] UefiCpuPkg: PiSmmCpuDxeSmm:

Re: [edk2] [patch] NetworkPkg:Enable Http Boot over Ipv6 stack

Yes, you are right, the comments should be "HTTPClient" in both ip4 and ip6 stack. Thanks -Original Message- From: Gary Ching-Pang Lin [mailto:g...@suse.com] Sent: Tuesday, November 03, 2015 3:37 PM To: Zhang, Lubo Cc: Ye, Ting; edk2-devel@lists.01.org; Wu, Jiaxin; Fu, Siyuan Subject:

Re: [edk2] [PATCH] UefiCpuPkg/CpuDxe: fix ExitBootServices() callback in the presence of SMIs

On 2015-10-28 01:59:09, Fan, Jeff wrote: > Jordan, > > Sending NMI to wake up AP had been discussed later year. It prevents > platform to provide own real NMI handler. > > For current implementation, after finished AP task, AP will check if > it is in Idle state or not. If yes, place itself into

Re: [edk2] [PATCH v2] ArmPlatformPkg/ArmJunoPkg: Add the correct SPI interrupt numbers for the MSI range

Hi Jeremy, On 2 November 2015 at 15:44, Jeremy Linton wrote: > The JunoR1 has a GICv2m which is a GICv2 with a little piece of hardware > that has some memory mapped locations that can trigger traditional SPI > interrupts. This allows some basic PCIe MSI capabilities. > >

Re: [edk2] [PATCH v2] ArmPlatformPkg/ArmJunoPkg: Add the correct SPI interrupt numbers for the MSI range

On 11/02/2015 02:11 PM, Ryan Harkin wrote: On 2 November 2015 at 15:44, Jeremy Linton wrote: -EFI_ACPI_6_0_GIC_MSI_FRAME_INIT(0, ARM_JUNO_GIV2M_MSI_BASE, 0, 0, 0) +EFI_ACPI_6_0_GIC_MSI_FRAME_INIT(0, ARM_JUNO_GIV2M_MSI_BASE, 0, 351-224, 224) I was talking with

Re: [edk2] [PATCH] ArmVirtPkg: fix barriers in ArmEnableMmu

On 2 November 2015 at 18:14, Mark Rutland wrote: > The ARM architecture requires a DSB to complete TLB maintenance, with a > subsequent ISB being required to synchronize subsequent items in the > current instruction stream against the completed TLB maintenance. > > The

[edk2] [Patch] CryptoPkg: Add one new API (Pkcs7GetCertificatesList) for certs retrieving.

Adding one new API (Pkcs7GetCertificatesList) to retrieve and sort all embedded certificates from Pkcs7 signedData. This new API will provide the support for UEFI 2.5 Secure-Boot AuditMode feature. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Qin Long

Re: [edk2] [Patch] CryptoPkg: Add one new API (Pkcs7GetCertificatesList) for certs retrieving.

Reviewed-by: Chao Zhang Thanks & Best regards Chao Zhang -Original Message- From: Long, Qin Sent: Tuesday, November 03, 2015 2:38 PM To: Ye, Ting; Zhang, Chao B Cc: edk2-devel@lists.01.org Subject: [Patch] CryptoPkg: Add one new API

[edk2] [Patch] BaseTools: Allow decimal values in the EDK II meta-data file.

Because the EDK II meta-data specifications already allow using decimal values in the EDK II Meta-data file [Defines] section, this patch update code to allow this usage. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu ---

Re: [edk2] [Patch] BaseTools: Allow decimal values in the EDK II meta-data file.

On 11/2/15 10:00 PM, Yonghong Zhu wrote: Because the EDK II meta-data specifications already allow using decimal values in the EDK II Meta-data file [Defines] section, this patch update code to allow this usage. By the way, "metadata" is a single word so no hyphen is needed. -- Bruce

Re: [edk2] [PATCH] MdeModulePkg: Add a BDS platform hook function PlatformBootManagerBeforeDefaultBoot to PlatformBootManagerLib

Sunny, Can you use the existing PlatformBootManagerAfterConsole() hook? Thanks, Ray -Original Message- From: El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hpe.com] Sent: Tuesday, October 27, 2015 11:16 PM To: Wang, Sunny (HPS SW) ; edk2-devel@lists.01.org

Re: [edk2] [PATCH 2/2] ShellPkg/UefiDpLib: Support dumping cumulative data

Code looks good. I can't tell must about your UNI file change. Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Monday, November 02, 2015 1:12 AM > To:

[edk2] [PATCH 2/2] ShellPkg/UefiDpLib: Support dumping cumulative data

Add a new option -c to dump cumulative data. For example: shell> dp -c ==[ Cumulative ] (Times in microsec.) Cumulative Average ShortestLongest Name CountDurationDurationDurationDuration LoadImage: 200 1007000 0

[edk2] [PATCH 1/2] ShellPkg/UefiDpLib: Fix a DP cumulative data issue

The value of PERF_CUM_DATA.Count and PERF_CUM_DATA.Duration field keep cumulating on every execution of dp. Initialize the CumData at dp's entry point. Signed-off-by: Cinnamon Shia --- ShellPkg/Library/UefiDpLib/Dp.c | 27 ++- 1 file changed, 26

Re: [edk2] RE uses the version information in application .EFI file: edk2-devel Digest, Vol 4, Issue 214

Carl: OK. Your patch only updates Optional64, and miss Optional32. It is by purpose? Thanks Liming -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Miller, Carl H Sent: Thursday, October 29, 2015 9:59 PM To: edk2-devel@lists.01.org Subject:

Re: [edk2] [PATCH 6/6] OvmfPkg/PlatformPei: Set PcdCpuMaxLogicalProcessorNumber using QEMU fw_cfg

On 11/02/15 23:29, Jordan Justen wrote: > On 2015-11-02 12:16:12, Laszlo Ersek wrote: >> On 11/02/15 19:53, Alex Williamson wrote: >>> On Mon, 2015-11-02 at 19:31 +0100, Laszlo Ersek wrote: On 10/31/15 18:50, Laszlo Ersek wrote: > I'm very sorry, but I don't think I can spend time on

[edk2] [Patch] MdePkg: Add more DataBits support to Port80 output

The BasePostCodeLibPort80 instance just prints UINT8 to IoPort 80. Some boards may support 16bit or 32bit. To support them, new PCD PcdPort80DataWidth is introduced to specify the width of data bits to Port80. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao

[edk2] [Patch 03/11] MdeModulePkg: Use BmCharToUint in BmIsKeyOptionVariable

The patch also moves the BmCharToUint to BmMisc.c because it belongs to misc functions. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Library/UefiBootManagerLib/BmHotkey.c | 10 -

[edk2] [Patch 11/11] MdeModulePkg: Enable PlatformRecovery in BdsDxe driver

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Universal/BdsDxe/Bds.h | 3 - MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 405 --- 2 files changed, 95

[edk2] [Patch 07/11] MdeModulePkg: Add Platform recovery support

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Include/Library/UefiBootManagerLib.h | 1 + .../Library/UefiBootManagerLib/BmLoadOption.c | 130 ++---

[edk2] [Patch 09/11] MdeModulePkg: Use UefiSpec.h defined macro to replace L"xxx" string

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git

[edk2] [Patch 02/11] MdeModulePkg: Add Bm prefix for internal functions

Change VARIABLE_VISITOR to BM_VARIABLE_VISITOR Change ForEachVariable to BmForEachVariable Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Library/UefiBootManagerLib/BmLoadOption.c | 2 +-

[edk2] [Patch 01/11] MdePkg: Add Platform Recovery definitions.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Eric Dong --- MdePkg/Include/Uefi/UefiSpec.h | 1 + 1 file changed, 1 insertion(+) diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h index

[edk2] [Patch 04/11] MdeModulePkg: Use BM_OPTION_NAME_LEN instead of sizeof L"Boot####"

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Library/UefiBootManagerLib/BmHotkey.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[edk2] [Patch 06/11] MdeModulePkg: Support to expand File device path

To support platform recovery, File device path expanding capability is added. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 76

[edk2] [Patch 00/11] Add Platform Recovery support

OS Recovery will be added later. Ruiyu Ni (11): MdePkg: Add Platform Recovery definitions. MdeModulePkg: Add Bm prefix for internal functions MdeModulePkg: Use BmCharToUint in BmIsKeyOptionVariable MdeModulePkg: Use BM_OPTION_NAME_LEN instead of sizeof L"Boot" MdeModulePkg: Use

[edk2] [Patch 10/11] MdeModulePkg: Add PlatformRecovery#### pointing to default file path

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Eric Dong --- MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 38 +--- 1 file changed, 30 insertions(+), 8 deletions(-) diff --git

Re: [edk2] [PATCH] MdeModulePkg: Fix memory leak issues

Hi Ray, Good catch! Thanks. :) I updated patch for this. Please help to review it again. Regards, Sunny Wang -Original Message- From: Ni, Ruiyu [mailto:ruiyu...@intel.com] Sent: Saturday, October 31, 2015 6:59 AM To: Wang, Sunny (HPS SW) Cc: Tian, Feng;

[edk2] invalid_parameter when setting variable from application

Hi, I tried to set a 'EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS' variable from an UEFI application but SetVariable just returns invalid parameter for some reason.(with the global GUID) I'm setting the variable in the exact same way as other existing

[edk2] [PATCH v2] MdeModulePkg: Fix memory leak issues

Fix memory leak issues Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Sunny Wang --- MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c

Re: [edk2] [PATCH v2] MdeModulePkg: Fix memory leak issues

Reviewed-by: Ruiyu Ni -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Sunny Wang Sent: Monday, November 2, 2015 6:53 PM To: edk2-devel@lists.01.org Cc: el...@hpe.com Subject: [edk2] [PATCH v2] MdeModulePkg: Fix memory

Re: [edk2] [PATCH] MdeModulePkg: PiSmmCore: Remove confusing CopyMem() of SMM_ENTRY_CONTEXT

Good to me. Reviewed-by: Feng Tian -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Michael Kinney Sent: Saturday, October 31, 2015 08:18 To: edk2-devel@lists.01.org Subject: [edk2] [PATCH] MdeModulePkg: PiSmmCore: Remove

Re: [edk2] [PATCH 6/6] OvmfPkg/PlatformPei: Set PcdCpuMaxLogicalProcessorNumber using QEMU fw_cfg

On 11/02/15 23:49, Laszlo Ersek wrote: > On 11/02/15 23:29, Jordan Justen wrote: >> On 2015-11-02 12:16:12, Laszlo Ersek wrote: >>> On 11/02/15 19:53, Alex Williamson wrote: On Mon, 2015-11-02 at 19:31 +0100, Laszlo Ersek wrote: > On 10/31/15 18:50, Laszlo Ersek wrote: > >> I'm

Re: [edk2] PCI: "rejected due to resource confliction"

Laszlo's understanding is correct. The device was rejected because the pci host bridge driver cannot allocate such big resource for the whole PCI device tree. 1/0/0 claims the biggest resource so it was rejected. The PciIo protocol/PCI handle isn't created for that device. Shell doesn't depend

Re: [edk2] [PATCH] MdeModulePkg: PiSmmCore: Remove confusing CopyMem() of SMM_ENTRY_CONTEXT

one comment below On 10/31/15 01:17, Michael Kinney wrote: > A subset of fields in the EFI_SMM_SYSTEM_TABLE2 structure are identical > to the fields in the SMM_ENTRY_CONTEXT structure. CopyMem() is used to > transfer the contents of the SMM_ENTRY_CONTEXT structure into the >

[edk2] [patch 2/2] MdeModulePkg/ScsiDisk: Enlarge the value of SCSI_DISK_TIMEOUT to 30s

As ScsiDisk and ScsiBus driver are used to manage SCSI or ATAPI devices, the timeout value is updated to 30s to follow ATA/ATAPI spec in which the device may take up to 30s to respond command. The change is used to solve device compatibility issue found with a TEAC DV-W28S-WZ3 slim DVD plus a

[edk2] [patch 0/2] Fix a DVD device compatbility issue

This issue was raised with a TEAC DV-W28S-WZ3 slim DVD plus a SONY AccuCORE DVD-R media in which the DVD spends 8s to response READ_CAPACITY cmd after resetting the host machine. Tian Feng (2): MdeModulePkg/Ide: return correct status when DRQ is not ready for ATAPI MdeModulePkg/ScsiDisk:

[edk2] [patch 1/2] MdeModulePkg/Ide: return correct status when DRQ is not ready for ATAPI

When executing ATAPI cmd at IDE mode, EFI_SUCCESS may be returned wrongly with old logic but in fact DRQ is not ready and the transaction doesn't get executed correctly at this time. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Feng Tian Cc: Star

Re: [edk2] [PATCH] ArmPkg/ArmLib: remove pointless sync barriers after each cache op

On 2 November 2015 at 14:43, Leif Lindholm wrote: > On Mon, Nov 02, 2015 at 02:24:18PM +0100, Ard Biesheuvel wrote: >> There is no need to issue a full data synchronization barrier and an >> instruction synchronization barrier after each and every set/way or >> MVA cache

Re: [edk2] SecurityPkg: PeiRsa2048Sha256GuidedSectionExtractLib error handling

Eugene: Thanks for the info. You are right. The temp solution logic in PeiRsa2048Sha256GuidedSectionExtractLib is no longer needed since EFI_PEI_SECURITY2_PPI is supported by Core. I will provide patch to remove them. For platform code, you need to make sure FVMAIN_COMPACT is processed

Re: [edk2] [PATCH] ArmPkg/ArmLib: remove pointless sync barriers after each cache op

On 2 November 2015 at 14:51, Ard Biesheuvel wrote: > On 2 November 2015 at 14:43, Leif Lindholm wrote: >> On Mon, Nov 02, 2015 at 02:24:18PM +0100, Ard Biesheuvel wrote: >>> There is no need to issue a full data synchronization barrier and an

[edk2] [PATCH] ShellPkg UefiDpLib: Use correct string length for the input UnicodeBuffer

Same as the beginning of function to use DP_GAUGE_STRING_LENGTH, but not DXE_PERFORMANCE_STRING_LENGTH. Cc: Jaben Carsey Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- ShellPkg/Library/UefiDpLib/DpUtilities.c |

[edk2] [PATCH] SecurityPkg: Remove temp return solution in PeiRsa2048Sha256 Section Lib

PeiCore supports EFI_PEI_SECURITY_PPI to handle section extraction failure. The wrong returning status is no longer needed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: "Zhang, Chao B" --- .../PeiRsa2048Sha256GuidedSectionExtractLib.c

Re: [edk2] [PATCH] SecurityPkg: Remove temp return solution in PeiRsa2048Sha256 Section Lib

Reviewed-by: Liming Gao -Original Message- From: Zhang, Chao B Sent: Tuesday, November 03, 2015 11:03 AM To: edk2-devel@lists.01.org Cc: eug...@hp.com; Gao, Liming Subject: [PATCH] SecurityPkg: Remove temp return solution in PeiRsa2048Sha256 Section Lib PeiCore