[edk2] [PATCH] UefiCpuPkg/MpInitLib: Avoid calling PEI services from AP

2018-07-01 Thread Ruiyu Ni
Today's MpInitLib PEI implementation directly calls PeiServices->GetHobList() from AP which may cause racing issue. This patch fixes this issue by duplicating IDT for APs. Because CpuMpData structure is stored just after IDT, the CpuMPData address equals to IDTR.BASE + IDTR.LIMIT + 1. v2: 1. Ad

Re: [edk2] [PATCH v3] UefiCpuPkg/MpInitLib: Avoid calling PEI services from AP

2018-07-01 Thread Ni, Ruiyu
On 6/30/2018 3:25 AM, Laszlo Ersek wrote: (1) I think that, right after the above line, you are missing: BufferSize = ALIGN_VALUE (BufferSize, 8); Yes. That's a bug in the patch. I will fix it in V4. It is not noticed in practice because AllocatePages (EFI_SIZE_TO_PAGES (BufferSize))

Re: [edk2] [Patch v2] NetworkPkg/HttpDxe: Fix the bug when parsing HTTP(S) message body.

2018-07-01 Thread Wu, Jiaxin
Hi Gary, Thanks the verification, in my part, HTTPS works well. Can you send me the failure wireshark packet/full debug message? (correct the typo) Thanks, Jiaxin > > > -Original Message- > > From: Gary Lin [mailto:g...@suse.com] > > Sent: Monday, July 2, 2018 12:17 PM > > To: Wu, Jia

[edk2] [patch] MdeModulePkg/SmmCorePerformanceLib: Update mPerformanceLength

2018-07-01 Thread Dandan Bi
In mSmmBootPerformanceTable there are two parts, one is the FPDT table header and the other is FPDT records. Currently: mPerformanceLength: The length of existing FPDT records. mMaxPerformanceLength: The maximum length allocated for mSmmBootPerformanceTable(including FPDT table header length and ex

Re: [edk2] [Patch v2] NetworkPkg/HttpDxe: Fix the bug when parsing HTTP(S) message body.

2018-07-01 Thread Wu, Jiaxin
Hi Gary, Thanks the verification, in my part, can't HTTPS works well. Can you send me the failure wireshark packet/debug message to me? Thanks, Jiaxin > -Original Message- > From: Gary Lin [mailto:g...@suse.com] > Sent: Monday, July 2, 2018 12:17 PM > To: Wu, Jiaxin > Cc: edk2-devel@

Re: [edk2] [Patch v2] NetworkPkg/HttpDxe: Fix the bug when parsing HTTP(S) message body.

2018-07-01 Thread Gary Lin
On Mon, Jul 02, 2018 at 10:19:30AM +0800, Jiaxin Wu wrote: > *v2: Resolve the conflict commit. > > HttpBodyParserCallback function is to parse the HTTP(S) message body so as to > confirm whether there is the next message header. But it doesn't record the > parsing message data/length correctly. >

[edk2] [PATCH 1/3] MdeModulePkg/ResetUtilityLib: Add pack(1) for reset data structure

2018-07-01 Thread Ruiyu Ni
RESET_UTILITY_GUID_SPECIFIC_RESET_DATA structure should be declared as pack(1). The patch adds the missing pack(1) pragma. (GUID *)((UINT8 *)&ResetData + OFFSET_OF (RESET_UTILITY_GUID_SPECIFIC_RESET_DATA, ResetSubtype)) is used to replace &ResetData.ResetSubType to resolve C4366 VS compiler warnin

[edk2] [PATCH 2/3] MdeModulePkg/[Pei|Dxe]ResetSystemLib: Add PPI/Protocol dependency

2018-07-01 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni Cc: Star Zeng --- MdeModulePkg/Library/DxeResetSystemLib/DxeResetSystemLib.inf | 3 +++ MdeModulePkg/Library/PeiResetSystemLib/PeiResetSystemLib.inf | 2 ++ 2 files changed, 5 insertions(+) diff --git a/MdeModulePkg

[edk2] [PATCH 0/3] Bug fix and minor enhancements to ResetSystem code

2018-07-01 Thread Ruiyu Ni
Ruiyu Ni (3): MdeModulePkg/ResetUtilityLib: Add pack(1) for reset data structure MdeModulePkg/[Pei|Dxe]ResetSystemLib: Add PPI/Protocol dependency MdeModulePkg/ResetSystemRuntimeDxe: Add more info in debug message MdeModulePkg/Library/DxeResetSystemLib/DxeResetSystemLib.inf | 3 +++ MdeModu

[edk2] [PATCH 3/3] MdeModulePkg/ResetSystemRuntimeDxe: Add more info in debug message

2018-07-01 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ruiyu Ni Cc: Star Zeng --- MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/ResetSystemRuntimeDxe/ResetSystem.c b/MdeMo

Re: [edk2] [Patch] BaseTool: Add cache for the result of SkipAutogen.

2018-07-01 Thread Feng, Bob C
Hi Liming, Based on my testing, a module will enter the CanSkip() multiple times even after CanSkip() return True. In this patch, I cache the module after CanSkip() return True to save the cost of later CanSkip() evaluation. I think the CanSkip() will always return True after it return True at

Re: [edk2] [Patch] BaseTool: Add cache for the result of SkipAutogen.

2018-07-01 Thread Gao, Liming
Bob: When will this cache take effective? Thanks Liming >-Original Message- >From: Feng, Bob C >Sent: Monday, July 02, 2018 9:16 AM >To: edk2-devel@lists.01.org >Cc: Feng, Bob C ; Gao, Liming >Subject: [Patch] BaseTool: Add cache for the result of SkipAutogen. > >Add a cache for the va

[edk2] [Patch v2] NetworkPkg/HttpDxe: Fix the bug when parsing HTTP(S) message body.

2018-07-01 Thread Jiaxin Wu
*v2: Resolve the conflict commit. HttpBodyParserCallback function is to parse the HTTP(S) message body so as to confirm whether there is the next message header. But it doesn't record the parsing message data/length correctly. This patch is refine the parsing logic so as to fix the potential fail

[edk2] [Patch] BaseTool: Add cache for the result of SkipAutogen.

2018-07-01 Thread BobCF
Add a cache for the value of skip ModuleAutoGen process flag. This cache can improve build performance. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng Cc: Liming Gao --- BaseTools/Source/Python/AutoGen/AutoGen.py | 4 BaseTools/Source/Python/Common/Global