Hi,
> One more question, there is already a
> QemuFwCfgLib library for ia32/x64. The naming of QemuFwCfgCommonLib
> under Library seems confusing. How about we put this library under
> /FDT and also name it as QemuFwCfgLib? Because ARM/RISC-V fw_cfg also
> depends on FDT.
Move it to
From: Michael Kubacki
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3512
In 32-bit PEI, the local variable pointers MigratedFvHeader and
RawDataFvHeader in EvacuateTempRam() will be 32-bit in size. The
pointers are currently passed to PeiServicesAllocatePages() which
expects a 64-bit
From: Michael Kubacki
Currently, the debug message that prints the name of a PEIM
being migrated is DEBUG_INFO while similar messages are
DEBUG_VERBOSE. This change updates the print level to
DEBUG_VERBOSE for consistency.
Cc: Jian J Wang
Cc: Liming Gao
Cc: Dandan Bi
Signed-off-by: Michael
From: Michael Kubacki
Corrects a typo in the return value description of MigratePeim() and
EvacuateTempRam():
"Succesfully" to "Successfully"
Cc: Jian J Wang
Cc: Liming Gao
Cc: Dandan Bi
Signed-off-by: Michael Kubacki
---
MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 4 ++--
From: Michael Kubacki
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3512
This patch series contains three patches. The first two are for
relatively minor improvments - a typo fix in function descriptions
and changing the error level of a debug print. The third patch
fixes a pointer size
Hi Caden,
The patch doesn't address all the comment I point out. If you think I am
incorrect, please feel free to point out. See below:
> -Original Message-
> From: Caden Kline
> Sent: Monday, August 23, 2021 10:44 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J ; Wu, Hao A ;
> Gao,
Reviewed-by: Zhichao Gao
Thanks,
Zhichao
> -Original Message-
> From: Abdul Lateef Attar
> Sent: Tuesday, August 24, 2021 11:30 PM
> To: devel@edk2.groups.io
> Cc: Ni, Ray ; Gao, Zhichao ;
> Abdul Lateef Attar
> Subject: [edk2-devel] [PATCH] ShellPkg: Parse I/O APIC and x2APIC
Reviewed-by: Zhichao Gao
Thanks,
Zhichao
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Sami
> Mujawar
> Sent: Wednesday, August 18, 2021 7:09 PM
> To: Christopher Jones ;
> devel@edk2.groups.io
> Cc: Ni, Ray ; Gao, Zhichao ; Ben
> Adderson ; Akanksha Jain
> ; Matteo
Reviewed-by: Zhichao Gao
Thanks,
Zhichao
> -Original Message-
> From: Sami Mujawar
> Sent: Wednesday, August 18, 2021 5:57 PM
> To: Christopher Jones ;
> devel@edk2.groups.io
> Cc: Ni, Ray ; Gao, Zhichao ;
> Alexei Fedorov ; Ben Adderson
> ; Akanksha Jain ;
> Matteo Carlini ; nd
>
For this patch set, Reviewed-by: Zhichao Gao
Thanks,
Zhichao
> -Original Message-
> From: Chris Jones
> Sent: Saturday, August 14, 2021 1:03 AM
> To: devel@edk2.groups.io
> Cc: Ni, Ray ; Gao, Zhichao ;
> sami.muja...@arm.com; ben.adder...@arm.com;
> akanksha.ja...@arm.com;
Reviewed-by: Zhiguang Liu
From: devel@edk2.groups.io on behalf of duntan
Sent: Tuesday, August 31, 2021 13:38
To: devel@edk2.groups.io
Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice
; You, Benjamin ; Liu, Zhiguang
; Tan, Dun
Subject: [edk2-devel] [Patch V2 2/2]
Thank Dun for the fix.
Reviewed-by: Zhiguang Liu
From: Tan, Dun
Sent: Tuesday, August 31, 2021 13:38
To: devel@edk2.groups.io
Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice
; You, Benjamin ; Liu, Zhiguang
; Tan, Dun
Subject: [Patch V2 1/2] UefiPayloadPkg: Fix the bug
Acked-by: Liming Gao for this patch set.
Thanks
Liming
> -邮件原件-
> 发件人: devel@edk2.groups.io 代表 Ard
> Biesheuvel
> 发送时间: 2021年8月30日 21:46
> 收件人: Rebecca Cran
> 抄送: Bob Feng ; Liming Gao
> ; Yuwei Chen ; Sean
> Brogan ; Sami Mujawar
> ; Leif Lindholm ; Ard
> Biesheuvel ;
On September 9, 2021 3:46 AM, Brijesh Singh wrote:
>
> Thank you so much Yao for reviewing the patches. Based on some comments
> from Gerd I may update code around the reset vector area (mainly use the
> metadata format etc). For your comments regarding the introducing a new
>
Hi, Pierre.
Thank you very much for quickly reply.
Based on Pierre's advice,
I deleted the edk2 directory that I had been using and ran the git clone of
edk2 again,
also used the master branch to specify the latest commit as of yesterday.
I also took Sami's advice and specified Python 3.7.
Hi, Sami.
Thank you very much for quickly reply.
> Please let me know if switching to Python 3.7 does not fix this issue for you.
As advised, I compiled using Python-3.7 (Python-3.7.9).
If I simply changed the Python version,
the build did not finish successfully and the same error occurred.
Many reasons on an Entrypoint not called.
I share my experence below (I encountered all of them before):
1) You update the code in a wrong tree.
2) The image is not rebuild. You still use old image.
3) The image is rebuild, but the image is not reburn to the flash.
4) The driver is not in final
Hi, Pierre.
Thank you very much for quickly reply.
Based on Pierre's advice,
I deleted the edk2 directory that I had been using and ran the git clone of
edk2 again,
,also used the master branch to specify the latest commit as of yesterday.
I also took Sami's advice and specified Python 3.7.
I forgot to mention that there may be problems with older python,
so I will basically use the newer versions of python as advised.
Best regards,
Shuuichrou.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#80385):
Hi, Sami.
Thank you very much for quickly reply.
> Please let me know if switching to Python 3.7 does not fix this issue for you.
As advised, I compiled using Python-3.7 (Python-3.7.9).
If I simply changed the Python version,
the build did not finish successfully and the same error occurred.
That sounds very reasonable to me. I have done something similar with basic
PPI/Protocol services to abstract phase specific differences that a lot of code
simply doesn't need to care about. I even like the idea of a general "phase
agnostic services library" that handles these kinds of things
Replace Info with Load, so we still get component loading details
Cc: Isaac Oram
Cc: Nate DeSimone
Cc: Chasel Chiu
Signed-off-by: Isaac Oram
---
Platform/Intel/WhitleyOpenBoardPkg/PlatformPkg.dsc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Question:
I’ve recently brought up with some that the GetVariable##() functions in
UefiLib are probably incorrect as they are written (because they assume things
like gRT and gBS availability). Would it perhaps make sense to move this
interface up into the MdePkg scope and leverage it to start
Add the VariableReadLib and VariableWriteLib instances to Components to
ensure build when building MinPlatformPkg.dsc.
Add a NULL library instance that provides the non-functional library
instance for VariableReadLib designed for all phase use.
Cc: Chasel Chiu
Cc: Nate DeSimone
Cc: Liming Gao
On 9/8/21 10:37 AM, Yao, Jiewen wrote:
Hi Stefan
According to our security policy, the PPI must be sent before EndOfDxe.
Then registering PlatformAuth clear at EndOfDxe is safe. I still don’t get your
point on why we have do in PlatformBds.
At least, I do want to make sure all X86
Thank you so much Yao for reviewing the patches. Based on some comments
from Gerd I may update code around the reset vector area (mainly use the
metadata format etc). For your comments regarding the introducing a new
PcdConfidentialComputingCategory I will look to see what I can come up
with
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3290
When BuildAtaDeviceNode() is called with L"Ata(0,0,0)" it creates a node
Ata(1,1,0). This leads to test to an assertion failed test result
../DevicePathToTextBBTestFunction.c:332:
Convert result: Ata(0,0,0) - Expected:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3390
According to the UEFI specification
EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL.SetState() may return EFI_UNSUPPORTED if
"the device does not support the ability to have its state set."
BBTestReadKeyStrokeExFunctionAutoTestCheckpoint1() must not
This adds more helper functions that assist in calculating the checksum,
locating an ACPI table by signature, and updating an AML integer object.
Cc: Ard Biesheuvel
Cc: Daniel Schaefer
Signed-off-by: Nhi Pham
Acked-by: Leif Lindholm
Reviewed-by: Abner Chang
---
Changes since v2:
+ Correct
Thanks Abner.
I will correct the description for the function
AcpiLocateTableBySignature in the v3.
Best regards,
Nhi
On 08/09/2021 11:53, Chang, Abner (HPS SW/FW Technologist) wrote:
After below comments are addressed,
Reviewed-by: Abner Chang
-Original Message-
From:
Zhichao,
I didn't mean to update UI logic to support scrolling. I meant to update BM
logic to show partial string of boot description.
Thanks,
Ray
> -Original Message-
> From: Gao, Zhichao
> Sent: Wednesday, September 8, 2021 10:29 AM
> To: Ni, Ray ; devel@edk2.groups.io
> Cc: Wang,
Hi Leif,
Thanks for your reviewing.
These functions in this patch are consumed in Ampere Mt. Jade platform.
We can defer merging this patch until I send the v3 of Ampere Mt. Jade
patchset out.
Best regards,
Nhi
On 08/09/2021 01:00, Leif Lindholm wrote:
Hi Nhi,
On Fri, Sep 03, 2021 at
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3611
This commit contains updates made to the Readme.md file
of edk2-libc repo to add the Python 3.6.8 license at
appropriate section in the file
Cc: Rebecca Cran
Cc: Michael D Kinney
---
Readme.md | 1 +
1 file changed, 1 insertion(+)
diff
Jayaprakash Nevara (1):
edk2-libc/Readme.md: Updated Readme.md with Python 3.6.8 License
details
Readme.md | 1 +
1 file changed, 1 insertion(+)
--
2.32.0.windows.2
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#80370):
On Wed, Sep 08, 2021 at 15:33:34 +, Abner Chang wrote:
> > > I prefer to have RISC-V instance because I
> > > am not sure if any fw_cfg interface changes would be made on RISC-V
> > > QEMU.
> >
> > The fw_cfg interface should not diverge. ia32/x64 works a bit different
> > because there is a
> -Original Message-
> From: 'Gerd Hoffmann' [mailto:kra...@redhat.com]
> Sent: Wednesday, September 8, 2021 9:37 PM
> To: Chang, Abner (HPS SW/FW Technologist)
> Cc: Leif Lindholm ; Yao, Jiewen ;
> devel@edk2.groups.io; gaoliming ; 'Ard
> Biesheuvel' ; Kinney, Michael D
> ; Ni, Ray ;
Hi Stefan
According to our security policy, the PPI must be sent before EndOfDxe.
Then registering PlatformAuth clear at EndOfDxe is safe. I still don’t get your
point on why we have do in PlatformBds.
At least, I do want to make sure all X86 implementation are align to one
solution.
Also, for
I believe this patch series is just waiting for approval from the ShellPkg
maintainers. As this has been waiting for several weeks, please could this get
reviewed soon.
Thanks,
Chris
From: Sami Mujawar
Sent: Wednesday, August 18, 2021 12:08 PM
To: Christopher
I believe this patch series is just waiting for approval from the ShellPkg
maintainers. As this has been waiting for several weeks, please could this get
reviewed soon.
Thanks,
Chris
From: Sami Mujawar
Sent: Wednesday, August 18, 2021 11:32 AM
To: Christopher
I believe this patch series is just waiting for approval from the ShellPkg
maintainers. As this has been waiting for several weeks, please could this get
reviewed soon.
Thanks,
Chris
From: Sami Mujawar
Sent: Wednesday, August 18, 2021 10:57 AM
To: Christopher
Fix some bugs in the original PeiDxeTpmPlatformHierarchyLib.c.
Signed-off-by: Stefan Berger
---
.../PeiDxeTpmPlatformHierarchyLib.c | 23 +--
1 file changed, 6 insertions(+), 17 deletions(-)
diff --git
This series imports code from the edk2-platforms project related to
disabling the TPM2 platform hierarchy in Ovmf and ArmVirtPkg. It
addresses the Ovmf aspects of the following bugs:
https://bugzilla.tianocore.org/show_bug.cgi?id=3510
https://bugzilla.tianocore.org/show_bug.cgi?id=3499
I have
Signed-off-by: Stefan Berger
---
ArmVirtPkg/ArmVirtCloudHv.dsc | 1 +
ArmVirtPkg/ArmVirtQemu.dsc | 2 ++
ArmVirtPkg/ArmVirtQemuKernel.dsc| 1 +
ArmVirtPkg/ArmVirtXen.dsc
If Tpm2Startup(TPM_SU_STATE) fails, call ConfigureTPMPlatformHierarchy()
to disable the platform hierarchy.
Signed-off-by: Stefan Berger
---
SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c | 2 ++
SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.inf | 1 +
2 files changed, 3 insertions(+)
diff --git
Import PeiDxeTpmPlatformHierarchyLib.c from edk2-platforms.
Signed-off-by: Stefan Berger
---
.../Include/Library/TpmPlatformHierarchyLib.h | 27 ++
.../PeiDxeTpmPlatformHierarchyLib.c | 266 ++
.../PeiDxeTpmPlatformHierarchyLib.inf | 45 +++
3 files changed,
Use the newly added function to disable the TPM2 platform hierarchy.
Do this after handling physical presence interface opcodes because
the TPM 2 commands they produce may require access to the platform
hierarchy.
Signed-off-by: Stefan Berger
---
Use the newly added function to disable the TPM2 platform hierarchy.
Signed-off-by: Stefan Berger
---
ArmVirtPkg/Library/PlatformBootManagerLib/PlatformBm.c | 6 ++
.../PlatformBootManagerLib/PlatformBootManagerLib.inf | 1 +
2 files changed, 7 insertions(+)
diff --git
Introduce the new PCD
gEfiSecurityPkgTokenSpaceGuid.PcdRandomizePlatformHierarchy.
Signed-off-by: Stefan Berger
---
.../PeiDxeTpmPlatformHierarchyLib.inf | 3 +--
SecurityPkg/SecurityPkg.dec | 6 ++
2 files changed, 7 insertions(+), 2
Add a NULL implementation of the library class TpmPlatformHierarchyLib
Signed-off-by: Stefan Berger
---
.../PeiDxeTpmPlatformHierarchyLib.c | 19
.../PeiDxeTpmPlatformHierarchyLib.inf | 31 +++
2 files changed, 50 insertions(+)
create mode 100644
Compile the added TPM related code now.
Signed-off-by: Stefan Berger
---
OvmfPkg/AmdSev/AmdSevX64.dsc| 2 ++
OvmfPkg/Bhyve/BhyveX64.dsc | 1 +
.../Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 1 +
Hi,
> > Are you expecting to need different functionality from the ArmVirtPkg
> > one? If not, you could move that one to
> > OvmfPkg/Library/QemuFwCfgCommonLib
> > or something.
> The interfaces provided by ARM QemuFwLib seems works fine with RISC-V
> OVMF, however the build fail with the
On 9/6/21 8:34 AM, Yao, Jiewen wrote:
2) I am curious, why you don't use a DXE driver, but choose to like to BDS lib
for the DXE case.
You also include a NULL lib there, which seems unnecessary, if you use a
DXE/PEI module.
The downside of linking to BDS lib is that you have to change all
> -Original Message-
> From: Leif Lindholm [mailto:l...@nuviainc.com]
> Sent: Wednesday, September 8, 2021 1:22 AM
> To: Chang, Abner (HPS SW/FW Technologist)
> Cc: Yao, Jiewen ; devel@edk2.groups.io; gaoliming
> ; 'Ard Biesheuvel' ;
> Kinney, Michael D ; Ni, Ray
> ; Schaefer, Daniel ;
On Wed, Sep 08, 2021 at 13:46:43 +0200, Gerd Hoffmann wrote:
> On Wed, Sep 08, 2021 at 12:13:12PM +0100, Leif Lindholm wrote:
> > On Sun, Aug 29, 2021 at 23:14:48 -0600, Rebecca Cran wrote:
> > > Linaro no longer do gcc releases - Arm creates them now.
> > >
> > > Update the
On Wed, Sep 08, 2021 at 13:33:51 +0200, Gerd Hoffmann wrote:
> On Wed, Sep 08, 2021 at 12:06:46PM +0100, Leif Lindholm wrote:
> > On Wed, Sep 08, 2021 at 11:01:11 +0200, Gerd Hoffmann wrote:
> > > Skip host bridge setup on microvm.
> > >
> > > Ref:
On Wed, Sep 08, 2021 at 12:13:12PM +0100, Leif Lindholm wrote:
> On Sun, Aug 29, 2021 at 23:14:48 -0600, Rebecca Cran wrote:
> > Linaro no longer do gcc releases - Arm creates them now.
> >
> > Update the gcc_[arm,aarch64]_linux_ext_dep.yaml files in BaseTools/Bin to
> > switch from Linaro's old
On Wed, Sep 08, 2021 at 12:06:46PM +0100, Leif Lindholm wrote:
> On Wed, Sep 08, 2021 at 11:01:11 +0200, Gerd Hoffmann wrote:
> > Skip host bridge setup on microvm.
> >
> > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
> > Signed-off-by: Gerd Hoffmann
> > Acked-by: Jiewen Yao
> > ---
On Sun, Aug 29, 2021 at 23:14:48 -0600, Rebecca Cran wrote:
> Linaro no longer do gcc releases - Arm creates them now.
>
> Update the gcc_[arm,aarch64]_linux_ext_dep.yaml files in BaseTools/Bin to
> switch from Linaro's old release to the latest gcc 10.3-2021.07 release
> from Arm and fix
On Wed, Sep 08, 2021 at 11:01:19 +0200, Gerd Hoffmann wrote:
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
> Signed-off-by: Gerd Hoffmann
> Reviewed-by: Jiewen Yao
Reviewed-by: Leif Lindholm
> ---
> Maintainers.txt | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git
On Wed, Sep 08, 2021 at 11:01:11 +0200, Gerd Hoffmann wrote:
> Skip host bridge setup on microvm.
>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
> Signed-off-by: Gerd Hoffmann
> Acked-by: Jiewen Yao
> ---
> OvmfPkg/PlatformPei/MemDetect.c | 4
> 1 file changed, 4
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Reviewed-by: Jiewen Yao
---
Maintainers.txt | 6 ++
1 file changed, 6 insertions(+)
diff --git a/Maintainers.txt b/Maintainers.txt
index 41f491bcaed1..52f9aa99ac7d 100644
--- a/Maintainers.txt
+++
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/README | 54 ++
1 file changed, 54 insertions(+)
create mode 100644 OvmfPkg/Microvm/README
diff --git a/OvmfPkg/Microvm/README
Shortcut PCI support for now (proper PCIe
support will be wired up later).
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Microvm has no LPC bridge, so drop the PciSioSerialDxe driver.
Use SerialDxe instead, with ioport hardcoded to 0x3f8 aka com1 aka ttyS0.
With this tianocore boots to uefi shell prompt on the serial console.
Direct kernel boot can be used too.
Ref:
Nothing to do here ;)
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c
Add + set microvm id for PcdOvmfHostBridgePciDevId.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Include/IndustryStandard/Microvm.h | 12
OvmfPkg/PlatformPei/Platform.c | 7 +++
2 files
Uses the generic event device to reset and poweroff.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc| 14 +--
.../BaseResetSystemLibMicrovm.inf | 37
Microvm needs ioapic hobs only.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/PlatformPei/Platform.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/OvmfPkg/PlatformPei/Platform.c
Skip host bridge setup on microvm.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/PlatformPei/MemDetect.c | 4
1 file changed, 4 insertions(+)
diff --git a/OvmfPkg/PlatformPei/MemDetect.c
Microvm has no acpi timer, so use the generic lib instead.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git
Microvm has no acpi timer, so go use XenTimerDxe
which uses the local apic instead.
Set PcdFSBClock to 1000 MHz, which is the lapic
timer frequency used by KVM.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
Microvm focus is virtio, so go drop support
for emulated scsi host adapters.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 9 -
OvmfPkg/Microvm/MicrovmX64.fdf | 9 -
2 files
Microvm has no SEV support.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 1 -
OvmfPkg/Microvm/MicrovmX64.fdf | 2 --
2 files changed, 3 deletions(-)
diff --git a/OvmfPkg/Microvm/MicrovmX64.dsc
Microvm has no TPM support.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Reviewed-by: Stefan Berger
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 76 +-
OvmfPkg/Microvm/MicrovmX64.fdf | 18
2 files
Guests depending on BIOS will probably not work that well with microvm
due to legacy hardware being not available.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 26 --
Without SMM secure boot isn't actually secure, so drop it too.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 15 ---
OvmfPkg/Microvm/MicrovmX64.fdf | 4
2 files changed, 19
Rename the firmware volume files (s/OVMF/MICROVM/).
Fix includes so they work with microvm config being in a subdirectory.
With this patch applied the build works.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
Microvm has no SMM support.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
OvmfPkg/Microvm/MicrovmX64.dsc | 86 +-
OvmfPkg/Microvm/MicrovmX64.fdf | 30
2 files changed, 1 insertion(+),
This patch series adds basic support for the
microvm machine type to OVMF.
Working:
- uefi shell @ serial console.
- direct kernel boot.
Comming in followup patches:
- virtio-mmio support.
- pcie support.
Usage:
qemu-system-x86_64 -nographic -M microvm -bios MICROVM.fd ...
v2
Create Microvm subdirectory. Copy OvmfPkgX64 .dsc and .fdf files
unmodified as starting point for MicrovmX64.
Changes come as separate patches, to simplify patch review and rebases.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3599
Signed-off-by: Gerd Hoffmann
Acked-by: Jiewen Yao
---
> +//
> +// Root bridge resource information for parsing fw-cfg data
> +//
> +#pragma pack(1)
> +typedef struct {
> + BOOLEAN set_pxm;
> + BOOLEAN mark_prefetchable_resources_as_cacheable;
> +
> + UINT8 root_bus_nr;
> + UINT8 pxm;
> + UINT8 num_hot_plug_slots;
> +
> + UINT8
Really sorry for the late response.
So this is a workaround for RCiEP device that is not compliant to the PCIe spec:
|> ARI is an optional capability. This capability must be implemented by each
|> Function in an ARI Device. It is not applicable to a Root Port, a Switch
|> Downstream Port, an
Hi, Brijesh
When I go thru the code I find a potential bug in MemEncryptSevEsIsEnabled().
In the current code both SEV and TDX leverage the OvmfWorkArea to record the
SEV/TDX information.
Byte [0] record the guest type, 0 for legacy guest. 1 for sev, 2 for tdx.
Byte [3:1] are reserved.
From
Yes, that is good idea.
> -Original Message-
> From: Xu, Min M
> Sent: Wednesday, September 8, 2021 10:30 AM
> To: Yao, Jiewen ; Brijesh Singh
> ; devel@edk2.groups.io
> Cc: James Bottomley ; Tom Lendacky
> ; Justen, Jordan L ;
> Ard Biesheuvel ; Erdem Aktas
> ; Michael Roth ; Gerd
>
84 matches
Mail list logo
