[edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib

2024-05-22 Thread Ning Feng
oid the issue. Cc: Ray Ni Cc: Jiaxin Wu Signed-off-by: Ning Feng --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 34 +++- 1 file changed, 23 insertions(+), 11 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index d724456

[edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib

2024-05-23 Thread Ning Feng
the issue. Cc: Ray Ni Signed-off-by: Ning Feng --- ...-CodeQL-Update-from-2.16.1-to-2.17.3.patch | 82 0001-Pkg-Module-UefiCpuPkg-MpLib.patch| 120 ++ UefiCpuPkg/Library/MpInitLib/MpLib.c | 34 +++-- 3 files changed, 226 insertions(+), 10 deleti

[edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib

2024-05-23 Thread Ning Feng
the issue. Cc: Ray Ni Signed-off-by: Ning Feng --- UefiCpuPkg/Library/MpInitLib/MpLib.c | 34 1 file changed, 24 insertions(+), 10 deletions(-) diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index d724456502..ae279c6ceb 100

[edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib

2024-05-23 Thread Ning Feng
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4778 MPInitlib have wrong expectation that BSP index should always be 0 in MpInitLibInitialize(), SwitchBsp(),ApWakeupFunction(). That will cause the data mismatch, if the initial BSP is not 0. Cc: Ray Ni Signed-off-by: Ning Feng --- UefiCpuPkg

Re: [edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib

2024-05-23 Thread Ning Feng
Hi, ray,i have already update the patch as per request. run the uncrustify from local, fixed the coding style issue. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119141): https://edk2.groups.io/g/devel/message/119141 Mute This Topic: h

[edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib:Do not assume BSP is #0.

2024-05-23 Thread Ning Feng
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4778 MPInitlib have wrong expectation that BSP index should always be 0 in MpInitLibInitialize(), SwitchBsp(),ApWakeupFunction(). That will cause the data mismatch, if the initial BSP is not 0. Cc: Ray Ni Signed-off-by: Ning Feng --- UefiCpuPkg

Re: [edk2-devel] [PATCH] Pkg-Module:UefiCpuPkg/MpLib

2024-05-23 Thread Ning Feng
hi, ray looks like after only changed the title, there created another groups, please check below link. thanks [PATCH] Pkg-Module:UefiCpuPkg/MpLib:Do not assume BSP is #0 (groups.io) ( https://edk2.groups.io/g/devel/message/119161 ) -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messa

[edk2-devel] [PATCH] UefiCpuPkg/MpLib:Do not assume BSP is #0.

2024-05-24 Thread Ning Feng
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4778 MPInitlib have wrong expectation that BSP index should always be 0 in MpInitLibInitialize(), SwitchBsp(),ApWakeupFunction(). That will cause the data mismatch, if the initial BSP is not 0. Cc: Ray Ni Signed-off-by: Ning Feng --- UefiCpuPkg

[edk2-devel] [UEFIPayLoad] While BootMenuApp build in UEFI payload, pcd in BootMenuApp will not be override. #include #includes

2021-11-17 Thread ning . feng
While BootMenuApp build in UEFI payload, pcd in BootMenuApp will not be override By Platform value. Have already check with liiu, zhiguang, need to fix at UefiPayloadPkg.dsc -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#83812): https://

[edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadPkg.dsc:Add BootManagerLib for BootManagerMenuApp

2021-11-17 Thread Ning Feng
Add PlatformBootManagerLibconstructor for BootManagerMenuApp, to get the value PcdBootManagefile overrided by platform side. Signed-off-by: Ning Feng Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You --- UefiPayloadPkg/UefiPayloadPkg.dsc | 6 -- 1 file changed, 4 insertions(+), 2

Re: [edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadPkg.dsc:Add BootManagerLib for BootManagerMenuApp

2021-11-18 Thread Ning Feng
We only need the constructor of this Lib to override the value patchable PcdBootManagefile. Null Lib will only link the constructor. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#83848): https://edk2.groups.io/g/devel/message/83848 Mute

[edk2-devel] [PATCH] UefiPayloadPkg/UefiPayloadPkg.dsc:Add BootManagerLib for BootManagerMenuApp

2021-11-18 Thread Ning Feng
Add PlatformBootManagerLibconstructor for BootManagerMenuApp, to get the value PcdBootManagefile overrided by platform side. Signed-off-by: Ning Feng Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You --- UefiPayloadPkg/UefiPayloadPkg.dsc | 6 -- 1 file changed, 4 insertions(+), 2

[edk2-devel] One question about memorymap

2021-12-01 Thread Ning Feng
If we build MMIO resurce Hob in PEI phase, should this MMIO resource range with runtime attribute? should  we see the range under Shell command "memmap"? and if we have Memorymap, E820 table will not use any more in UEFI mode? -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sen

[edk2-devel] [PATCH] UefiPayloadPkg/DebugPrintErrorLevelLibHob

2023-01-06 Thread Ning Feng
fix debug print error level hob not save correct debugprinterrorlevel should cover the case: Header.Lenth ==UNIVERSAL_PAYLOAD_SIZEOF_THROUGH_FIELD () Change-Id: Ibebe29fe8057e24448b31a8caa7ab6eb8054c39b BugZilla: https://bugzilla.tianocore.org/show_bug.cgi?id=4264 Signed-off-by: Ning Feng

Re: [edk2-devel] [PATCH] UefiPayloadPkg/DebugPrintErrorLevelLibHob

2023-01-08 Thread Ning Feng
fix debug print error level hob not save correct debugprinterrorlevel should cover the case: Header.Lenth ==UNIVERSAL_PAYLOAD_SIZEOF_THROUGH_FIELD () Change-Id: Ibebe29fe8057e24448b31a8caa7ab6eb8054c39b BugZilla: https://bugzilla.tianocore.org/show_bug.cgi?id=4264 Signed-off-by: Ning Feng

Re: [edk2-devel] [PATCH] UefiPayloadPkg/DebugPrintErrorLevelLibHob

2023-01-08 Thread Ning Feng
b not save correct DebugPrintErrorlevel Should cover the case: Header.Length == UNIVERSAL_PAYLOAD_SIZEOF_THROUGH_FIELD () Cc: Ray Ni mailto:ray...@intel.com>> Cc: Sean Rhodes mailto:sean@starlabs.systems>> Cc: James Lu mailto:james...@intel.com>> Cc: Gua Guo mailto:gua....@intel.com>&

Re: [edk2-devel] [PATCH] UefiPayloadPkg/DebugPrintErrorLevelLibHob

2023-01-09 Thread Ning Feng
AD_SIZEOF_THROUGH_FIELD () Cc: Ray Ni mailto:ray...@intel.com>> Cc: Sean Rhodes mailto:sean@starlabs.systems>> Cc: James Lu mailto:james...@intel.com>> Cc: Gua Guo mailto:gua@intel.com>> Signed-off-by: Ning Feng mailto:ning.f...@intel.com>> Reviewed-by: Gua Guo mail

[edk2-devel] [PATCH] MedModulePkg/DxeIplPeim: Fix pagetable protection region in 5 level paging

2024-11-11 Thread Ning Feng via groups.io
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4873 Currently the function does not cover the 5 level paging case. it will casued pagetable protection region set incorrectly. This patch do the enhancemant and with the patch protection region has been set correctly. Signed-off-by: Ning Feng

[edk2-devel] [PATCH] MedModulePkg/DxeIplPeim: Fix pagetable protection region in 5 level paging

2024-11-11 Thread Ning Feng via groups.io
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4873 Currently the function does not cover the 5 level paging case. it will casued pagetable protection region set incorrectly. This patch do the enhancemant and with the patch protection region has been set correctly. Signed-off-by: Ning Feng

[edk2-devel] [PATCH] MedModulePkg/DxeIplPeim: Fix pagetable protection region in 5 level paging

2024-11-11 Thread Ning Feng via groups.io
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4873 Currently the function does not cover the 5 level paging case. it will casued pagetable protection region set incorrectly. This patch do the enhancemant and with the patch protection region has been set correctly. Signed-off-by: Ning Feng

[edk2-devel] [PATCH] MedModulePkg/DxeIplPeim: Fix pagetable protection region in 5 level paging

2024-11-11 Thread Ning Feng via groups.io
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4873 Currently the function does not cover the 5 level paging case. it will casued pagetable protection region set incorrectly. This patch do the enhancemant and with the patch protection region has been set correctly. Signed-off-by: Ning Feng

[edk2-devel] [PATCH] MedModulePkg/DxeIplPeim: Fix pagetable protection region in 5 level paging

2024-11-11 Thread Ning Feng via groups.io
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4873 Currently the function does not cover the 5 level paging case. it will casued pagetable protection region set incorrectly. This patch do the enhancemant and with the patch protection region has been set correctly. Signed-off-by: Ning Feng