Hi Tien Hock
Can you help to review the patch?
Regards
Gaurav Jain
> -Original Message-
> From: Loh, Tien Hock
> Sent: Tuesday, April 7, 2020 1:23 PM
> To: Gaurav Jain ; Ard Biesheuvel
> ; Leif Lindholm
> Cc: devel@edk2.groups.io; Pankaj Bansal ;
> Haojian Zhuang
> Subject: RE: [EXT]
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 7/9] M
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 6/9] M
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Leif Lindholm ; Kinney, Michael D
; Gao, Liming ; Leif Lindholm
; Gilbert Chen ; Daniel Helmut
Schaef
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 8/9] M
Hi Abner,
In BaseIoLibIntrinsic.inf, you add the line "This I/O library only provides non
I/O read and write."
I supposed this is a description for ARM and RISC-V but not for other arch. If
so, please specify it because this is an inf files all arch are using.
Thanks
Zhiguang
-Original Mes
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 4/9] M
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 2/9] M
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 3/9] M
Reviewed-by: Zhiguang Liu
-Original Message-
From: devel@edk2.groups.io On Behalf Of Abner Chang
Sent: Friday, April 10, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: abner.ch...@hpe.com; Gilbert Chen ; Leif Lindholm
; Kinney, Michael D ;
Gao, Liming
Subject: [edk2-devel] [PATCH v1 1/9] M
About the "AtomicOp", I want to retract about the check for the Routing
Capability to set the platform policy about the blocking the AtomicOp requests.
The implementation seems good.
Thanks
Ashraf
> -Original Message-
> From: Javeed, Ashraf
> Sent: Tuesday, April 21, 2020 11:33 AM
> To:
About the "CompletionTimeout ", I want to retract about the AUTO option; the
implementation is good and device initialization should be skipped for this
option.
Regards
Ashraf
> -Original Message-
> From: Javeed, Ashraf
> Sent: Monday, April 20, 2020 6:53 PM
> To: Ni, Ray ; devel@edk2.
BEGIN:VCALENDAR
VERSION:2.0
PRODID:-//Groups.io Inc//Groups.io Calendar//EN
METHOD:CANCELLED
CALSCALE:GREGORIAN
BEGIN:VEVENT
UID:mlda.1580078539586725120.r...@groups.io
DTSTAMP:20200421T060158Z
ORGANIZER;CN=Brian Richardson:mailto:brian.richard...@intel.com
DTSTART:20200422T013000Z
DTEND:20200422T0
Good catch Zhiquang,
Do you have other comments on those patches? I can fix it at once and send the
patch to ML.
Thanks
Abner
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Zhiguang Liu
> Sent: Tuesday, April 21, 2020 1:28 PM
> To: devel@ed
Hi Abner
Why this line doesn't specify the architect?
+DEFINE GCC5_ASM_FLAGS = DEF(GCC5_RISCV_ALL_ASM_FLAGS)
-march=DEF(GCC5_RISCV64_ARCH) -mcmodel=medany -mabi=lp64
I think it should be GCC5_ RISCV_ASM_FLAGS, or it may affect other architect.
Thanks
Zhiguang
-Origina
Hi Laszlo,
I think it should be Ok.
But if can get ACK/R-B from the maintainer of MdeModulePkg, that would be
better, which follows process more.
Thanks,
Dandan
> -Original Message-
> From: Laszlo Ersek [mailto:ler...@redhat.com]
> Sent: Monday, April 20, 2020 8:15 PM
> To: Bi, Dandan
Even though submodule approach solves the problem after detaching BaseTools
from edk2, I think maintaining BaseTools with PIP is best possible approach.
Using PIP and versioning helps to easy environment setup for users over
submodule with TAG approach.
Moving all non-python based tools to p
Install script currently attempts to run EdkRepo
as root, which fails. The fix ensures that EdkRepo
is run under a user account.
Signed-off-by: Nate DeSimone
Cc: Ashley E Desimone
Cc: Puja Pandya
Cc: Erik Bjorge
Cc: Bret Barkelew
Cc: Prince Agyeman
---
edkrepo_installer/linux-scripts/instal
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Ashley E Desimone
Sent: Monday, April 20, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: Desimone, Nathaniel L ; Pandya, Puja
; Bjorge, Erik C ; Bret
Barkelew ; Agyeman, Prince
Subject: [edk2-devel] [edk
when Fv not include any FFS, will encounter not enough space to add issue.
Change-Id: Ie401f0edcf3f9b6bcae706a877b284e840f61fe9
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FirmwareModuleManagement.c | 12
1 file changed, 12 insertions(+)
dif
I noticed a few things wrong with the v2 series:
o BhyveFwCtlLibNull is no longer used, and could be deleted.
o Some changes from the v1 AcpiPlatformDxe patch were left in OvmfPkg.
o BhyvePkg/License.txt still referred to OvmfPkg. I've updated it so
everything under BhyvePkg should be BSD-2-Clau
Make a copy of OvmfPkg/PlatformPei under BhyvePkg with the changes
that are needed to support the bhyve hypervisor.
Signed-off-by: Rebecca Cran
---
BhyvePkg/PlatformPei/AmdSev.c | 106 +
BhyvePkg/PlatformPei/ClearCache.c | 111 +
BhyvePkg/PlatformPei/Cmos.c | 58 ++
On bhyve, the ACPI timer is located at a fixed IO address; it need
not be programmed into, nor fetched from, the PMBA -- power
management base address -- register of the PCI host bridge.
Signed-off-by: Rebecca Cran
Reviewed-by: Laszlo Ersek
---
OvmfPkg/Include/IndustryStandard/Bhyve.h | 16
Add a null implementation library for QemuFwCfgLib, in order to
support building PciHostBridgeLib for bhyve.
Signed-off-by: Rebecca Cran
---
.../Library/QemuFwCfgLib/QemuFwCfgLibNull.inf | 37
OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgNull.c | 209 ++
2 files changed, 246 inse
bhyve uses the older VESA BIOS Extensions 2.0, so add the mode info
structure to IndustryStandard/LegacyVgaBios.h
Signed-off-by: Rebecca Cran
Acked-by: Laszlo Ersek
---
.../Include/IndustryStandard/LegacyVgaBios.h | 38 +++
1 file changed, 38 insertions(+)
diff --git a/OvmfPkg
Make a copy of OvmfPkg/AcpiPlatformDxe under BhyvePkg, with the changes
needed to support the bhyve hypervisor.
Signed-off-by: Rebecca Cran
---
BhyvePkg/AcpiPlatformDxe/AcpiPlatform.c | 250 +++
BhyvePkg/AcpiPlatformDxe/AcpiPlatform.h | 76 ++
BhyvePkg/AcpiPlatform
PATCH V3 sent.
> -Original Message-
> From: Gao, Liming [mailto:liming@intel.com]
> Sent: Tuesday, April 21, 2020 8:57 AM
> To: devel@edk2.groups.io; Chang, Abner (HPS SW/FW Technologist)
>
> Subject: RE: [edk2-devel] [PATCH v1 0/1] BaseTools: BaseTools changes for
> RISC-V platform.
Python code changes for building EDK2 RISC-V platform.
Signed-off-by: Abner Chang
Co-authored-by: Gilbert Chen
Co-authored-by: Daniel Helmut Schaefer
Reviewed-by: Leif Lindholm
Cc: Bob Feng
Cc: Liming Gao
Cc: Leif Lindholm
Cc: Gilbert Chen
---
BaseTools/Source/Python/Common/DataType.py
BZ for entire RISC-V edk2 port,
https://bugzilla.tianocore.org/show_bug.cgi?id=2672
These commits are verified by below PR,
https://github.com/tianocore/edk2/pull/512
Abner Chang (3):
BaseTools: BaseTools changes for RISC-V platform.
BaseTools: BaseTools changes for RISC-V platform.
BaseToo
C code changes for building EDK2 RISC-V platform.
Signed-off-by: Abner Chang
Co-authored-by: Gilbert Chen
Co-authored-by: Daniel Helmut Schaefer
Reviewed-by: Leif Lindholm
Cc: Bob Feng
Cc: Liming Gao
Cc: Leif Lindholm
Cc: Gilbert Chen
---
BaseTools/Source/C/Common/PeCoffLib.h |
Tools definitions template file changes for building EDK2 RISC-V platform.
Signed-off-by: Abner Chang
Co-authored-by: Gilbert Chen
Co-authored-by: Daniel Helmut Schaefer
Reviewed-by: Leif Lindholm
Cc: Bob Feng
Cc: Liming Gao
Cc: Leif Lindholm
Cc: Gilbert Chen
---
BaseTools/Conf/tools_def
Hi Laszlo,
I'm sad when saw the message. but I still appreciate the advice provided by you.
I think that this is not the only documentation issue, and should discuss in
other topic.
Hi Jian, Chao,
Just for this issue, please give some feedback.
Best Regards
Guomin
> -Original Message
Many changes from the first patch series, hopefully incorporating all
feedback I received. Changes include switching to the SPDX license
headers, updating Maintainers.txt, moving PlatformPei and
AcpiPlatformDxe under BhyvePkg, changing FILE_GUIDs to be unique
(except AcpiTables, which needs to
On bhyve, the ACPI timer is located at a fixed IO address; it need
not be programmed into, nor fetched from, the PMBA -- power
management base address -- register of the PCI host bridge.
Signed-off-by: Rebecca Cran
Reviewed-by: Laszlo Ersek
---
OvmfPkg/Include/IndustryStandard/Bhyve.h | 16
Add a null implementation of BhyveFwCtlLib, to support building
AcpiPlatformDxe for Qemu/Xen.
Signed-off-by: Rebecca Cran
---
.../BhyveFwCtlLibNull/BhyveFwCtlLibNull.c | 53 +++
.../BhyveFwCtlLibNull/BhyveFwCtlLibNull.inf | 39 ++
2 files changed, 92 insertions(
Make a copy of OvmfPkg/PlatformPei under BhyvePkg with the changes
that are needed to support the bhyve hypervisor.
Signed-off-by: Rebecca Cran
---
BhyvePkg/PlatformPei/AmdSev.c | 106 +
BhyvePkg/PlatformPei/ClearCache.c | 111 +
BhyvePkg/PlatformPei/Cmos.c | 58 ++
Make a copy of OvmfPkg/AcpiPlatformDxe under BhyvePkg, with the changes
needed to support the bhyve hypervisor.
Signed-off-by: Rebecca Cran
---
BhyvePkg/AcpiPlatformDxe/AcpiPlatform.c | 250 +++
BhyvePkg/AcpiPlatformDxe/AcpiPlatform.h | 76 ++
BhyvePkg/AcpiPlatform
On Windows, building BhyveFwCtlLib using VS2019 causes a call to
memcpy to be generated. Work around this by adding IntrinsicsLib to
resolve the reference.
Signed-off-by: Rebecca Cran
---
BhyvePkg/Library/BhyveFwCtlLib/BhyveFwCtlLib.inf | 1 +
1 file changed, 1 insertion(+)
diff --git a/BhyvePk
bhyve uses the older VESA BIOS Extensions 2.0, so add the mode info
structure to IndustryStandard/LegacyVgaBios.h
Signed-off-by: Rebecca Cran
Acked-by: Laszlo Ersek
---
.../Include/IndustryStandard/LegacyVgaBios.h | 38 +++
1 file changed, 38 insertions(+)
diff --git a/OvmfPkg
Add a null implementation library for QemuFwCfgLib, in order to
support bhyve.
Signed-off-by: Rebecca Cran
---
.../Library/QemuFwCfgLib/QemuFwCfgLibNull.inf | 37
OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgNull.c | 209 ++
2 files changed, 246 insertions(+)
create mode 100644
*Reminder:* TianoCore Bug Triage - APAC / NAMO
*When:* Wednesday, 22 April 2020, 9:30am to 10:30am, (GMT+08:00) Asia/Chongqing
*Where:* https://zoom.com.cn/j/493235016
View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=758253 )
*Organizer:* Brian Richardson brian.richard...@intel.com
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Liming Gao
> Sent: Tuesday, April 21, 2020 8:55 AM
> To: devel@edk2.groups.io; Chang, Abner (HPS SW/FW Technologist)
>
> Subject: Re: [edk2-devel] [PATCH v1 0/9] MdePkg changes for RISC-V edk2
Reviewed-by: Liming Gao
-Original Message-
From: Chiu, Chasel
Sent: 2020年4月21日 8:53
To: devel@edk2.groups.io
Cc: Desimone, Nathaniel L ; Gao, Liming
Subject: [edk2-platforms: PATCH] Revert "MinPlatformPkg/PlatformInitPei:
ReportCpuHobLib."
The patch broke all OpenBoardPkg build so r
For this change, I suggest to separate it to three patches. One is for
tools_def.template, another is for Build Python, last one is for BaseTools C
tool.
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chang, Abner
(HPS SW/FW Technologist)
Sent: 2020年4月10日 15:47
To: devel@e
Abner:
Here is my minor comments.
1. For RISC-V processor related definitions, have they been defined in public
definitions?
2. For MdePkg/BaseSynchronizationLib, why name file SynchronizationAsm.S? Can
it name to Synchronization.S?
Thanks
Liming
-Original Message-
From: devel@edk2
The patch broke all OpenBoardPkg build so reverted:
944af47a8c83115dcd5dae581e5c3770a12a2ed2.
Cc: Nate DeSimone
Cc: Liming Gao
Signed-off-by: Chasel Chiu
---
Platform/Intel/MinPlatformPkg/PlatformInit/Library/ReportCpuHobLib/ReportCpuHobLib.c
| 32
Platfor
Reviewed-by: Liming Gao for this patch set.
-Original Message-
From: Michael Kubacki
Sent: 2020年4月16日 1:10
To: devel@edk2.groups.io; Zhang, Chao B ; Bi, Dandan
; Wu, Hao A ; Wang, Jian J
; Gao, Liming
Cc: Kinney, Michael D ; Ni, Ray ;
Sean Brogan
Subject: Re: [PATCH v2 0/7] Add li
Reviewed-by: Erik Bjorge
-Original Message-
From: devel@edk2.groups.io On Behalf Of Ashley E Desimone
Sent: Monday, April 20, 2020 3:21 PM
To: devel@edk2.groups.io
Cc: Desimone, Nathaniel L ; Pandya, Puja
; Bjorge, Erik C ; Bret
Barkelew ; Agyeman, Prince
Subject: [edk2-devel] [edk2-
It has been merged. https://github.com/tianocore/edk2/pull/521
-Original Message-
From: devel@edk2.groups.io On Behalf Of PierreGondois
Sent: 2020年4月21日 2:27
To: devel@edk2.groups.io; Gao, Liming ; Liu, Zhiguang
Cc: Kinney, Michael D ; Sami Mujawar
; nd
Subject: Re: [edk2-devel] [PATC
Remote the 'delimeters='=' from the instantation of
config parser objects in the BaseConfig class and update
the manifest-repos section of the edkrepo.cfg to remove the
'=' from after the manifest repo name.
Signed-off-by: Ashley E Desimone
Cc: Nate DeSimone
Cc: Puja Pandya
Cc: Erik Bjorge
Cc:
The default build of the shell does not include AcpiView.
However, there is a second version of the shell built from
ShellPkg.dsc that does include the AcpiView command.
#
# Build a second version of the shell with all commands integrated
#
ShellPkg/Application/Shell/Shell.inf {
I believe the reason that you don't see ACPIVIEW in any profile is largely
because the SMBIOSVIEW utility was around at the time of the creation of the
Shell specification and I don't believe the ACPIVIEW one was.
Thanks,
Mike Rothman
(迈克 罗斯曼 / माइकल रोथ्मेन् / Михаил Ротман / משה רוטמן)
רועה ע
smbiosview (and other standard commands) are defined in the UEFI Shell
specification, which specifies the profiles/for each command (UEFI Shell Spec
2.2, Table 11). For instance, smbiosview is defined as part of the Debug1
profile. This is why the code registers that command in
UefiShellDebug1C
I noticed there's an acpiview command, in UefiShellAcpiViewCommandLib,
but unlike similar commands like smbiosview it's not included in any of
the shell profiles. Is that deliberate?
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570
Update pytools configuration to enable EmulatorPkg, ArmVirtPkg,
and OvmfPkg.
Update documentation for the status of those packages.
Remove future work items that are now complete.
Cc: Sean Brogan
Cc: Bret Barkelew
Cc:
From: Michael Kubacki
The following 6 patches adds support for "Platform CI" for ArmVirtPkg,
OvmfPkg, and EmulatorPkg.
Each readme has live status and links to the builds as well as details
of how to build and run the same way the CI server will.
ArmVirtPkg:
https://github.com/spbrogan/edk2/blo
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570
Add ArmVirtPkg to Core CI matrix
Add EmulatorPkg to Core CI matrix
Add OvmfPkg to Core CI matrix
Cc: Sean Brogan
Cc: Bret Barkelew
Cc: Michael D Kinney
Cc: Liming Gao
Signed-off-by: Sean Brogan
Signed-off-by: Michael
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570
Add template for Platform CI steps for a Pytools based build.
Add README to describe the template and how to use it.
Add helpful information for working with azurepipelines, templates, and
lessons learned.
Cc: Sean Brogan
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570
Add new Azure Pipeline definitions to build and run OvmfPkg with:
* Ubuntu GCC5
* Windows VS2019
Add PyTool based build of OvmfPkg
Add extdep for managing the iasl dependency
Add OvmfPkg.ci.yaml for Core CI
Add README.
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570
Add new Azure Pipeline definitions to build and run ArmVirtPkg with:
* Ubuntu GCC5
Add PyTool based build of ArmVirtPkg
Add extdep for managing the iasl dependency
Add ArmVirtPkg.ci.yaml for Core CI
Add ReadMe.rst for st
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570
Add new Azure Pipeline definitions to build and run EmulatorPkg with:
* Ubuntu GCC5
* Windows VS2019
Add PyTool based build of EmulatorPkg
Add EmulatorPkg.ci.yaml for Core CI
Add ReadMe.rst for status, details and inst
On 04/14/20 19:38, Nikita Leshenko wrote:
> This causes the device to forget about the reply frame. We allocated the
> reply frame in EfiBootServicesData type memory, and code executing after
> ExitBootServices() is permitted to overwrite it.
>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id
On 04/14/20 19:38, Nikita Leshenko wrote:
> The controller supports up to 8 targets (Not reported by the
> controller, but based on the implementation of the virtual device),
> report them in GetNextTarget and GetNextTargetLun. The firmware will
> then try to communicate with them and create a bloc
Hello everyone,
Just to push the patch forward a bit: the patch has received reviewed-by from
Liming Gao, Zhiguang Liu and Sami Mujawar,
Regards,
Pierre
-Original Message-
From: devel@edk2.groups.io On Behalf Of Liming Gao via
groups.io
Sent: Friday, April 10, 2020 4:19 AM
To: Liu, Zhi
Hello Bob, Michael and Liming,
I don't have a clear idea on which direction to go in order to enable building
ASL files before C files in edk2. I would need an answer to agree on which
direction I should continue to investigate. For the record, the matching patch
in edk2 is available at https://
Laszlo,
PIP vs Submodules:
The issue with submodule and not using python packages (pip is really just a
super convenient helper to install that consistently) is that it requires the
paradigm of file system python path management. Think of this as edk1 vs edk2
public includes (if you remember b
On 04/14/20 19:38, Nikita Leshenko wrote:
> Machines should be able to boot after this commit. Tested with different
> Linux distributions (Ubuntu, CentOS) and different Windows
> versions (Windows 7, Windows 10, Server 2016).
>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2390
> Signed-o
Agreed. I’d like to see edk2 Core, edk2 Optional, and edk2 Platform. I think
that’s an appropriate division that helps address some of the placement and
dependency concerns a number of us (not just MS 😊) have identified.
- Bret
From: devel@edk2.groups.io on beha
I support somewhere other than edk2 core.
I would offer a third option to think about; create a new tianocore repo. For
optional features or features that don't have broad community adoption it would
allow for those downstream consumers to easily pick and chose their
consumption. It also help
Hi Samer,
> -Original Message-
> From: Samer El-Haj-Mahmoud
> Sent: Sunday, April 19, 2020 5:31 AM
> To: devel@edk2.groups.io; vladimir.olovyanni...@broadcom.com; Ray Ni
> ; Zhichao Gao
> Cc: nd
> Subject: RE: [edk2-devel] [PATCH 1/1] ShellPkg/DynamicCommand: add
> HttpDynamicCommand
>
>
That is not the same use case. You are always including
NetworkPkg/Network.fdf.inc and then inside of it you are using NETWORK_ENABLE
flag to include the associated drivers or not include anything at all but the
file exists. My use case differs where the file may not exist and that is why I
hav
Introduce XenDebugLibIoPort which is enabled with
DEBUG_ON_HYPERVISOR_CONSOLE which send the debug output to Xen's
console.
It's a copy PlatformDebugLibIoPort which always write to the IO port.
It works with both Xen HVM guest and Xen PVH guest whereas the default
PlatformDebugLibIoPort works only
Remove support for DEBUG_ON_SERIAL_PORT because OvmfXen can't be build
with it due to a circular dependency:
DebugLib: BaseDebugLibSerialPort ->
SerialPortLib : XenConsoleSerialPortLib ->
XenHypercallLib : XenHypercallLib ->
DebugLib
Also, if that dependency is fixed, I think it
Patch series available in this git branch:
git://xenbits.xen.org/people/aperard/ovmf.git br.ovmfxen-debug-io-v1
Remove non working DEBUG_ON_SERIAL_PORT, then add a new LibIoPort which always
writes to the IO port.
Issues was discovered and discuss in this mail threads:
"OvmfPkg XenPkg: X
Hi edk2 community members,
Per to the short discussion with Mike and Ray, we would like to know your
opinions regard to which edk2 repo to accommodate RedfishPkg, edk2/RedfshPkg
or edk2-platforms/Features/RedfishPkg?
The current direction of edk2 open source is going to reduce the size and some
Hi,
> The proper way to enable ramfb resolution change again is adding sanity
> checks for ramfb resolution / pointer / etc. on the QEMU side.
Pointer *is* checked. ramfb creates a mapping, and if that fails due to
the pointer not being valid it bails out.
Sanity-checking the resolution is th
On 2020.04.20 13:43, Ard Biesheuvel wrote:
On 4/20/20 1:55 PM, Pete Batard wrote:
On 2020.04.20 07:45, Ard Biesheuvel wrote:
...
I rejected ACPI 6.3 table upgrades because, in their current form,
the only thing they achieve is losing the ability to boot an OS that
predates ACPI 6.3. Every pie
Hi,
> So I would say that the symptom you see is a QEMU v4.1.0 regression. The
> QemuRamfbGraphicsOutputSetMode() function in the OVMF ramfb driver
> certainly needs the QemuFwCfgWriteBytes() call to work, for changing the
> resolution.
Oh? QemuRamfbGraphicsOutputSetMode() can be called multip
On 04/14/20 19:38, Nikita Leshenko wrote:
> diff --git a/OvmfPkg/Include/IndustryStandard/FusionMptScsi.h
> b/OvmfPkg/Include/IndustryStandard/FusionMptScsi.h
Some notes related to edk2's CamelCase style:
> +#pragma pack (1)
> +typedef struct {
> + UINT8 WhoInit;
> + UINT8 Reserved1;
Thanks Ard!
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Ard
> Biesheuvel via groups.io
> Sent: Monday, April 20, 2020 9:25 AM
> To: Samer El-Haj-Mahmoud ;
> devel@edk2.groups.io
> Cc: Leif Lindholm ; Pete Batard ; Andrei
> Warkentin (awarken...@vmware.com)
> Subject: R
On 4/19/20 3:04 PM, Samer El-Haj-Mahmoud wrote:
Fix an ASSERT with the TFTP dynamic Shell command on the
RPi3 and RPi4 when running DEBUG builds. Also, enable the
command by default for all builds.
Cc: Leif Lindholm
Cc: Ard Biesheuvel
Cc: Pete Batard
Cc: Andrei Warkentin
Samer El-Haj-Mahmou
Ray,
Sorry for late response as I got into other priorities. Let's try to finish
this ASAP.
Certainly the implementation of yours has reduced the code size in great deal
by deprecating the EFI encodings for the PCIe features, and using the actual HW
values along with the global definition of AUT
BZ for entire RISC-V edk2 port,
https://bugzilla.tianocore.org/show_bug.cgi?id=2672
These commits are verified by below PR,
https://github.com/tianocore/edk2/pull/512
Abner Chang (1):
BaseTools: BaseTools changes for RISC-V platform.
BaseTools/Source/C/Common/PeCoffLib.h | 9 +
Base
BaseTools changes for building EDK2 RISC-V platform.
The changes made to build_rule.template is to avoid build errors
caused by GCC5 tool chain.
Signed-off-by: Abner Chang
Co-authored-by: Gilbert Chen
Co-authored-by: Daniel Helmut Schaefer
Reviewed-by: Leif Lindholm
Cc: Bob Feng
Cc: Liming G
I have no objection to dropping the "enable TFTP by default". I already stated
that to Ard in my original reply. That is not really a big issue (or an issue
at all) since we have the -D build option to enable the command in some builds,
and the net is no functionality change for end-users.
In g
On 04/17/20 03:40, Ni, Ray wrote:
> From: Matthew Carlson mailto:mac...@microsoft.com>>
> Sent: Wednesday, April 15, 2020 4:42 AM
> To: Ni, Ray mailto:ray...@intel.com>>; Sean Brogan
> mailto:sean.bro...@microsoft.com>>
> Subject: Discussion: Basetools a separate repo
>
> Hello Ray,
>
> I sent
On 4/20/20 1:55 PM, Pete Batard wrote:
On 2020.04.20 07:45, Ard Biesheuvel wrote:
...
I rejected ACPI 6.3 table upgrades because, in their current form, the
only thing they achieve is losing the ability to boot an OS that
predates ACPI 6.3. Every piece of the platform currently being
describe
On Wed, Apr 15, 2020 at 04:04:35PM +0200, Laszlo Ersek wrote:
> On 04/14/20 20:01, Rebecca Cran wrote:
> > I was trying to build OvmfPkg/XenPkg -a X64 -t GCC5 -b DEBUG
> > -DDEBUG_ON_SERIAL_PORT=TRUE, but the build fails. Both plain DEBUG and
> > RELEASE builds without trying to put the debug outpu
Dandan,
On 04/18/20 17:14, Samer El-Haj-Mahmoud wrote:
> Jian, Hao,
>
> Can you please push this simple patch? It has been reviewed a while ago and
> ready to be committed.
should we consider your R-b on the first patch sufficient for pushing?
You are marked as "R" for "MdeModulePkg: ACPI modu
On 04/20/20 07:25, Ashish Singhal wrote:
> I am trying to change my top-level FDF file layout by conditionally including
> an FDF depending on a build time define being passed or not.
> Any help getting around this
Try including unconditionally, and pushing down the !if into the includee.
This
On 04/16/20 18:00, Nikita Leshenko wrote:
>
>
>> On 16 Apr 2020, at 12:53, Laszlo Ersek wrote:
>>
>> On 04/14/20 19:38, Nikita Leshenko wrote:
[...]
> I break out the inner structs into separate typedefs, wrap all of them in
> #pragma pack (1) / pack () and then create the wrapping aligning un
On 2020.04.20 07:45, Ard Biesheuvel wrote:
On 4/20/20 4:41 AM, awarken...@vmware.com wrote:
Hi Andrew,
This particular issue revolves around enabling a feature by default in
the DSC for Pi 3/4 platforms - the TFTP Shell command. Clearly,
enabling this for Pi bears no effect on other Tiano pla
On 04/16/20 09:33, Guomin Jiang wrote:
> File paramter should be optional according to the descritpion.
>
> Guomin Jiang (6):
> SecurityPkg/TPM: measure UEFI images without associated device paths
> again
> SecurityPkg/DxeImageAuth: Mark the File parameter as option
> SecurityPkg/DxeImag
On 04/16/20 09:33, Guomin Jiang wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2652
>
> According to the File description, the File is optional and can be NULL.
>
> Signed-off-by: Guomin Jiang
> Cc: Jian J Wang
> Cc: Hao A Wu
> Cc: Dandan Bi
> Cc: Liming Gao
> ---
> MdeModuleP
On 04/16/20 09:33, Guomin Jiang wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2652
>
> According to the description, the File is OPTIONAL and can be NULL.
>
> Signed-off-by: Guomin Jiang
> Cc: Michael D Kinney
> Cc: Liming Gao
> ---
> MdePkg/Include/Protocol/Security2.h | 2 +-
Its been observed that in MenuManagerMenuApp when user
selects a different BootOption using Up/Down key, the
current Cursor position is not chaning.
Still points to the old BootOption.
This changes first dispalys/redraws the old BootOption
followed by new BootOption. Doing so will make current
cur
I'm glad we can reach a consensus that ramfb needs sanity checks. And well,
I'm probably at fault with the hijacking.
Your QEMU/TCG in QEMU/TCG example also made me realize a deeper problem,
though: your setting still can't escape the host display / physical GPU
issue. The middle display layers be
Removes MmDriverDispatchHandler() declaration.
All MM drivers are dispatched at init time, hence
implemention of MmDriverDispatchHandler() removed.
Signed-off-by: Abdul Lateef Attar
---
StandaloneMmPkg/Core/StandaloneMmCore.h | 22 --
1 file changed, 22 deletions(-)
diff --
While testing this i found that the Uefi Variable module is loaded much
later after AmdSevDxe module is loaded, hence the SetVariable() runtime
service is available much later, so now i am setting this new
Uefi variable as part of AmdIoMmuDxe module IoMmuMap() call.
Thanks,
Ashish
On Fri, Apr 17
This is with reference to adding SEV Live migration support in OVMF,
which basically is adding a hypercall as part of the
MemEncryptSevClearPageEncMask() function.
We unconditionally make a hypercall and if the hypercall succeeds
then it can sets a new UEFI environment variable
"SevLiveMigrati
1 - 100 of 116 matches
Mail list logo
