Re: [edk2-devel] [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level

2022-01-10 Thread Huang, Jenny
Reviewed-by: Jenny Huang -Original Message- From: Sheng, W Sent: Tuesday, January 11, 2022 9:27 AM To: devel@edk2.groups.io Cc: Ni, Ray ; Chaganty, Rangasai V ; Huang, Jenny ; Kowalewski, Robert Subject: [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level Reduce the debug level

Re: [edk2-devel] [PATCH] BaseTools: Fix wrong variable header size

2022-01-10 Thread Bob Feng
This patch is good to me. Reviewed-by: Bob Feng -Original Message- From: Chen, Lin Z Sent: Tuesday, January 11, 2022 12:58 PM To: Feng, Bob C ; Gao, Liming ; Chen, Christine ; devel@edk2.groups.io Cc: Li, Zhuangzhi ; Zhang, Di ; Chen, Lin Z ; Chen Subject: [PATCH] BaseTools: Fix

[edk2-devel] [PATCH] BaseTools: Fix wrong variable header size

2022-01-10 Thread Chen Lin Z
From: "Chen, Lin Z" There are two type variable header and their size are different, need to use matched size when calculating offset info, otherwise it'll destroy other variables content when patching. Signed-off-by: Chen, Lin Z --- BaseTools/Source/Python/AutoGen/GenVar.py | 6 +- 1

Re: [edk2-devel] [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level

2022-01-10 Thread Chaganty, Rangasai V
Reviewed-by: Sai Chaganty -Original Message- From: Sheng, W Sent: Monday, January 10, 2022 5:27 PM To: devel@edk2.groups.io Cc: Ni, Ray ; Chaganty, Rangasai V ; Huang, Jenny ; Kowalewski, Robert Subject: [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level Reduce the debug level

回复: 回复: [edk2-devel] [PATCH] MdeModulePkg\CoreDxe: Allow DXE Drivers to use untested memory

2022-01-10 Thread gaoliming
Sean: Yes. Platform can report the tested memory for the specific range. Stacy: Can you evaluate Sean's solution for your problem? Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Sean > 发送时间: 2022年1月11日 10:47 > 收件人: devel@edk2.groups.io; gaolim...@byosoft.com.cn; >

[edk2-devel] Cancelled Event: TianoCore Bug Triage - APAC / NAMO - Tuesday, January 11, 2022 #cal-cancelled

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

回复: [edk2-devel] Event: TianoCore Bug Triage - APAC / NAMO - 01/11/2022 #cal-reminder

2022-01-10 Thread gaoliming
Few issues are submitted this week. Let’s cancel the meeting. 3799 EDK2 Code unassig...@tianocore.org UNCO armsoftfloatlib not correctly declared and introduces dependency on ArmPkg

Re: 回复: [edk2-devel] [PATCH] MdeModulePkg\CoreDxe: Allow DXE Drivers to use untested memory

2022-01-10 Thread Sean
if this is auto promotion is happening in the core then what is the value of memory testing and tracking that state. Is memory testing state a necessary feature of the Dxe Core? I think it makes more sense that if you platform wants to use a given range your platform should either test it

Re: [edk2-devel] [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level

2022-01-10 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Sheng, W > Sent: Tuesday, January 11, 2022 9:27 AM > To: devel@edk2.groups.io > Cc: Ni, Ray ; Chaganty, Rangasai V > ; Huang, Jenny > ; Kowalewski, Robert > Subject: [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level > > Reduce

Re: [edk2-devel] [PATCH] UefiPayloadPkg: Change the user interface name of the Uiapp

2022-01-10 Thread Ni, Ray
> +[Rule.Common.UEFI_APPLICATION.UI] > + FILE APPLICATION = $(NAMED_GUID) { > +PE32 PE32 $(INF_OUTPUT)/$(MODULE_NAME).efi > +UISTRING="Enter Setup" > +VERSION STRING="$(INF_VERSION)" Optional BUILD_NUM=$(BUILD_NUMBER) > + } > + > \ No newline at end

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

2022-01-10 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

Re: [edk2-devel] [PATCH 08/10] OvmfPkg: Update Sec to support Tdvf Config-B

2022-01-10 Thread Min Xu
On January 10, 2022 3:56 PM, Gerd Hoffmann wrote: > On Fri, Jan 07, 2022 at 06:13:37AM +, Xu, Min M wrote: > > On January 3, 2022 4:02 PM, Gerd Hoffmann wrote: > > > > > > > PCDs cannot be set in SEC phase, so the values should be saved in > > > > a Hob (for example, PLATFORM_INFO_HOB). In

Re: [edk2-devel] [edk2platforms][PATCH v4 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2022-01-10 Thread Chaganty, Rangasai V
Reviewed-by: Sai Chaganty -Original Message- From: Czajkowski, Maciej Sent: Tuesday, December 07, 2021 7:21 AM To: devel@edk2.groups.io Cc: Ni, Ray ; Chaganty, Rangasai V Subject: [edk2platforms][PATCH v4 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol Added IntelDieInfo header into

回复: [edk2-devel] [PATCH] MdeModulePkg\CoreDxe: Allow DXE Drivers to use untested memory

2022-01-10 Thread gaoliming
Stacy: This fix covers the case with AllocateAddress allocation type. I agree this fix. Reviewed-by: Liming Gao Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Stacy Howell > 发送时间: 2022年1月8日 3:36 > 收件人: devel@edk2.groups.io > 抄送: Stacy Howell ; Dandan Bi > ; Liming Gao > 主题:

[edk2-devel] Issues with CLANGDWARF tool specification and X64 -- am I nuts or what?

2022-01-10 Thread Bill Paul
Hello all: Recently I discovered that you can enable CSM compatibility mode in OVMF and decided to build some images with this feature Because Of Reasons (tm). My platform is: FreeBSD/amd64 12.2-RELEASE Clang/LLVM 10.0.1 QEMU 6.2.0 These days FreeBSD uses Clang as its native compiler.

[edk2-devel] [PATCH] IntelSiliconPkg/IntelVTdDxe: Reduce Debug Level

2022-01-10 Thread Sheng Wei
Reduce the debug level from DEBUG_INFO to DEBUG_VERBOSE in function SubmitQueuedInvalidationDescriptor (). REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3801 Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Jenny Huang Cc: Robert Kowalewski Signed-off-by: Sheng Wei ---

Re: [edk2-devel] [PATCH] Wiki: Add optional steps for developer to run CI test before sending

2022-01-10 Thread Guomin Jiang
Reviewed-by: Guomin Jiang Guomin > -Original Message- > From: devel@edk2.groups.io On Behalf Of > Zhiguang Liu > Sent: Monday, January 10, 2022 11:26 AM > To: devel@edk2.groups.io > Cc: Gao, Liming > Subject: [edk2-devel] [PATCH] Wiki: Add optional steps for developer to run > CI test

回复: [edk2-devel] [PATCH] Wiki: Add optional steps for developer to run CI test before sending

2022-01-10 Thread gaoliming
Zhiguang: Thanks for your update. Reviewed-by: Liming Gao Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Zhiguang Liu > 发送时间: 2022年1月10日 11:26 > 收件人: devel@edk2.groups.io > 抄送: Liming Gao > 主题: [edk2-devel] [PATCH] Wiki: Add optional steps for developer to run CI > test before

回复: [edk2-devel] [PATCH v2 1/6] MdeModulePkg: VariableSmmRuntimeDxe: Fix Variable Policy Message Length

2022-01-10 Thread gaoliming
Reviewed-by: Liming Gao > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Kun Qin > 发送时间: 2021年12月21日 9:33 > 收件人: devel@edk2.groups.io > 抄送: Jian J Wang ; Liming Gao > ; Hao A Wu ; Bret > Barkelew ; Michael Kubacki > > 主题: [edk2-devel] [PATCH v2 1/6] MdeModulePkg: VariableSmmRuntimeDxe: > Fix

Re: [edk2-devel] [PATCH 00/79] Migration of RiscV*Pkg

2022-01-10 Thread Abner Chang
I forget this again. I am going to create one for this. Thanks Abner > -Original Message- > From: gaoliming > Sent: Tuesday, January 11, 2022 9:01 AM > To: devel@edk2.groups.io; Chang, Abner (HPS SW/FW Technologist) > > Cc: 'Sean Brogan' ; 'Bret Barkelew' > ; 'Michael D Kinney' > ; 'Leif

Re: [edk2-devel] [PATCH v2 3/6] SourceLevelDebugPkg: Replace Opcode with the corresponding instructions.

2022-01-10 Thread Wu, Hao A
Reviewed-by: Hao A Wu Best Regards, Hao Wu > -Original Message- > From: Lou, Yun > Sent: Monday, January 10, 2022 11:13 PM > To: devel@edk2.groups.io > Cc: Lou, Yun ; Wu, Hao A > Subject: [PATCH v2 3/6] SourceLevelDebugPkg: Replace Opcode with the > corresponding instructions. > >

回复: [edk2-devel] [PATCH 00/79] Migration of RiscV*Pkg

2022-01-10 Thread gaoliming
Abner: Is there one BZ for this migration? Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Abner Chang > 发送时间: 2022年1月8日 10:26 > 收件人: devel@edk2.groups.io > 抄送: abner.ch...@hpe.com; Sean Brogan ; > Bret Barkelew ; Michael D Kinney > ; Leif Lindholm ; Liming > Gao ; Daniel

回复: [edk2-devel] [PATCH v2 2/6] MdePkg: Replace Opcode with the corresponding instructions.

2022-01-10 Thread gaoliming
Jason: Thanks for your update. Reviewed-by: Liming Gao for this patch set. Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Jason Lou > 发送时间: 2022年1月10日 23:13 > 收件人: devel@edk2.groups.io > 抄送: Jason ; Michael D Kinney > ; Liming Gao ; > Zhiguang Liu > 主题: [edk2-devel] [PATCH v2

回复: [edk2-devel] [PATCH v2 00/10] Create new target for Cloud Hypervisor

2022-01-10 Thread gaoliming
Can you submit one BZ (https://bugzilla.tianocore.org/ ) to track this feature? Thanks Liming > -邮件原件- > 发件人: devel@edk2.groups.io 代表 Boeuf, > Sebastien > 发送时间: 2022年1月11日 1:47 > 收件人: devel@edk2.groups.io > 抄送: jiewen@intel.com; jordan.l.jus...@intel.com; kra...@redhat.com; >

[edk2-devel][edk2-platforms][PATCH V1 1/1] WhitleyOpenBoardPkg/Include: Remove duplicate file

2022-01-10 Thread Oram, Isaac W
Switch to using the IntelSiliconPkg version of the file as both were identical. Cc: Nate DeSimone Cc: Chasel Chiu Signed-off-by: Isaac Oram --- Platform/Intel/WhitleyOpenBoardPkg/Include/Library/SpiFlashCommonLib.h | 98

Re: [edk2-devel] [edk2-platforms PATCH 0/8] Update Arm platforms following addition of EFI_MP_SERVICES_PROTOCOL support in edk2

2022-01-10 Thread Rebecca Cran
Thanks! It looks like the existing changes covered Platform/Marvell/Cn913xDb/Cn913xDbA.dsc and Armada7k8kCapsule isn't something that would be broken by my changes. -- Rebecca Cran On 12/19/21 03:32, Marcin Wojtas wrote: Hi, czw., 16 gru 2021 o 23:09 Rebecca Cran napisał(a): On

Re: [edk2-devel] [PATCH v2 0/6] MM communicate functionality in variable policy

2022-01-10 Thread Kun Qin
Hi MdeModulePkg and ArmPkg maintainers, It has been another week since this v2 patch series has been sent out for review. Could you please take a look and provide feedback? Any input is appreciated. Regards, Kun -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this

Re: [edk2-devel] [PATCH] OvmfPkg/ResetVector: Removing SEV-ES CPUID bit check

2022-01-10 Thread Lendacky, Thomas via groups.io
On 1/10/22 9:29 AM, Peter Gonda wrote: On Fri, Jan 7, 2022 at 3:54 PM Tom Lendacky wrote: On 1/7/22 11:04 AM, Peter Gonda wrote: The SEV-ES bit of Fn800-001F[EAX] - Bit 3 is used for a host to determine support for running SEV-ES guests. It should not be checked by a guest to determine if it

[edk2-devel] [PATCH v2 10/10] OvmfPkg/CloudHv: add Maintainers.txt entry

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Signed-off-by: Sebastien Boeuf --- Maintainers.txt | 5 + 1 file changed, 5 insertions(+) diff --git a/Maintainers.txt b/Maintainers.txt index 71c42bddae..3bda97ef25 100644 --- a/Maintainers.txt +++ b/Maintainers.txt @@ -440,6 +440,11 @@ F:

[edk2-devel] [PATCH v2 09/10] OvmfPkg/CloudHv: Remove CSM support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't need the support for legacy BIOS, therefore the CSM support can be removed. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 20 OvmfPkg/CloudHv/CloudHvX64.fdf | 6 -- 2 files

[edk2-devel] [PATCH v2 08/10] OvmfPkg/CloudHv: Remove USB support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any USB controller or device, therefore the support can be removed. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 11 --- OvmfPkg/CloudHv/CloudHvX64.fdf | 10 -- 2 files changed,

[edk2-devel] [PATCH v2 07/10] OvmfPkg/CloudHv: Remove video support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf No need for video or virtio-gpu support since Cloud Hypervisor doesn't emulate any of these. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 16 OvmfPkg/CloudHv/CloudHvX64.fdf | 4 2 files changed, 20

[edk2-devel] [PATCH v2 06/10] OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor doesn't rely on the FwCfg mechanism, remove the libraries imports when possible. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 -- 1 file changed, 2 deletions(-) diff --git

[edk2-devel] [PATCH v2 05/10] OvmfPkg/CloudHv: Remove Q35 specifics

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Anything specific to the QEMU Q35 platform is not relevant for the CloudHv target. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 10 -- 1 file changed, 10 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH v2 04/10] OvmfPkg/CloudHv: Remove legacy 8259 PIC support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor does not emulate any 8259 PIC, therefore there's no reason to load the corresponding driver for it. Acked-by: Jiewen Yao Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 4 OvmfPkg/CloudHv/CloudHvX64.fdf | 1 - 2 files changed, 5

[edk2-devel] [PATCH v2 03/10] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any LPC bridge, therefore we simply need to rely on the serial I/O port to be connected as a console. It reuses the code from Xen since it's very generic. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH v2 02/10] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why instead of relying on it as the timer UEFI services, rely on the XenTimerDxe implementation. This is not Xen specific, as it simply uses the local APIC timer triggering interrupts on the vector 32.

[edk2-devel] [PATCH v2 01/10] OvmfPkg/CloudHv: Add new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the new target CloudHvX64, copied directly from OvmfPkgX64. The point is to create a target dedicated for Cloud Hypervisor rather than trying to support both QEMU and Cloud Hypervisor on the same target. Improvements and cleanups will be performed in follow up

[edk2-devel] [PATCH v2 00/10] Create new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor and QEMU pc/q35 are quite different, it makes more sense to create a dedicated OVMF target for Cloud Hypervisor rather than trying to support both VMMs from the same OvmfPkgX64 target. That's the reason why this series introduces a new target called

Re: [edk2-devel] [PATCH 01/79] ProcessorPkg/Include: Add header files of RISC-V processor package

2022-01-10 Thread Michael D Kinney
Hi Abner, I see comments from Leif as well. Do you think a discussion in the design meeting Ray Ni runs would be valuable to review all the modules/libs/includes and discuss options for the best location for them to reside in the edk2 repos? Thanks, Mike > -Original Message- > From:

Re: [edk2-devel] [PATCH 01/79] ProcessorPkg/Include: Add header files of RISC-V processor package

2022-01-10 Thread Michael D Kinney
Hi Abner, I would also like to see some proposals on adding the RiscV CPU scoped content to the existing MdePkg/UefiCpuPkg instead of adding a new top level CPU package. There is already some work started to move some of the ARM specific content from ARM CPU packages into MdePkg. Thanks, Mike

Re: [edk2-devel] [PATCH] OvmfPkg/ResetVector: Removing SEV-ES CPUID bit check

2022-01-10 Thread Peter Gonda via groups.io
On Fri, Jan 7, 2022 at 3:54 PM Tom Lendacky wrote: > > On 1/7/22 11:04 AM, Peter Gonda wrote: > > The SEV-ES bit of Fn800-001F[EAX] - Bit 3 is used for a host to > > determine support for running SEV-ES guests. It should not be checked by > > a guest to determine if it is running under SEV-ES.

[edk2-devel] [PATCH v2 6/6] BaseTools: Upgrade the version of NASM tool

2022-01-10 Thread Jason Lou
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3790 Upgrade the version of NASM tool to avoid compilation errors when compiling NASM code change. Signed-off-by: Jason Lou Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen --- BaseTools/Conf/tools_def.template | 6 +++--- 1 file changed, 3

[edk2-devel] [PATCH v2 5/6] MdePkg: Remove the macro definitions regarding Opcode.

2022-01-10 Thread Jason Lou
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3790 Remove the macro definitions regarding Opcode because new version of NASM tool(e.g. v2.15.05) supports the corresponding instructions. Note: This patch need to be merged after other NASM code change to avoid compilation errors.

[edk2-devel] [PATCH v2 4/6] UefiCpuPkg: Replace Opcode with the corresponding instructions.

2022-01-10 Thread Jason Lou
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3790 Replace Opcode with the corresponding instructions. The code changes have been verified with CompareBuild.py tool, which can be used to compare the results of two different EDK II builds to determine if they generate the same binaries.

[edk2-devel] [PATCH v2 1/6] MdeModulePkg: Replace Opcode with the corresponding instructions.

2022-01-10 Thread Jason Lou
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3790 Replace Opcode with the corresponding instructions. The code changes have been verified with CompareBuild.py tool, which can be used to compare the results of two different EDK II builds to determine if they generate the same binaries.

[edk2-devel] [PATCH v2 3/6] SourceLevelDebugPkg: Replace Opcode with the corresponding instructions.

2022-01-10 Thread Jason Lou
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3790 Replace Opcode with the corresponding instructions. The code changes have been verified with CompareBuild.py tool, which can be used to compare the results of two different EDK II builds to determine if they generate the same binaries.

[edk2-devel] [PATCH v2 2/6] MdePkg: Replace Opcode with the corresponding instructions.

2022-01-10 Thread Jason Lou
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3790 Replace Opcode with the corresponding instructions. The code changes have been verified with CompareBuild.py tool, which can be used to compare the results of two different EDK II builds to determine if they generate the same binaries.

Re: [edk2-devel] [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Yao, Jiewen
Exactly > -Original Message- > From: Boeuf, Sebastien > Sent: Monday, January 10, 2022 10:54 PM > To: Yao, Jiewen ; devel@edk2.groups.io > Cc: kra...@redhat.com; Justen, Jordan L > Subject: Re: [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console > > On Mon, 2022-01-10 at 14:46 +,

Re: [edk2-devel] [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 14:46 +, Yao, Jiewen wrote: > Please copy the description as comment to the code. Do you mean you want me to use my commit message as a comment in the code to explain why we have this special use case for Cloud Hypervisor? I'll be happy to do it, just making sure that's

Re: [edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 14:49 +, Yao, Jiewen wrote: > Thanks. For 1~2,4~9, Acked-by: Jiewen Yao Thanks for the quick review :) > > Please also update > https://github.com/tianocore/edk2/blob/master/Maintainers.txt > to add a CloudHv section with you as Reviewer. Similar to below >

Re: [edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor

2022-01-10 Thread Yao, Jiewen
Thanks. For 1~2,4~9, Acked-by: Jiewen Yao Please also update https://github.com/tianocore/edk2/blob/master/Maintainers.txt to add a CloudHv section with you as Reviewer. Similar to below OvmfPkg: microvm-related modules F: OvmfPkg/Microvm/ F: OvmfPkg/Include/IndustryStandard/Microvm.h

Re: [edk2-devel] [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Yao, Jiewen
Please copy the description as comment to the code. Thank you Yao Jiewen > -Original Message- > From: Boeuf, Sebastien > Sent: Monday, January 10, 2022 10:26 PM > To: devel@edk2.groups.io > Cc: Yao, Jiewen ; Justen, Jordan L > ; kra...@redhat.com; Boeuf, Sebastien > > Subject: [PATCH

[edk2-devel] [PATCH 9/9] OvmfPkg/CloudHv: Remove CSM support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't need the support for legacy BIOS, therefore the CSM support can be removed. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 20 OvmfPkg/CloudHv/CloudHvX64.fdf | 6 -- 2 files changed, 26 deletions(-)

[edk2-devel] [PATCH 8/9] OvmfPkg/CloudHv: Remove USB support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any USB controller or device, therefore the support can be removed. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 11 --- OvmfPkg/CloudHv/CloudHvX64.fdf | 10 -- 2 files changed, 21 deletions(-) diff

[edk2-devel] [PATCH 7/9] OvmfPkg/CloudHv: Remove video support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf No need for video or virtio-gpu support since Cloud Hypervisor doesn't emulate any of these. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 16 OvmfPkg/CloudHv/CloudHvX64.fdf | 4 2 files changed, 20 deletions(-) diff --git

[edk2-devel] [PATCH 6/9] OvmfPkg/CloudHv: Reduce dependency on QemuFwCfg

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor doesn't rely on the FwCfg mechanism, remove the libraries imports when possible. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 2 -- 1 file changed, 2 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH 5/9] OvmfPkg/CloudHv: Remove Q35 specifics

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Anything specific to the QEMU Q35 platform is not relevant for the CloudHv target. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 10 -- 1 file changed, 10 deletions(-) diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH 4/9] OvmfPkg/CloudHv: Remove legacy 8259 PIC support

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor does not emulate any 8259 PIC, therefore there's no reason to load the corresponding driver for it. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc | 4 OvmfPkg/CloudHv/CloudHvX64.fdf | 1 - 2 files changed, 5 deletions(-) diff

[edk2-devel] [PATCH 3/9] OvmfPkg/CloudHv: Connect serial console

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate any LPC bridge, therefore we simply need to rely on the serial I/O port to be connected as a console. It reuses the code from Xen since it's very generic. Signed-off-by: Sebastien Boeuf --- OvmfPkg/CloudHv/CloudHvX64.dsc

[edk2-devel] [PATCH 2/9] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why instead of relying on it as the timer UEFI services, rely on the XenTimerDxe implementation. This is not Xen specific, as it simply uses the local APIC timer triggering interrupts on the vector 32.

[edk2-devel] [PATCH 1/9] OvmfPkg/CloudHv: Add new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Adding the new target CloudHvX64, copied directly from OvmfPkgX64. The point is to create a target dedicated for Cloud Hypervisor rather than trying to support both QEMU and Cloud Hypervisor on the same target. Improvements and cleanups will be performed in follow up

[edk2-devel] [PATCH 0/9] Create new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
From: Sebastien Boeuf Since Cloud Hypervisor and QEMU pc/q35 are quite different, it makes more sense to create a dedicated OVMF target for Cloud Hypervisor rather than trying to support both VMMs from the same OvmfPkgX64 target. That's the reason why this series introduces a new target called

Re: [edk2-devel] [edk2-non-osi][PATCH 1/1] Platform/RaspberryPi: Update TF-A to v2.6 to enable the PCI SMCCC interface

2022-01-10 Thread Ard Biesheuvel
On Mon, 10 Jan 2022 at 14:43, Pete Batard wrote: > > The RPi4 has a single nonstandard PCI config region. It is broken into two > pieces, the root port config registers and a window to a single device's > config space which can move between devices. However there isn't (yet) an > authoritative

[edk2-devel] [edk2-non-osi][PATCH 1/1] Platform/RaspberryPi: Update TF-A to v2.6 to enable the PCI SMCCC interface

2022-01-10 Thread Pete Batard
The RPi4 has a single nonstandard PCI config region. It is broken into two pieces, the root port config registers and a window to a single device's config space which can move between devices. However there isn't (yet) an authoritative public document on this since the available BCM2711 reference

Re: [edk2-devel] [PATCH 01/79] ProcessorPkg/Include: Add header files of RISC-V processor package

2022-01-10 Thread Leif Lindholm
On Sat, Jan 08, 2022 at 12:07:53 +0800, Abner Chang wrote: > (This is migrated from edk2-platforms:Silicon/RISC-V) > > RISC-V processor package library definitions. > > IndustryStandard/RiscV.h > -Add RiscV.h which conform with RISC-V Privilege Spec v1.10. > > RiscVImpl.h > -Definition of EDK2

Re: [edk2-devel] [PATCH 00/79] Migration of RiscV*Pkg

2022-01-10 Thread Leif Lindholm
Hey Abner, Happy to see this happen. But (as expected) some comments below. On Sat, Jan 08, 2022 at 10:25:37 +0800, Abner Chang wrote: > This is the migration of RISC-V related packages from edk2-platforms repo to > edk2 repo, as the RISC-V edk2 port is getting mature and the demands of RISC-V >

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Gerd Hoffmann
Hi, > > So, what problem you are facing which makes you think a separate > > target would work better?  The timer thing should be a non-issue as > > we plan to switch over OvmfPkgX64 to use apic timer anyway. > Well I have a problem regarding SerialDxe because it breaks a bit QEMU > since

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 11:45 +0100, kra...@redhat.com wrote: > On Mon, Jan 10, 2022 at 09:13:44AM +, Boeuf, Sebastien wrote: > > Hi all, > > > > So far I've been able to patch the OvmfPkgX64 target to make it > > work for both > > QEMU and Cloud Hypervisor, but as I try to enable more features

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 10:35 +, Yao, Jiewen wrote: > Looking at current OvmfPkg today. We have: > OvmfPkg\OvmfPkgIa32.fdf > OvmfPkg\OvmfPkgIa32X64.fdf > OvmfPkg\OvmfPkgX64.fdf > OvmfPkg\OvmfXen.fdf > OvmfPkg\AmdSev\AmdSevX64.fdf > OvmfPkg\Bhyve\BhyveX64.fdf > OvmfPkg\Microvm\MicrovmX64.fdf > >

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Gerd Hoffmann
On Mon, Jan 10, 2022 at 09:13:44AM +, Boeuf, Sebastien wrote: > Hi all, > > So far I've been able to patch the OvmfPkgX64 target to make it work for both > QEMU and Cloud Hypervisor, but as I try to enable more features (EFI shell for > instance) the gap is getting bigger and harder to keep

Re: [edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Yao, Jiewen
Looking at current OvmfPkg today. We have: OvmfPkg\OvmfPkgIa32.fdf OvmfPkg\OvmfPkgIa32X64.fdf OvmfPkg\OvmfPkgX64.fdf OvmfPkg\OvmfXen.fdf OvmfPkg\AmdSev\AmdSevX64.fdf OvmfPkg\Bhyve\BhyveX64.fdf OvmfPkg\Microvm\MicrovmX64.fdf And we will have OvmfPkg\IntelTdx\IntelTdxX64.fdf soon. I think it is OK

[edk2-devel] Creating new target for Cloud Hypervisor

2022-01-10 Thread Boeuf, Sebastien
Hi all, So far I've been able to patch the OvmfPkgX64 target to make it work for both QEMU and Cloud Hypervisor, but as I try to enable more features (EFI shell for instance) the gap is getting bigger and harder to keep them working together. That's why I'm thinking about creating an OvmfCh

Re: [edk2-devel] EFI shell with microvm

2022-01-10 Thread Boeuf, Sebastien
On Mon, 2022-01-10 at 00:16 +, Xu, Min M wrote: > On January 7, 2022 9:37 PM, Gerd Hoffmann wrote: > > > > > > tianocore doesn't use interrupts (other than timer). > > > > > > Yes I realized that by diving into the code. I can see microvm > > > using > > > the Xen timer while OvmfPkgX64 uses

[edk2-devel] [PATCH] MdeModulePkg/HiiDatabaseDxe: Add Support for authenticated variable

2022-01-10 Thread Long1 Huang
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3796 Database.c: 1. Replace PcdGetExPtr with PcdGetExPtr. 2. Add FindAuthVariableData function to parse authenticated variable type for getting a correct default value in PcdNvStoreDefaultValueBuffer. Cc: Liming Gao Cc: Chen

[edk2-devel] [PATCH] REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3796

2022-01-10 Thread Long1 Huang
MdeModulePkg/HiiDatabaseDxe: Add Support for authenticated variable Database.c: 1. Replace PcdGetExPtr with PcdGetExPtr. 2. Add FindAuthVariableData function to parse authenticated variable type for getting a correct default value in PcdNvStoreDefaultValueBuffer. Cc: Liming Gao