Re: [edk2-devel] [PATCH] MdeModulePkg/SMM: Support to unregister SMI handler inside SMI handler

2024-03-06 Thread Zhiguang Liu
Hi Laszlo, We talked about how to unregister SMI handler inside other SMI handler. And the conclusion was there is no such usage so we don't need support it. However, I find some platform does need to unregister SMI handler inside other SMI handler. The design you introduced below is great to

Re: [edk2-devel] [edk2-redfish-client][PATCH 2/2] RedfishClientPkg: Remove unused header files.

2024-03-06 Thread Chang, Abner via groups.io
[AMD Official Use Only - General] Reviewed-by: Abner Chang > -Original Message- > From: devel@edk2.groups.io On Behalf Of Nickle > Wang via groups.io > Sent: Wednesday, March 6, 2024 10:11 PM > To: devel@edk2.groups.io > Cc: Chang, Abner ; Igor Kulchytskyy > > Subject: [edk2-devel]

Re: [edk2-devel] Topics for March TianoCore Community Meeting

2024-03-06 Thread Michael D Kinney
Meeting canceled this month. No agenda topics received. Mike > -Original Message- > From: devel@edk2.groups.io On Behalf Of Michael > D Kinney > Sent: Tuesday, March 5, 2024 11:08 PM > To: devel@edk2.groups.io > Cc: Kinney, Michael D > Subject: [edk2-devel] Topics for March TianoCore

[edk2-devel] Cancelled Event: TianoCore Community Meeting - APAC/NAMO - Thursday, March 7, 2024 #cal-cancelled

2024-03-06 Thread Group Notification
BEGIN:VCALENDAR VERSION:2.0 PRODID:-//Groups.io Inc//Groups.io Calendar//EN METHOD:CANCEL REFRESH-INTERVAL;VALUE=DURATION:PT1H X-PUBLISHED-TTL:PT1H CALSCALE:GREGORIAN BEGIN:VTIMEZONE TZID:America/Los_Angeles LAST-MODIFIED:20231222T233358Z

Re: [edk2-devel] [edk2-redfish-client][PATCH v3 2/2] RedfishClientPkg/Edk2RedfishResourceConfigLib: support config2 protocol

2024-03-06 Thread Chang, Abner via groups.io
[AMD Official Use Only - General] Reviewed-by: Abner Chang > -Original Message- > From: Nickle Wang > Sent: Thursday, March 7, 2024 11:24 AM > To: devel@edk2.groups.io > Cc: Chang, Abner ; Igor Kulchytskyy > ; Nick Ramirez > Subject: [edk2-redfish-client][PATCH v3 2/2] >

Re: [edk2-devel] [edk2-redfish-client][PATCH v3 1/2] RedfishClientPkg: introduce Redfish resource config2 protocol.

2024-03-06 Thread Chang, Abner via groups.io
[AMD Official Use Only - General] Reviewed-by: Abner Chang > -Original Message- > From: Nickle Wang > Sent: Thursday, March 7, 2024 11:24 AM > To: devel@edk2.groups.io > Cc: Chang, Abner ; Igor Kulchytskyy > ; Nick Ramirez > Subject: [edk2-redfish-client][PATCH v3 1/2]

[edk2-devel] Event: TianoCore Community Meeting - APAC/NAMO - Thursday, March 7, 2024 #cal-reminder

2024-03-06 Thread Group Notification
*Reminder: TianoCore Community Meeting - APAC/NAMO* *When:* Thursday, March 7, 2024 7:30pm to 8:30pm (UTC-08:00) America/Los Angeles *Where:*

[edk2-devel] [edk2-redfish-client][PATCH v3 2/2] RedfishClientPkg/Edk2RedfishResourceConfigLib: support config2 protocol

2024-03-06 Thread Nickle Wang via groups.io
-Support Redfish resource config2 protocol in Edk2RedfishResourceConfigLib. This library will try Redfish resource config2 protocol first. And if there is no matching protocol found, Redfish resource config protocol will be used. -Because EdkIIRedfishResourceConfigLib is updated to support Redfish

