Re: [edk2-devel] [Patch 08/12] SecurityPkg: Update YAML to ignore specific ECC files/errors

2021-11-29 Thread Qi Zhang
Reviewed-by: Qi Zhang -Original Message- From: Kinney, Michael D Sent: Wednesday, November 24, 2021 5:21 AM To: devel@edk2.groups.io Cc: Zhang, Qi1 ; Kumar, Rahul1 ; Sean Brogan ; Bret Barkelew ; Liming Gao ; Michael Kubacki Subject: [Patch 08/12] SecurityPkg: Update YAML to ignore

Re: [edk2-devel] [Patch 09/12] ShellPkg: Update YAML to ignore specific ECC files/errors

2021-11-29 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Kinney, Michael D > Sent: Wednesday, November 24, 2021 5:21 AM > To: devel@edk2.groups.io > Cc: Ni, Ray ; Gao, Zhichao ; Sean > Brogan ; Bret > Barkelew ; Liming Gao > ; Michael Kubacki > > Subject: [Patch 09/12] ShellPkg: Update YAML

Re: [edk2-devel] [Patch 05/12] EmulatorPkg: Update YAML to ignore specific ECC files/errors

2021-11-29 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Kinney, Michael D > Sent: Wednesday, November 24, 2021 5:21 AM > To: devel@edk2.groups.io > Cc: Andrew Fish ; Ni, Ray ; Sean Brogan > ; Bret Barkelew > ; Liming Gao ; Michael > Kubacki > > Subject: [Patch 05/12] EmulatorPkg: Update

[edk2-devel] [PATCH 1/1] UefiPayloadPkg: Update the Module Info rather than create new Info.

2021-11-29 Thread Guomin Jiang
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3754 1. DxeCore will use ModuleInfo to install LoadedImage protocol for DxeCore. 2. DxeIpl will create the ModuleInfo of UniversalPayload. and UniversalPayload will create the ModuleInfo of DxeCore. 3. Only the first ModuleInfo will be used

[edk2-devel] [RFC] Git tag after extended hard freeze

2021-11-29 Thread Michael Kubacki
The edk2-stable202111 tag was created on commit bb1bba3 on Nov 26th 2021 as described in https://github.com/tianocore/edk2/releases/tag/edk2-stable202111. During the ongoing extended hard freeze, several changes related to code formatting and CI plugins are being applied as described in

[edk2-devel] Event: TianoCore Bug Triage - APAC / NAMO - 11/30/2021 #cal-reminder

2021-11-29 Thread devel@edk2.groups.io Calendar
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:PUBLISH REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20201011T015911Z

[edk2-devel] [PATCH] BaseTools/VfrCompile: Correct Bit Field Flags for numeric/one of

2021-11-29 Thread Long1 Huang
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3752 Add Bit mask to numeric/one of opcode to set correctly Flags for Bit Field. VfrSyntax.g: Set "LFlags &= EDKII_IFR_DISPLAY_BIT" before "LFlags |= (EDKII_IFR_NUMERIC_SIZE_BIT & (_GET_CURRQEST_VARSIZE()));" VfrFormPkg.h: update "if (LFlags &

[edk2-devel] [PATCH] BaseTools/VfrCompile: Correct Bit Field Flags for numeric/one of

2021-11-29 Thread Long1 Huang
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3752 Add Bit mask to numeric/one of opcode to set correctly Flags for Bit Field. VfrSyntax.g: Set "LFlags &= EDKII_IFR_DISPLAY_BIT" before "LFlags |= (EDKII_IFR_NUMERIC_SIZE_BIT & (_GET_CURRQEST_VARSIZE()));" VfrFormPkg.h: update "if (LFlags &

[edk2-devel] [PATCH EDK2 v1 1/1] MdeModulePkg/DxeCorePerformanceLib:Variable Initial

2021-11-29 Thread wenyi,xie via groups.io
SmmBootRecordDataSize is initialized in InternalGetSmmPerData, but this function may fail. so to avoid using SmmBootRecordDataSize without intialization, set it to 0 at first. Cc: Jian J Wang Cc: Liming Gao Cc: Dandan Bi Signed-off-by: Wenyi Xie ---

[edk2-devel] [PATCH EDK2 v1 0/1] MdeModulePkg/DxeCorePerformanceLib:Variable Initial

2021-11-29 Thread wenyi,xie via groups.io
Main Changes : 1.Initiallize SmmBootRecordDataSize to 0. Wenyi Xie (1): MdeModulePkg/DxeCorePerformanceLib:Variable Initial MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c | 2 ++ 1 file changed, 2 insertions(+) -- 2.20.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io

[edk2-devel] [PATCH v1 2/2] ArmPkg: MmCommunicationDxe: Update MM communicate input arguments checks

2021-11-29 Thread Kun Qin
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3751 Current MM communicate routine from ArmPkg would conduct few steps before proceeding with SMC calls. However, some inspection steps are different from PI specification. This patch updated MM communicate input argument inspection routine to

[edk2-devel] [PATCH v1 0/2] MM communicate functionality in variable policy

2021-11-29 Thread Kun Qin
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3709 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3751 Currently, setups with variable policy operations used together with MM communicate from ArmPkg could fail with `EFI_INVALID_PARAMETER`. This was due to the errors from 2 following

[edk2-devel] [PATCH v1 1/2] MdeModulePkg: VariableSmmRuntimeDxe: Fix Variable Policy Message Length

2021-11-29 Thread Kun Qin
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3709 In EDKII implementation of variable policy, the DXE runtime agent would communicate to MM to disable, register or query policies. However, these operations populate the value of MessageLength that includes communicate header to include MM

