Re: [edk2-devel] [PATCH v2 1/1] OvmfPkg:Add variable store is full debug info

2023-09-07 Thread Zhenyu Zhang
> My suggestion is to wrap the newly added DEBUG statement with an if: > > if (OptionIndex == -1) { > Status = EfiBootManagerAddLoadOptionVariable (, MAX_UINTN); > +if (Status == EFI_OUT_OF_RESOURCES) { > + DEBUG ((DEBUG_ERROR, "ERROR: Variable store is full.\n")); > +} >

Re: [edk2-devel] [PATCH v2] MdeModulePkg/XhciDxe: Use Performance Timer for XHCI Timeouts

2023-09-07 Thread Wu, Hao A
Thanks. I am fine with the proposal below. Could you please help to follow the step 11 in page: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process and check the CI test results for the upcoming patch? Best Regards, Hao Wu > -Original Message- > From: Henz,

[edk2-devel] [PATCH v1 1/1] MdeModulePkg/BootMaintenanceManagerUiLib: Check array index before access

2023-09-07 Thread Michael Kubacki
From: Michael Kubacki Many arrays are defined with a length of MAX_MENU_NUMBER in FormGuid.h. Two of those are BootOptionOrder and DriverOptionOrder. In UpdatePage.c, a pointer is set to either of those arrays. The array buffer is accessed using an index whose range is checked after the pointer

Re: [edk2-devel] [edk2-platfoms][PATCH V1] WhitleyOpenBoardPkg :Updated PcdFlashFvSecuritySize for JunctionCity and Aowanda platform

2023-09-07 Thread Nate DeSimone
Pushed: https://github.com/tianocore/edk2-platforms/commit/bb6841e -Original Message- From: Ramkumar Krishnamoorthi Sent: Thursday, August 31, 2023 10:53 AM To: devel@edk2.groups.io Cc: DOPPALAPUDI, HARIKRISHNA ; Ponnusamy, Suresh ; KARPAGAVINAYAGAM, MANICKAVASAKAM ; Venkatesan,

Re: [edk2-devel] [edk2-platfoms][PATCH V1] WhitleyOpenBoardPkg :Updated PcdFlashFvSecuritySize for JunctionCity and Aowanda platform

2023-09-07 Thread Nate DeSimone
Reviewed-by: Nate DeSimone -Original Message- From: Ramkumar Krishnamoorthi Sent: Thursday, August 31, 2023 10:53 AM To: devel@edk2.groups.io Cc: DOPPALAPUDI, HARIKRISHNA ; Ponnusamy, Suresh ; KARPAGAVINAYAGAM, MANICKAVASAKAM ; Venkatesan, Selvaraj ; Ramkumar Krishnamoorthi ; Isaac

Re: [edk2-devel] [PATCH v5] MdeModulePkg/HiiDatabase: Fix incorrect AllocateCopyPool size

2023-09-07 Thread Dandan Bi
Reviewed-by: Dandan Bi Thanks, Dandan -Original Message- From: Mike Beaton Sent: Thursday, September 7, 2023 11:35 AM To: devel@edk2.groups.io Cc: Dong, Eric ; Bi, Dandan ; Ard Biesheuvel ; Mike Beaton Subject: [PATCH v5] MdeModulePkg/HiiDatabase: Fix incorrect AllocateCopyPool

Re: 回复: [edk2-devel] [PATCH v1 1/1] ShellPkg/UefiShellNetwork2CommandsLib: Check array index before access

2023-09-07 Thread Michael Kubacki
Hi Liming, I'm running the CodeQL CLI (https://docs.github.com/en/code-security/codeql-cli/getting-started-with-the-codeql-cli) locally against the code with some new queries. The queries in the codeql/cpp-queries pack listed here are relatively easy to experiment with

回复: [edk2-devel] [PATCH 0/2] Add New Intel Processor family

2023-09-07 Thread gaoliming via groups.io
Avinash: Seemly, this patch set doesn't include the change in ShellPkg. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Avinash > 发送时间: 2023年9月8日 3:00 > 收件人: devel@edk2.groups.io > 主题: [edk2-devel] [PATCH 0/2] Add New Intel Processor family > > Content-Type: text/plain;

回复: [edk2-devel] [PATCH v5] MdeModulePkg/HiiDatabase: Fix incorrect AllocateCopyPool size

2023-09-07 Thread gaoliming via groups.io
Mike: Thanks for your detail information in the commit message. This change is very clear. Reviewed-by: Liming Gao Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Mike Beaton > 发送时间: 2023年9月7日 11:35 > 收件人: devel@edk2.groups.io > 抄送: Eric Dong ; Dandan Bi ; > Ard Biesheuvel ;

回复: [edk2-devel] [PATCH v1 1/1] ShellPkg/UefiShellNetwork2CommandsLib: Check array index before access

2023-09-07 Thread gaoliming via groups.io
Michael: How do you detect those issues? Do you use the tool or do code review? For this change, Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Michael > Kubacki > 发送时间: 2023年9月7日 1:41 > 收件人: devel@edk2.groups.io > 抄送: Zhichao Gao ; Michael D Kinney > > 主题:

回复: [edk2-devel] [PATCH v1 1/1] MdePkg/Library/BaseRngLib: Fix include guard

2023-09-07 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Rebecca Cran > 发送时间: 2023年9月7日 3:47 > 收件人: devel@edk2.groups.io; mikub...@linux.microsoft.com > 抄送: Michael D Kinney ; Liming Gao > ; Zhiguang Liu ; > Rebecca Cran > 主题: Re: [edk2-devel] [PATCH v1 1/1] MdePkg/Library/BaseRngLib: Fix > include

回复: [edk2-devel] [PATCH v1 1/1] MdePkg/Library/TdxLib: Remove unnecessary comparison

2023-09-07 Thread gaoliming via groups.io
Reviewed-by: Liming Gao > -邮件原件- > 发件人: Yao, Jiewen > 发送时间: 2023年9月7日 7:32 > 收件人: devel@edk2.groups.io; mikub...@linux.microsoft.com > 抄送: Kinney, Michael D ; Gao, Liming > ; Liu, Zhiguang ; > Rebecca Cran > 主题: RE: [edk2-devel] [PATCH v1 1/1] MdePkg/Library/TdxLib: Remove >

Re: [edk2-devel] [PATCH v1 1/1] ArmVirtPkg: Enable Early Serial For DxeCore

2023-09-07 Thread Sean
On 9/7/2023 1:54 PM, Laszlo Ersek wrote: On 9/7/23 19:50, Sean Brogan wrote: I would argue that by declaring that your library class supports type DXE_CORE (or any core type) that you have declared you understand the uniqueness of the environment and have accounted for it. For instances that

Re: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

2023-09-07 Thread Michael D Kinney
I was asking about the property of the global variables being used in this patch. Are they already guaranteed to be in BSD format and in range 0..9. If so, then no additional code changes would be required. However, it would be good to add comments about the properties of those global

Re: [edk2-devel] [PATCH v1 1/1] ShellPkg/UefiShellNetwork2CommandsLib: Check array index before access

2023-09-07 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, September 6, 2023 10:41 AM > To: devel@edk2.groups.io > Cc: Gao, Zhichao ; Kinney, Michael D > > Subject: [PATCH v1 1/1] ShellPkg/UefiShellNetwork2CommandsLib: Check > array index

Re: [edk2-devel] [PATCH v1 1/1] MdePkg/Library/TdxLib: Remove unnecessary comparison

2023-09-07 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Wednesday, September 6, 2023 10:34 AM > To: devel@edk2.groups.io > Cc: Kinney, Michael D ; Gao, Liming > ; Liu, Zhiguang ; > Rebecca Cran > Subject: [PATCH v1 1/1] MdePkg/Library/TdxLib:

Re: [edk2-devel] [PATCH v1 1/1] MdePkg/Library/BaseRngLib: Fix include guard

2023-09-07 Thread Michael D Kinney
Reviewed-by: Michael D Kinney > -Original Message- > From: Rebecca Cran > Sent: Wednesday, September 6, 2023 12:47 PM > To: devel@edk2.groups.io; mikub...@linux.microsoft.com > Cc: Kinney, Michael D ; Gao, Liming > ; Liu, Zhiguang ; > Rebecca Cran > Subject: Re: [edk2-devel] [PATCH v1

Re: [edk2-devel] [PATCH v1 7/7] .pytool/Plugin: Add DebugMacroCheck

2023-09-07 Thread Michael Kubacki
There's not package level control at the moment from a given package CI YAML file. We've treated this as fundamental for "core" packages (those often built from a single CISettings.py file) and on-by-default but with a disable option via the env var for individual platform build files (like

Re: [edk2-discuss][edk2-devel] UEFI variables not getting stored in nvram disk

2023-09-07 Thread Laszlo Ersek
On 9/7/23 22:24, Het Gala wrote: > > > > Forwarded Message > Subject: [edk2-discuss] UEFI variables not getting stored in nvram disk > Date: Mon, 17 Jul 2023 23:24:45 +0530 > From: Het Gala > To: disc...@edk2.groups.io > CC: Kallol Biswas [C] > > >

Re: [edk2-devel] [PATCH v1 1/1] ArmVirtPkg: Enable Early Serial For DxeCore

2023-09-07 Thread Laszlo Ersek
On 9/7/23 19:50, Sean Brogan wrote: > I would argue that by declaring that your library class supports type > DXE_CORE (or any core type) that you have declared you understand the > uniqueness of the environment and have accounted for it. > > For instances that support DXE_CORE or MM_CORE module

Re: [edk2-devel] VT100 terminal for UEFI shell

2023-09-07 Thread Andrew Fish via groups.io
> On Sep 7, 2023, at 12:11 PM, Marcin Juszkiewicz > wrote: > > W dniu 7.09.2023 o 19:40, Andrew (EFI) Fish pisze: >>> On Sep 7, 2023, at 8:00 AM, Marcin Juszkiewicz wrote: >>> >>> Is there a way to have VT100 (or any other black/white, non-ANSI) terminal >>> for UEFI Shell? >>> >>> I do

Re: [edk2-devel] [edk2][RFC] OvmfPkg/AcpiPlatformDxe: patch FADT PSCI bits if FDT advertises it

2023-09-07 Thread Laszlo Ersek
On 9/7/23 17:17, Ard Biesheuvel wrote: > On Thu, 7 Sept 2023 at 16:51, Laszlo Ersek wrote: >> >> On 9/7/23 16:27, Ard Biesheuvel wrote: >>> On Mon, 16 Jan 2023 at 12:39, Evgeny Iakovlev >>> wrote: EL3 firmware may implement PSCI interface on aarch64 platforms, including

Re: [edk2-devel] VT100 terminal for UEFI shell

2023-09-07 Thread Marcin Juszkiewicz
W dniu 7.09.2023 o 19:40, Andrew (EFI) Fish pisze: On Sep 7, 2023, at 8:00 AM, Marcin Juszkiewicz wrote: Is there a way to have VT100 (or any other black/white, non-ANSI) terminal for UEFI Shell? I do many runs of QEMU/sbsa-ref with logging and all those ANSI colour codes only make

Re: [edk2-devel] [PATCH 1/1] OvmfPkg/RiscVVirt/README.md: bring your own OpenSBI

2023-09-07 Thread Andrei Warkentin
This looks fine to me as far as an edit, but I didn't validate the commands. Reviewed-by: Andrei Warkentin > -Original Message- > From: Laszlo Ersek > Sent: Thursday, September 7, 2023 9:58 AM > To: ler...@redhat.com; edk2-devel-groups-io > Cc: Warkentin, Andrei ; Ard Biesheuvel > ;

[edk2-devel] [PATCH 2/2] MdePkg/SmBios.h : Add New Intel Processor family

2023-09-07 Thread Avinash
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4547 Add New Intel Processor family for SMBIOS Type 4 Hex value - 16h Name - Intel® Processor Signed-off-by: avinashbhargava Cc: Zhiguang Liu Cc: Dandan Bi Cc: Star Zeng Cc: Zhichao Gao Cc: Benny Lin Cc: Gua Guo Cc: Prakashan Krishnadas

[edk2-devel] [PATCH v1 1/1] MdePkg/SmBios.h: Add New Intel Processor family for SMBIOS

2023-09-07 Thread Avinash
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4547 Add New Intel Processor family for SMBIOS Hex value - 16h Name - Intel® Processor Signed-off-by: avinashbhargava Cc: Zhiguang Liu Cc: Dandan Bi Cc: Star Zeng Cc: Zhichao Gao --- MdePkg/Include/IndustryStandard/SmBios.h | 1 + 1 file

[edk2-devel] [PATCH 0/2] Add New Intel Processor family

2023-09-07 Thread Avinash
Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit *** BLURB HERE *** Avinash (1): MdePkg/SmBios.h : Add New Intel Processor family Nhi Pham (1): ShellPkg/SmbiosView: Update display of PCIe system slot ID

Re: [edk2-devel] [PATCH v2] MdeModulePkg/XhciDxe: Use Performance Timer for XHCI Timeouts

2023-09-07 Thread Henz, Patrick
I don’t have strong opinions either. If we would prefer to cache the values returned by GetPerformanceCounterProperties I would be more than happy to do that. I could also modify XhcGetElapsedTime to return the elapsed ticks instead of the elapsed time in nano seconds, the functions that call

Re: [edk2-devel] [PATCH v1 1/1] ArmVirtPkg: Enable Early Serial For DxeCore

2023-09-07 Thread Sean
I would argue that by declaring that your library class supports type DXE_CORE (or any core type) that you have declared you understand the uniqueness of the environment and have accounted for it. For instances that support DXE_CORE or MM_CORE module types we often use a global variable (to

Re: [edk2-devel] VT100 terminal for UEFI shell

2023-09-07 Thread Andrew Fish via groups.io
> On Sep 7, 2023, at 8:00 AM, Marcin Juszkiewicz > wrote: > > Is there a way to have VT100 (or any other black/white, non-ANSI) terminal > for UEFI Shell? > > I do many runs of QEMU/sbsa-ref with logging and all those ANSI colour codes > only make problems. > These are the supported

Re: [edk2-devel] [edk2-platforms][PATCH 1/1] Maintainers.txt: Update maintainers for Ampere platforms

2023-09-07 Thread Leif Lindholm
On Thu, Sep 07, 2023 at 22:46:43 +0700, Nhi Pham via groups.io wrote: > This adds Rebecca Cran as reviewer, updates myself as maintainer, and > Leif as reviewer. Also, removes Vu Nguyen and Thang Nguyen due to no > longer working on EDK2. > > Signed-off-by: Nhi Pham Reviewed-by: Leif Lindholm

Re: [edk2-devel] [edk2-platforms][PATCH 1/1] Maintainers.txt: Update maintainers for Ampere platforms

2023-09-07 Thread Rebecca Cran via groups.io
On 9/7/2023 9:46 AM, Nhi Pham wrote: This adds Rebecca Cran as reviewer, updates myself as maintainer, and Leif as reviewer. Also, removes Vu Nguyen and Thang Nguyen due to no longer working on EDK2. Signed-off-by: Nhi Pham --- Maintainers.txt | 7 +++ 1 file changed, 3 insertions(+), 4

[edk2-devel] [edk2-platforms][PATCH 1/1] Maintainers.txt: Update maintainers for Ampere platforms

2023-09-07 Thread Nhi Pham via groups.io
This adds Rebecca Cran as reviewer, updates myself as maintainer, and Leif as reviewer. Also, removes Vu Nguyen and Thang Nguyen due to no longer working on EDK2. Signed-off-by: Nhi Pham --- Maintainers.txt | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git

Re: [edk2-devel] [PATCH v1 2/2] StandaloneMmPkg: Fix HOB space and heap space conflicted issue

2023-09-07 Thread Nhi Pham via groups.io
On 9/6/2023 11:22 PM, Oliver Smith-Denny wrote: On 9/6/2023 1:50 AM, Ard Biesheuvel wrote: On Wed, 6 Sept 2023 at 09:56, Nhi Pham wrote: On 9/6/2023 1:33 PM, Ni, Ray wrote: [EXTERNAL EMAIL NOTICE: This email originated from an external sender. Please be mindful of safe email handling and

Re: [edk2-devel] [PATCH v1 1/1] ArmVirtPkg: Enable Early Serial For DxeCore

2023-09-07 Thread Oliver Smith-Denny
On 9/7/2023 6:10 AM, Laszlo Ersek wrote: (replying on the webui... sorry!) The problem is actually embedded in MdePkg and MdeModulePkg. - In DxeMain() (and in functions called by DxeMain()), we call DebugLib APIs *before* reaching ProcessLibraryConstructorList(). - In ArmVirtQemu, we

Re: [edk2-devel] [edk2][RFC] OvmfPkg/AcpiPlatformDxe: patch FADT PSCI bits if FDT advertises it

2023-09-07 Thread Ard Biesheuvel
On Thu, 7 Sept 2023 at 16:51, Laszlo Ersek wrote: > > On 9/7/23 16:27, Ard Biesheuvel wrote: > > On Mon, 16 Jan 2023 at 12:39, Evgeny Iakovlev > > wrote: > >> > >> EL3 firmware may implement PSCI interface on aarch64 platforms, > >> including qemu-tcg-aarch64. However, EL3 firmware does not

Re: [edk2-devel] [PATCH 1/1] OvmfPkg/RiscVVirt/README.md: bring your own OpenSBI

2023-09-07 Thread Laszlo Ersek
On 9/7/23 16:58, Laszlo Ersek wrote: > Explain how users can compose their pre-OS environment purely from > binaries they've built themselves. > > Cc: Andrei Warkentin > Cc: Ard Biesheuvel > Cc: Gerd Hoffmann > Cc: Jiewen Yao > Cc: Jordan Justen > Cc: Sunil V L > Signed-off-by: Laszlo Ersek

Re: [edk2-devel] [Patch V2 1/5] MdeModulePkg: add MpService2Ppi field in SMM_S3_RESUME_STATE

2023-09-07 Thread Wang, Jian J
Reviewed-by: Jian J Wang Regards, Jian > -Original Message- > From: Tan, Dun > Sent: Wednesday, September 06, 2023 11:22 AM > To: devel@edk2.groups.io; Tan, Dun > Cc: Ni, Ray ; Wang, Jian J ; Gao, > Liming > Subject: RE: [edk2-devel] [Patch V2 1/5] MdeModulePkg: add MpService2Ppi >

[edk2-devel] VT100 terminal for UEFI shell

2023-09-07 Thread Marcin Juszkiewicz
Is there a way to have VT100 (or any other black/white, non-ANSI) terminal for UEFI Shell? I do many runs of QEMU/sbsa-ref with logging and all those ANSI colour codes only make problems. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online

[edk2-devel] [PATCH 1/1] OvmfPkg/RiscVVirt/README.md: bring your own OpenSBI

2023-09-07 Thread Laszlo Ersek
Explain how users can compose their pre-OS environment purely from binaries they've built themselves. Cc: Andrei Warkentin Cc: Ard Biesheuvel Cc: Gerd Hoffmann Cc: Jiewen Yao Cc: Jordan Justen Cc: Sunil V L Signed-off-by: Laszlo Ersek --- Notes: rendered version:

Re: [edk2-devel] [edk2][RFC] OvmfPkg/AcpiPlatformDxe: patch FADT PSCI bits if FDT advertises it

2023-09-07 Thread Laszlo Ersek
On 9/7/23 16:27, Ard Biesheuvel wrote: > On Mon, 16 Jan 2023 at 12:39, Evgeny Iakovlev > wrote: >> >> EL3 firmware may implement PSCI interface on aarch64 platforms, >> including qemu-tcg-aarch64. However, EL3 firmware does not usually own >> pulling and deploying ACPI tables from qemu fw_cfg.

Re: [edk2-devel] [edk2][RFC] OvmfPkg/AcpiPlatformDxe: patch FADT PSCI bits if FDT advertises it

2023-09-07 Thread Ard Biesheuvel
On Mon, 16 Jan 2023 at 12:39, Evgeny Iakovlev wrote: > > EL3 firmware may implement PSCI interface on aarch64 platforms, > including qemu-tcg-aarch64. However, EL3 firmware does not usually own > pulling and deploying ACPI tables from qemu fw_cfg. Thus the only way > EL3 can advertise PSCI on

Re: [edk2-devel] [PATCH] ArmPkg/ArmPsciMpServices Add EFI_NOT_READY return

2023-09-07 Thread Ard Biesheuvel
On Thu, 29 Jun 2023 at 22:47, Jeff Brasen via groups.io wrote: > > Add EFI_NOT_READY return if the CPU can not be enabled if the > processor is already on. > > This can occur in normal use if the CPU is still being turned off from > a previous call when this is called again. > > Signed-off-by:

[edk2-devel] Now: TianoCore edk2-test Bug Triage Meeting - Thursday, September 7, 2023 #cal-notice

2023-09-07 Thread Group Notification
*TianoCore edk2-test Bug Triage Meeting* *When:* Thursday, September 7, 2023 10:00pm to 11:00pm (UTC+08:00) Asia/Shanghai *Where:* https://armltd.zoom.us/j/91247522013?pwd=ei9nUndTbG9oWEROS2M1aVREZkpiQT09=addon *Organizer:* Edhaya Chandran edhaya.chand...@arm.com (

Re: [edk2-devel] [PATCH v3 1/1] ArmVirtPkg: use PcdTerminalTypeGuidBuffer for VirtioSerial console

2023-09-07 Thread Ard Biesheuvel
On Thu, 6 Jul 2023 at 14:30, Gerd Hoffmann wrote: > > Be consistent with pl011-based serial console setup. > > Suggested-by: Laszlo Ersek > Signed-off-by: Gerd Hoffmann Acked-by: Ard Biesheuvel I'll queue this one up too > --- > ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c | 9

Re: [edk2-devel] [PATCH 1/1] OvmfPkg/Include: remove QemuSmramSaveStateMap.h

2023-09-07 Thread Ard Biesheuvel
On Tue, 4 Jul 2023 at 11:52, Gerd Hoffmann wrote: > > The qemu/kvm SMM emulation uses the AMD SaveState layout. > > So, now that we have AMD SaveState support merged we can just use > Amd/SmramSaveStateMap.h, QemuSmramSaveStateMap.h is not needed any more. > > Signed-off-by: Gerd Hoffmann

Re: [edk2-devel] [PATCH 1/1] OvmfPkg/PlatformBootManagerLib: setup virtio-mmio devices.

2023-09-07 Thread Ard Biesheuvel
On Thu, 6 Jul 2023 at 17:31, Gerd Hoffmann wrote: > > Add DetectAndPreparePlatformVirtioDevicePath() helper function > to setup virtio-mmio devices. Start with virtio-serial support. > > This makes virtio console usable with microvm. > > Signed-off-by: Gerd Hoffmann Acked-by: Ard Biesheuvel

Re: [edk2-devel] [PATCH v2 1/1] OvmfPkg/README: Document Secure Boot

2023-09-07 Thread Ard Biesheuvel
On Tue, 4 Jul 2023 at 09:28, Gerd Hoffmann wrote: > > On Fri, Jun 30, 2023 at 02:26:03AM +0400, Joursoir wrote: > > Add the new section for Secure Boot. > > > > Signed-off-by: Alexander Goncharov > > Acked-by: Gerd Hoffmann > Thanks all Queued as #4800 -=-=-=-=-=-=-=-=-=-=-=- Groups.io

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

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

Re: [edk2-devel] [PATCH v2 1/1] OvmfPkg:Add variable store is full debug info

2023-09-07 Thread Oliver Steffen
Quoting Yao, Jiewen (2023-09-07 14:26:31) > I don't think using ASSERT is a good idea here. > Some context: We observed that EDK2 hits an ASSERT (Out of Resources) when booting with a full variable store. The message provided in this case is not helpful for non-experts. I think the goal here is

Re: [edk2-devel] [PATCH v1 1/1] ArmVirtPkg: Enable Early Serial For DxeCore

2023-09-07 Thread Laszlo Ersek
(replying on the webui... sorry!) The problem is actually embedded in MdePkg and MdeModulePkg. - In DxeMain() (and in functions called by DxeMain()), we call DebugLib APIs *before* reaching ProcessLibraryConstructorList(). - In ArmVirtQemu, we resolve the DXE Core's DebugLib dependency to

[edk2-devel] [PATCH v3 7/8] Sophgo/SG2042Pkg: Add SG2042Pkg.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> Add SG2042Pkg for Sophgo SG2042 platform. Provides PCD tokens. Signed-off-by: caiyuqing379 <202235...@mail.sdu.edu.cn> Co-authored-by: USER0FISH Cc: dahogn Cc: meng-cz Cc: yli147 Cc: ChaiEvan Cc: Sunil V L Cc: Leif Lindholm ---

[edk2-devel] [PATCH v3 1/8] Sophgo/SG2042Pkg: Add SmbiosPlatformDxe module.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> This driver installs SMBIOS information for SG2042. Install hardware information by creating an SMBIOS table which includes BIOS version, system manufacturer, product name, processor, memory, slots, storage, and other. Signed-off-by: caiyuqing379

[edk2-devel] [PATCH v3 4/8] Sophgo/SG2042Pkg: Add base MMC driver.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> This driver implements the MMC Host protocol, which is used by SD interface driver that the Sophgo SG2042 EVB supports. Add this driver in Sophgo/SG2042Pkg leveraging the one form Embedded Package. Signed-off-by: caiyuqing379

Re: [edk2-devel] [PATCH v2 0/8] EDK2 on RISC-V Sophgo SG2042 platform

2023-09-07 Thread caiyuqing_hz
Hi, Leif Thanks for your comments. We have conducted some testing and completed the third version of the patch. 1) Blurb We have added some descriptions in Blurb to explain the current status, testing situation, and limitations of the project. 2) Milk-V Pioneer board Our team does not have a

[edk2-devel] [PATCH v3 6/8] Sophgo/SG2042_EVB_Board: Add Sophgo SG2042 platform.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> Add infrastructure files to build edk2-platforms for Sophgo SG2042 platform. It follows PEI less design. Signed-off-by: caiyuqing379 <202235...@mail.sdu.edu.cn> Co-authored-by: USER0FISH Cc: dahogn Cc: meng-cz Cc: yli147 Cc: ChaiEvan Cc: Sunil

[edk2-devel] [PATCH v3 3/8] Sophgo/SG2042Pkg: Add Sophgo SDHCI driver.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> This driver implements Sophgo SDHCI controller, which provides the necessary interfaces for handling communication and data transfer with SD cards. Signed-off-by: caiyuqing379 <202235...@mail.sdu.edu.cn> Co-authored-by: USER0FISH Cc: dahogn Cc:

[edk2-devel] [PATCH v3 5/8] Sophgo/SG2042Pkg: Add SEC module.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> This module supports Sophgo SG2042 EVB platform. It uses the PEI less design. Add this module in SG2042Pkg leveraging the one from OvmfPkg/RiscVVirt. Signed-off-by: caiyuqing379 <202235...@mail.sdu.edu.cn> Co-authored-by: USER0FISH Cc: dahogn Cc:

[edk2-devel] [PATCH v3 0/8] EDK2 on RISC-V Sophgo SG2042 platform

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> Description: Deploy EDK2 to run on 64-core CPU under RISC-V architecture, and successfully boot to OS. Implementation can be found on https://github.com/AII-SDU/edk2-platforms/tree/devel-Sophgo/SG2042Pkg/Platform/Sophgo. Current progress and

[edk2-devel] [PATCH v3 2/8] Sophgo/SG2042Pkg: Add PlatformUpdateMmuDxe module.

2023-09-07 Thread caiyuqing_hz
From: caiyuqing379 <202235...@mail.sdu.edu.cn> SG2042 (Xuantie C920) MMU can be enabled in SV39 mode. C920 has five customizable page properties that control whether the page is Strong order(SO), Cacheable(C), Bufferable(B), Shareable(SH), Trustable(Sec). This driver modifies the page table

Re: [edk2-devel] [PATCH v2 1/1] OvmfPkg:Add variable store is full debug info

2023-09-07 Thread Yao, Jiewen
I don't think using ASSERT is a good idea here. Why not return ERROR? > -Original Message- > From: devel@edk2.groups.io On Behalf Of Zhenyu > Zhang > Sent: Thursday, September 7, 2023 7:11 PM > To: devel@edk2.groups.io > Cc: zheny...@redhat.com; ostef...@redhat.com; kra...@redhat.com;

[edk2-devel] [PATCH v2 1/1] OvmfPkg:Add variable store is full debug info

2023-09-07 Thread Zhenyu Zhang
From: "Zhenyu Zhang" When the variable store is full, edk2 will directly assert. Add debug information to help users understand what caused the assertion. Actual results: RecordVarErrorFlag (0xEF) 9A144FE2A47E:937FE521-95AE-4D1A-8929- 48BCD90AD31A - 0x0003 - 0x7E CommonVariableSpace =

Re: [edk2-devel] [PATCH v5] MdeModulePkg/HiiDatabase: Fix incorrect AllocateCopyPool size

2023-09-07 Thread Ard Biesheuvel
On Thu, 7 Sept 2023 at 05:35, Mike Beaton wrote: > > The immediately preceding call, GetBestLanguage, plus the implementation of > HiiGetString, which is called immediately afterwards, make it clear that > BestLanguage is a null-terminated ASCII string, and not just a five byte, > non-null

Re: [edk2-devel][PATCH v1 1/1] ArmVirtPkg: Enable Early Serial For DxeCore

2023-09-07 Thread Ard Biesheuvel
On Wed, 6 Sept 2023 at 23:49, Oliver Smith-Denny wrote: > > Currently, ArmVirtPkg does not provide a serial library for DxeCore, > so any early prints are missed. These prints are extremely valuable > for debugging. > > The early serial port lib used by PeiCore and PEIMs is also > applicable to

Re: [edk2-devel] [PATCH v1 1/1] OvmfPkg/Bhyve: don't exit early if RSDP is not found in memory

2023-09-07 Thread Ard Biesheuvel
On Thu, 7 Sept 2023 at 10:34, Corvin Köhne wrote: > > If OVMF fails to find the RSDP in memory, it should fall back installing > the statically provided ACPI tables. > > Signed-off-by: Corvin Köhne > Cc: Ard Biesheuvel > Cc: Jiewen Yao > Cc: Jordan Justen > Cc: Gerd Hoffmann > Cc: Rebecca

Re: [edk2-devel] [PATCH] MdeModulePkg/SmbiosDxe: Fix BcdRevision is not match with SMBIOS version

2023-09-07 Thread Lien, HoraceX
Hi Mike, Could you please reply for me? If you want to filter range 0-9, then I will send PR again. Thanks, Horace Lien -Original Message- From: devel@edk2.groups.io On Behalf Of Lien, HoraceX Sent: Friday, September 1, 2023 3:06 PM To: Kinney, Michael D ; devel@edk2.groups.io Cc: Liu,

[edk2-devel] [PATCH v2] Pyrite support - Secure erase is only available if encryption is supported.

2023-09-07 Thread Linus Liu
From: Linus Liu https://bugzilla.tianocore.org/show_bug.cgi?id=3004 Cc: Qi Zhang Cc: Rahul Kumar Cc: Jiewen Yao Cc: Tina Chen Cc: Xiao X Chen --- SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git

[edk2-devel] [PATCH v1 0/1] OvmfPkg/Bhyve: properly fall back to static ACPI tables

2023-09-07 Thread Corvin Köhne
CI: https://github.com/tianocore/edk2/pull/4799 Corvin Köhne (1): OvmfPkg/Bhyve: don't exit early if RSDP is not found in memory OvmfPkg/Bhyve/AcpiPlatformDxe/AcpiPlatform.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) -- 2.42.0 -=-=-=-=-=-=-=-=-=-=-=-

[edk2-devel] [PATCH v1 1/1] OvmfPkg/Bhyve: don't exit early if RSDP is not found in memory

2023-09-07 Thread Corvin Köhne
If OVMF fails to find the RSDP in memory, it should fall back installing the statically provided ACPI tables. Signed-off-by: Corvin Köhne Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Cc: Rebecca Cran Cc: Peter Grehan ---

Re: [edk2-devel] [PATCH v1] Pyrite support - Secure erase is only available if encryption is supported.

2023-09-07 Thread Yao, Jiewen
Thanks. 1) I think we need an else branch for " SupportedAttributes->MediaEncryption ", to assign " AvalDiskActions->SecureErase = 0;" > -AvalDiskActions->SecureErase = 1; > > +if (SupportedAttributes->MediaEncryption) { > > + AvalDiskActions->SecureErase = 1; > > +} > >

Re: [edk2-devel] [PATCH v2] MdeModulePkg/XhciDxe: Use Performance Timer for XHCI Timeouts

2023-09-07 Thread Wu, Hao A
Speaking of PcAtChipsetPkg/Library/AcpiTimerLib, the implementation of below instances: * PeiAcpiTimerLib.inf * DxeAcpiTimerLib.inf * StandaloneMmAcpiTimerLib.inf will not calculate the TSC frequency upon every call of GetPerformanceCounterProperties(). Best Regards, Hao Wu > -Original

Re: [edk2-devel] [PATCH v2] MdeModulePkg/XhciDxe: Use Performance Timer for XHCI Timeouts

2023-09-07 Thread Wu, Hao A
I do not have strong opinion on this considering it is an IO driver. The call of GetTimeInNanoSecond() is also likely to invoke GetPerformanceCounterProperties() call. I will let the patch owner to decide on the open raised below. Best Regards, Hao Wu > -Original Message- > From: Mike

Re: [edk2-devel] [PATCH v2] MdeModulePkg/XhciDxe: Use Performance Timer for XHCI Timeouts

2023-09-07 Thread Mike Maslenkin
Hello Hao Wu, Isn't GetPerformanceCounterProperties (, ) call too "heavy" for XHCI paths? May be it's better to cache timer direction once? I'm asking in a context of PcAtChipsetPkg/Library/AcpiTimerLib instance used by a number of Intel's platform in edk-platforms. For this library

[edk2-devel] [PATCH] MdeModulePkg: Support customized FV Migration Information

2023-09-07 Thread Wang Fan
From: Wang Fan REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4533 There are use cases which not all FVs need be migrated from TempRam to permanent memory before TempRam tears down. This new guid is introduced to avoid unnecessary FV migration to improve boot performance. Platform can

Re: [edk2-devel] [PATCH V9 0/2] Support RSA4096 and RSA3072

2023-09-07 Thread Yao, Jiewen
Reviewed-by: Jiewen Yao Merged https://github.com/tianocore/edk2/pull/4798 > -Original Message- > From: Sheng, W > Sent: Thursday, September 7, 2023 9:57 AM > To: devel@edk2.groups.io > Cc: Yao, Jiewen ; Wang, Jian J ; > Xu, Min M ; Chen, Zeyi ; Wang, > Fiona ; Lu, Xiaoyu1 ; Jiang, >