[edk2-devel] [edk2-redfish-client][PATCH v3 1/2] RedfishClientPkg: introduce Redfish resource config2 protocol.

2024-03-06 Thread Nickle Wang via groups.io
Introduce EDKII Redfish Resource Config2 Protocol. Comparing to Redfish Resource Config Protocol, this protocol adds new parameter called "JsonText". This allows Redfish caller driver to pass JSON context to callee. Callee save the effort of getting same data from Redfish service again. This is

[edk2-devel] [edk2-redfish-client][PATCH v3 0/2] introduce Redfish Resource Config2 protocol.

2024-03-06 Thread Nickle Wang via groups.io
v3: merge 2/3 and 3/3 in v2 together so we don't break the build. v2: address review comment about the description of "JsonText" parameter. Introduce EDKII Redfish Resource Config2 Protocol so that caller can pass Redfish data to callee when caller uses "expand" query parameter to get multiple

Re: [edk2-devel] GuestPhysAddrSize questions

2024-03-06 Thread Paolo Bonzini
On Mon, Mar 4, 2024 at 6:24 PM Tom Lendacky wrote: > > On 3/4/24 07:09, Gerd Hoffmann wrote: > >Hi, > > > >>> 23:16 GuestPhysAddrSize Maximum guest physical address size in bits. > >>> This number applies only to guests using > >>> nested > >>>

Re: [edk2-devel] [PATCH v2 07/10] EmulatorPkg: auto-generate SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: devel@edk2.groups.io On Behalf Of Laszlo > Ersek > Sent: Wednesday, March 6, 2024 1:10 PM > To: edk2-devel-groups-io > Cc: Andrew Fish ; Ni, Ray > Subject: Re: [edk2-devel] [PATCH v2 07/10] EmulatorPkg: auto-generate > SEC