Re: [edk2-devel] [PATCH 5/5] [edk2-platforms] Platform/DeveloperBox: Expand NvStorage sizes

2021-11-29 Thread Masami Hiramatsu
Hi Leif, 2021年11月29日(月) 22:43 Leif Lindholm : > > On Sat, Nov 27, 2021 at 16:48:45 +0900, Masami Hiramatsu wrote: > > > On Fri, Nov 05, 2021 at 18:24:01 +0900, Masami Hiramatsu wrote: > > > > Expand NvStorage Variable size and FTW spare/working size > > > > for the DeveloperBox platform. > > > >

Re: [edk2-devel] [PATCH] OvmfPkg/MemEncryptSevLib: check CPUID when read msr during PEI phase

2021-11-29 Thread Brijesh Singh via groups.io
On 11/29/21 1:04 PM, Tom Lendacky wrote: On 11/25/21 7:12 AM, qi zhou wrote:  From 5b10265fa5c7b5ca728b4f18488089de6535ed28 Mon Sep 17 00:00:00 2001 From: Qi Zhou Date: Thu, 25 Nov 2021 20:25:55 +0800 Subject: [PATCH] OvmfPkg/MemEncryptSevLib: check CPUID when read msr during   PEI phase

Re: [edk2-devel] [PATCH v13 00/32] Add AMD Secure Nested Paging (SEV-SNP) support

2021-11-29 Thread Brijesh Singh via groups.io
Hi Edk2 Maintainers, The SNP support development finished almost two months ago. I have addressed all the feedback from the Ovmf maintainer, and the patches are ready to be merged. We are waiting for UefiCpuPkg maintainers to ack the patches as it touches a few areas in the EDK2 core. As

Re: [edk2-devel] [PATCH] OvmfPkg/MemEncryptSevLib: check CPUID when read msr during PEI phase

2021-11-29 Thread Lendacky, Thomas via groups.io
On 11/25/21 7:12 AM, qi zhou wrote: From 5b10265fa5c7b5ca728b4f18488089de6535ed28 Mon Sep 17 00:00:00 2001 From: Qi Zhou Date: Thu, 25 Nov 2021 20:25:55 +0800 Subject: [PATCH] OvmfPkg/MemEncryptSevLib: check CPUID when read msr during PEI phase Tested on Intel Platform, It is like 'SEV-ES

[edk2-devel] Uncrustify Conversion Detailed Plan and Extended Hard Freeze Update #3

2021-11-29 Thread Michael D Kinney
Hello, Changes from Update #2 * Changed order of PRs swapping (4) and (5). The PR that activates increases the max CI agent job time is independent of all the other PRs and its review is complete, so it can be

Re: [edk2-devel] [PATCH] UefiPayloadPkg: Increase SystemMemoryUefiRegionSize from 32M to 64M

2021-11-29 Thread Guo Dong
Reviewed-by: Guo Dong -Original Message- From: Liu, Zhiguang Sent: Sunday, November 28, 2021 10:30 PM To: devel@edk2.groups.io Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice ; You, Benjamin Subject: [PATCH] UefiPayloadPkg: Increase SystemMemoryUefiRegionSize from 32M to 64M Current, the

Re: [edk2-devel] [PATCH 2/2] UefiPayloadPkg: Add performance measurement feature

2021-11-29 Thread Guo Dong
Reviewed-by: Guo Dong -Original Message- From: Liu, Zhiguang Sent: Wednesday, November 24, 2021 2:15 AM To: devel@edk2.groups.io Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice ; You, Benjamin Subject: [PATCH 2/2] UefiPayloadPkg: Add performance measurement feature Add a Macro to enable

Re: [edk2-devel] [PATCH 1/2] UefiPayloadPkg: Add missing Guid gUefiAcpiBoardInfoGuid

2021-11-29 Thread Guo Dong
Reviewed-by: Guo Dong -Original Message- From: devel@edk2.groups.io On Behalf Of Zhiguang Liu Sent: Wednesday, November 24, 2021 2:15 AM To: devel@edk2.groups.io Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice ; You, Benjamin Subject: [edk2-devel] [PATCH 1/2] UefiPayloadPkg: Add missing Guid

Re: [edk2-devel] [Patch 11/12] UefiPayloadPkg: Update YAML to ignore specific ECC files/errors

2021-11-29 Thread Guo Dong
It looks good to me. Reviewed-by: Guo Dong -Original Message- From: Kinney, Michael D Sent: Tuesday, November 23, 2021 2:21 PM To: devel@edk2.groups.io Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice ; You, Benjamin ; Sean Brogan ; Bret Barkelew ; Liming Gao ; Michael Kubacki Subject:

Re: [edk2-devel] [PATCH 5/5] [edk2-platforms] Platform/DeveloperBox: Expand NvStorage sizes

2021-11-29 Thread Leif Lindholm
On Sat, Nov 27, 2021 at 16:48:45 +0900, Masami Hiramatsu wrote: > > On Fri, Nov 05, 2021 at 18:24:01 +0900, Masami Hiramatsu wrote: > > > Expand NvStorage Variable size and FTW spare/working size > > > for the DeveloperBox platform. > > > > > > Since the size of the NvStorage VariableSize is not

Re: [edk2-devel] one possible issue with ovmf fvb

2021-11-29 Thread Gerd Hoffmann
Hi, > On real X64 hardware you can’t be in 64-bit mode without pageables, > and you can’t put pageables in the ROM. Thus PEI ends up being IA32 > (i386) and this usually means the DXE Core gets loaded < 4 GiB, and > the initial memory map for DXE is memory < 4 GiB. Well, to exact there are two