Re: [edk2-devel] [edk2-platforms PATCH v2 2/4] SimicsOpenBoardPkg: auto-gen & fix SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: devel@edk2.groups.io On Behalf Of Laszlo > Ersek > Sent: Wednesday, March 6, 2024 1:10 PM > To: edk2-devel-groups-io > Cc: Desimone, Nathaniel L ; Chaganty, > Rangasai V > Subject: Re: [edk2-devel] [edk2-platforms PATCH v2

Re: [edk2-devel] [PATCH] Maintainers.txt: remove Laszlo's entries

2024-03-06 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: devel@edk2.groups.io On Behalf Of Laszlo > Ersek > Sent: Wednesday, March 6, 2024 1:06 PM > To: edk2-devel-groups-io > Cc: Andrew Fish ; Ard Biesheuvel > ; Gerd Hoffmann ; Yao, > Jiewen ; Leif Lindholm > ; Kinney, Michael D > ;

Re: [edk2-devel] [PATCH v2 01/10] pip-requirements.txt: require edk2-basetools version 0.1.51

2024-03-06 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: Laszlo Ersek > Sent: Wednesday, March 6, 2024 1:09 PM > To: edk2-devel-groups-io > Cc: Feng, Bob C ; Joey Vagedes > ; Liming Gao ; > Kinney, Michael D ; Rebecca Cran > ; Sean Brogan ; Chen, > Christine > Subject: Re:

Re: [edk2-devel] [PATCH v2 01/10] pip-requirements.txt: require edk2-basetools version 0.1.51

2024-03-06 Thread Rebecca Cran
Reviewed-by: Rebecca Cran On 3/5/2024 4:38 AM, Laszlo Ersek wrote: The edk2-basetools commit that corresponds to edk2 commit bac9c74080cf ("BaseTools/AutoGen: declare ProcessLibraryConstructorList() for SEC modules", 2024-02-29) is 5b7161de22ee ("BaseTools/AutoGen: declare

Re: [edk2-devel] [edk2-platforms PATCH v2 2/4] SimicsOpenBoardPkg: auto-gen & fix SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Laszlo Ersek
Can I please get a quick R-b for this patch -- it's urgent because of . Thank you, Laszlo On 3/5/24 13:01, Laszlo Ersek wrote: > declares a bogus > ProcessLibraryConstructorList() for SimicsOpenBoardPkg's SEC module. Rely > on AutoGen for

Re: [edk2-devel] [PATCH v2 10/10] UefiPayloadPkg: auto-generate SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Laszlo Ersek
Can I please get a quick R-b for this patch -- it's urgent because of . Thank you, Laszlo On 3/5/24 12:38, Laszlo Ersek wrote: > Rely on AutoGen for declaring ProcessLibraryConstructorList(). > > Build-tested with: > > python

Re: [edk2-devel] [PATCH v2 09/10] UefiCpuPkg: auto-generate SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Laszlo Ersek
Can I please get a quick R-b for this patch -- it's urgent because of . Thank you, Laszlo On 3/5/24 12:38, Laszlo Ersek wrote: > Rely on AutoGen for declaring ProcessLibraryConstructorList(). > > Build-tested with: > > build -a X64 -b DEBUG -m

Re: [edk2-devel] [PATCH v2 08/10] IntelFsp2Pkg: auto-generate SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Laszlo Ersek
Can I please get a quick R-b for this patch -- it's urgent because of . Thank you, Laszlo On 3/5/24 12:38, Laszlo Ersek wrote: > Rely on AutoGen for declaring ProcessLibraryConstructorList(). > > Build-tested with: > > build -a X64 -b DEBUG -m

Re: [edk2-devel] [PATCH v2 07/10] EmulatorPkg: auto-generate SEC ProcessLibraryConstructorList() decl

2024-03-06 Thread Laszlo Ersek
Can I please get a quick R-b for this patch -- it's urgent because of . Thank you, Laszlo On 3/5/24 12:38, Laszlo Ersek wrote: > Rely on AutoGen for declaring ProcessLibraryConstructorList(). > > Build-tested with: > > build -a X64 -b DEBUG -m

Re: [edk2-devel] [PATCH v2 01/10] pip-requirements.txt: require edk2-basetools version 0.1.51

2024-03-06 Thread Laszlo Ersek
Can I please get a quick R-b for this patch -- it's urgent because of . Thank you, Laszlo On 3/5/24 12:38, Laszlo Ersek wrote: > The edk2-basetools commit that corresponds to edk2 commit bac9c74080cf > ("BaseTools/AutoGen: declare

[edk2-devel] [PATCH] Maintainers.txt: remove Laszlo's entries

2024-03-06 Thread Laszlo Ersek
Red Hat and I have mutually and amicably agreed to separate. Remove my entries from "Maintainers.txt". Cc: Andrew Fish Cc: Ard Biesheuvel Cc: Gerd Hoffmann Cc: Jiewen Yao Cc: Leif Lindholm Cc: Michael D Kinney Cc: Rahul Kumar Cc: Ray Ni Cc: Sami Mujawar Signed-off-by: Laszlo Ersek ---

Re: [edk2-devel][PATCH v2 0/3] Fix Runtime Granularity Issues

2024-03-06 Thread Oliver Smith-Denny
Hi Liming, Friendly ping again, can you please review this? It fixes a UEFI spec violation. Thanks, Oliver On 2/26/2024 8:46 AM, Oliver Smith-Denny wrote: Hi Liming, Now that the stable tag is finished, can you review this MdeModulePkg patch? Thanks, Oliver On 2/16/2024 5:27 PM, Oliver

[edk2-devel] Event: TianoCore Community Meeting EMEA/NAMO - Thursday, March 7, 2024 #cal-reminder

2024-03-06 Thread Group Notification
*Reminder: TianoCore Community Meeting EMEA/NAMO* *When:* Thursday, March 7, 2024 8:00am to 9:00am (UTC-08:00) America/Los Angeles *Where:* Microsoft Teams meeting Join on your computer or mobile app Click here to join the meeting Meeting ID: 226 323 011 029 Passcode: hMRCj6 Download Teams |

Re: [edk2-devel] [edk2-stable202311][PATCH] BaseTools: Python VfrCompiler implementation

2024-03-06 Thread Chang, Abner via groups.io
[AMD Official Use Only - General] Hi Christine, Is the VFR python tool upstream to edk2-staging? May I know where can I get it? I would like to try it and see the flexibility of output some VFR information in JSON/YAML file. For edk2 Redfish, we are looking for the solution to automatically

Re: [edk2-devel] [edk2-redfish-client][PATCH 1/3] RedfishClientPkg: introduce Redfish resource config2 protocol.

2024-03-06 Thread Nickle Wang via groups.io
> Can we say "The additional JSON data in ASCII string format passed to callee. > The > usage is defined between the Redfish feature driver and subordinate Redfish > resource feature driver."? Seems to me more clear about the usage. > Also apply this description to the following functions. >

[edk2-devel] [edk2-redfish-client][PATCH v2 3/3] RedfishClientPkg/Features: support config2 protocol

2024-03-06 Thread Nickle Wang via groups.io
Because EdkIIRedfishResourceConfigLib is updated to support Redfish resource config2 protocol, update corresponding functions in feature drivers. Signed-off-by: Nickle Wang Cc: Abner Chang Cc: Igor Kulchytskyy Cc: Nick Ramirez --- .../Features/Bios/v1_0_9/Common/BiosCommon.c | 14

[edk2-devel] [edk2-redfish-client][PATCH v2 2/3] RedfishClientPkg/Edk2RedfishResourceConfigLib: support config2 protocol

2024-03-06 Thread Nickle Wang via groups.io
Support Redfish resource config2 protocol in Edk2RedfishResourceConfigLib. This library will try Redfish resource config2 protocol first. And if there is no matching protocol found, Redfish resource config protocol will be used. Signed-off-by: Nickle Wang Cc: Abner Chang Cc: Igor Kulchytskyy

[edk2-devel] [edk2-redfish-client][PATCH v2 1/3] RedfishClientPkg: introduce Redfish resource config2 protocol.

2024-03-06 Thread Nickle Wang via groups.io
Introduce EDKII Redfish Resource Config2 Protocol. Comparing to Redfish Resource Config Protocol, this protocol adds new parameter called "JsonText". This allows Redfish caller driver to pass JSON context to callee. Callee save the effort of getting same data from Redfish service again. This is

[edk2-devel] [edk2-redfish-client][PATCH v2 0/3] introduce Redfish Resource Config2 protocol.

2024-03-06 Thread Nickle Wang via groups.io
v2: address review comment about the description of "JsonText" parameter. Introduce EDKII Redfish Resource Config2 Protocol so that caller can pass Redfish data to callee when caller uses "expand" query parameter to get multiple Redfish resources all together. Then, callee does not get Redfish

[edk2-devel] [edk2-redfish-client][PATCH 2/2] RedfishClientPkg: Remove unused header files.

2024-03-06 Thread Nickle Wang via groups.io
stdarg.h and stdio.h are not used in RedfishClientPkg. Signed-off-by: Nickle Wang Cc: Abner Chang Cc: Igor Kulchytskyy --- RedfishClientPkg/PrivateInclude/Crt/stdarg.h | 14 -- RedfishClientPkg/PrivateInclude/Crt/stdio.h | 14 -- 2 files changed, 28 deletions(-)

回复: [edk2-devel] [PATCH v4 2/3] MdeModulePkg: Update ReceiveData and SendData function description

2024-03-06 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Qingyu > 发送时间: 2024年2月26日 11:06 > 收件人: devel@edk2.groups.io > 抄送: Liming Gao ; Michael D Kinney > > 主题: [edk2-devel] [PATCH v4 2/3] MdeModulePkg: Update ReceiveData and > SendData function description > > AtaBusDxe,

回复: [edk2-devel] [PATCH v4 1/3] MdePkg: Update ReceiveData and SendData function description

2024-03-06 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Qingyu > 发送时间: 2024年2月26日 11:06 > 收件人: devel@edk2.groups.io > 抄送: Michael D Kinney ; Liming Gao > ; Zhiguang Liu > 主题: [edk2-devel] [PATCH v4 1/3] MdePkg: Update ReceiveData and > SendData function description > > Refer

[edk2-devel] 回复: [PATCH] SignedCapsulePkg: Update GetImage and GetImageInfo description details

2024-03-06 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Pethaiyan Madhan > 发送时间: 2024年2月27日 18:04 > 收件人: devel@edk2.groups.io > 抄送: Pethaiyan Madhan ; Michael D Kinney > ; Liming Gao ; > Zhiguang Liu ; Yi Li ; GuoX Xu > > 主题: [PATCH] SignedCapsulePkg: Update GetImage and GetImageInfo > description

[edk2-devel] 回复: [PATCH V4] FmpDevicePkg: GetImageInfo Add missing conditions

2024-03-06 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Pethaiyan Madhan > 发送时间: 2024年2月27日 17:46 > 收件人: devel@edk2.groups.io > 抄送: Pethaiyan Madhan ; Michael D Kinney > ; Liming Gao ; > Zhiguang Liu ; Yi Li ; GuoX Xu > > 主题: [PATCH V4] FmpDevicePkg: GetImageInfo Add missing conditions > > 1.For

[edk2-devel] 回复: [PATCH] MdePkg: Update GetImage , GetImageInfo description details

2024-03-06 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Pethaiyan Madhan > 发送时间: 2024年2月27日 17:59 > 收件人: devel@edk2.groups.io > 抄送: Pethaiyan Madhan ; Michael D Kinney > ; Liming Gao ; > Zhiguang Liu ; Yi Li ; GuoX Xu > > 主题: [PATCH] MdePkg: Update GetImage , GetImageInfo description details > >

[edk2-devel] Event: TianoCore edk2-test Bug Triage Meeting - Thursday, March 7, 2024 #cal-reminder

2024-03-06 Thread Group Notification
*Reminder: TianoCore edk2-test Bug Triage Meeting* *When:* Thursday, March 7, 2024 10:00pm to 11:00pm (UTC+08:00) Asia/Shanghai *Where:* https://armltd.zoom.us/j/94348061758?pwd=Q3RDeFA5K2JFaU5jdWUxc1FnaGdyUT09=addon *Organizer:* Edhaya Chandran edhaya.chand...@arm.com (

[edk2-devel] [PATCH v1 2/2] UefiCpuPkg/PiSmmCpuDxeSmm: Make RunningApCount on exclusive cacheline

2024-03-06 Thread Wu, Jiaxin
For non blocking mode during SmmMpBroadcastProcedure, multiple APs might contended access the RunningApCount in the PROCEDURE_TOKEN: Step1: RunningApCount is initialized to the mMaxNumberOfCpus (See GetFreeToken). Step2: Decrease RunningApCount if the AP is not present (See InterlockedDecrement

[edk2-devel] [PATCH v1 1/2] UefiCpuPkg/PiSmmCpuDxeSmm: Remove one round of Busy spinlock

2024-03-06 Thread Wu, Jiaxin
For both non blocking and blocking: Remove unnecessary a pair of AcquireSpinLock and ReleaseSpinLock for each AP. It's target to improve SmmStartupAllAPs performance. No functional impact, see below analysis: During first acquire spinLock, InternalSmmStartupAllAPs will return EFI_NOT_READY if

[edk2-devel] [PATCH v1 0/2] SMM MP service performance Improvement

2024-03-06 Thread Wu, Jiaxin
The series patch are to improve the SMM MP service performance. Cc: Ray Ni Cc: Laszlo Ersek Cc: Eric Dong Cc: Zeng Star Cc: Gerd Hoffmann Cc: Rahul Kumar Signed-off-by: Jiaxin Wu Jiaxin Wu (2): UefiCpuPkg/PiSmmCpuDxeSmm: Remove one round of Busy spinlock UefiCpuPkg/PiSmmCpuDxeSmm:

[edk2-devel] [PATCH edk2-platforms v6 7/7] Platform/SbsaQemu: add DeviceTree fallbacks to parse memory information

2024-03-06 Thread Marcin Juszkiewicz
From: Xiong Yining Add the DeviceTree fallbacks to parsing the information about memory if the related SMC calls Failed. Signed-off-by: Xiong Yining Signed-off-by: Chen Baozi --- .../SbsaQemuHardwareInfoLib.c | 106 +++- 1 file changed, 104

[edk2-devel] [PATCH edk2-platforms v6 6/7] Platform/SbsaQemu: get the information of memory via SMC calls

2024-03-06 Thread Marcin Juszkiewicz
From: Xiong Yining Provide functions to check for memory information: - amount of memory nodes - memory address - NUMA node id for memory Values are read from TF-A using platform specific SMC calls. Signed-off-by: Xiong Yining Signed-off-by: Chen Baozi ---

[edk2-devel] [PATCH edk2-platforms v6 4/7] Platform/SbsaQemu: move FdtHandlerLib to SbsaQemuHardwareInfoLib

2024-03-06 Thread Marcin Juszkiewicz
There is no need for EDK2 to know that there is DeviceTree around. All hardware information is read using functions from SbsaQemuHardwareInfoLib library. Library fallbacks to parsing DT if needed. Signed-off-by: Marcin Juszkiewicz Reviewed-by: Leif Lindholm ---

[edk2-devel] [PATCH edk2-platforms v6 5/7] Platform/SbsaQemu: hang if there is no cpu information

2024-03-06 Thread Marcin Juszkiewicz
In case we do not have cpu information (SMC call fails, our minimal DT lacks info) we shutdown system. Signed-off-by: Marcin Juszkiewicz --- .../Library/SbsaQemuHardwareInfoLib/SbsaQemuHardwareInfoLib.inf| 2 ++ .../SbsaQemu/Library/SbsaQemuHardwareInfoLib/SbsaQemuHardwareInfoLib.c | 3

[edk2-devel] [PATCH edk2-platforms v6 3/7] Platform/SbsaQemu: use PcdCoreCount directly

2024-03-06 Thread Marcin Juszkiewicz
During platform initialization we read amount of cpu cores and set PcdCoreCount so there is no need to call FdtHandler. Signed-off-by: Marcin Juszkiewicz --- Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf | 6 ++ Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.c

[edk2-devel] [PATCH edk2-platforms v6 2/7] Platform/SbsaQemu: read amount of cpus during init

2024-03-06 Thread Marcin Juszkiewicz
We read it once and store in Pcd for future use. Signed-off-by: Marcin Juszkiewicz --- .../SbsaQemu/Drivers/SbsaQemuPlatformDxe/SbsaQemuPlatformDxe.inf | 4 +++- .../SbsaQemu/Drivers/SbsaQemuPlatformDxe/SbsaQemuPlatformDxe.c | 9 + 2 files changed, 8 insertions(+), 5 deletions(-)

[edk2-devel] [PATCH edk2-platforms v6 0/7] get rid of DeviceTree from SbsaQemu

2024-03-06 Thread Marcin Juszkiewicz
We want to stop parsing DeviceTree to gather hardware information. Instead we ask TF-A for those details using SMC calls. On real hardware platform it could be asking on-board Embedded Controller. Hardware information (CPU, Memory) is now in SbsaQemuHardwareInfoLib together with new code for

[edk2-devel] [PATCH edk2-platforms v6 1/7] Platform/SbsaQemu: add SbsaQemuHardwareInfoLib

2024-03-06 Thread Marcin Juszkiewicz
This library provides functions to check for hardware information. For now it covers CPU ones: - amount of cpu cores - MPIDR value for cpu core - NUMA node id for cpu core Values are read from TF-A using platform specific SMC calls. Signed-off-by: Marcin Juszkiewicz ---