This new patch adds memory pages initialization( ZeroMem ) based on patch v1.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#81315): https://edk2.groups.io/g/devel/message/81315
Mute This Topic: https://groups.io/mt/85967849/21656
Group O
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3634
The memory allocated through "PeiAllocatePool" is located in HOB, and
in DXE phase, the HOB will be migrated to a different location.
After the migration, the data stored in the HOB stays the same, but the
address of pointer to the memory(su
If an EFI application frequently repeats SetTime and GetTime,
the I2C bus can be busy and failed to start. To fix this issue,
add waiting loop for the bus busy status. (Usually, it is
enough to read 3 times for checking, but for safety this
sets 10 for timeout.)
This also clean up the code path a
For OvmfPkg change, Acked-by: Jiewen Yao
Need ARM/RISC-V expert to give Reviewed-by.
Thank you
Yao Jiewen
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Abner
> Chang
> Sent: Thursday, September 30, 2021 8:46 AM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel ; Leif Lind
Thanks Liming for the review.
Help push the patches if no other feedback is received. 😊
Star
-Original Message-
From: devel@edk2.groups.io On Behalf Of gaoliming
Sent: 2021年9月28日 13:31
To: devel@edk2.groups.io; Zeng, Star
Subject: 回复: [edk2-devel] [PATCH V2 0/2] Add ProcessorUpgradeSo
Relocate FdtPciHostBridgeLib to OvmfPkg/Fdt, this library is
leverage by both ARM and RISC-V archs. Also use
PcdPciMmio32Translation and PcdPciMmio64Translation
PCDs provided by MdePkg instead of ArmPkg.
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Jiewe
PcdPciMmio32Translation and PcdPciMmio64Translation PCDs are added
to MdePkg as the common PCDs for ARM and RSIC-V archs.
The one under ArmPkg is removed in the next patch.
Signed-off-by: Abner Chang
Cc: Michael D Kinney
Cc: Liming Gao
Cc: Zhiguang Liu
Cc: Ard Biesheuvel
Cc: Leif Lindholm
C
Signed-off-by: Abner Chang
Cc: Leif Lindholm
Cc: Gerd Hoffmann
Cc: Daniel Schaefer
Cc: Sunil V L
Reviewed-by: Daniel Schaefer
---
OvmfPkg/Fdt/FdtPciHostBridgeLib/FdtPciHostBridgeLib.inf | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/OvmfPkg/Fdt/FdtPciHostBridgeLib/FdtPci
Relocate VirtioFdtDxe to OvmfPkg/Fdt, this driver is leverage by
both ARM and RISC-V archs.
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Jiewen Yao
Cc: Jordan Justen
Cc: Gerd Hoffmann
Cc: Daniel Schaefer
Cc: Sunil V L
Reviewed-by: Daniel Schaefer
-
In V3: Address comments on V2.
In V2: Remove HPE license on the files that just moved around or
the changes in the file are just code removal.
edk2 BZ #: 3665
edk2 platform corresponding changes will be submitted after
this pactch set is reviewed.
This pacthes set is to migrate some module
This PCD is moved from ArmPkg that is used to set the base address
of PCI MMIO window that provides I/O access. We relocate this PCD
because this PCD is common to ARM and RSIC-V arch.
Signed-off-by: Abner Chang
Cc: Michael D Kinney
Cc: Liming Gao
Cc: Zhiguang Liu
Cc: Ard Biesheuvel
Cc: Leif L
Relocate PciPcdProducerLib to OvmfPkg/Fdt, this library is
leverage by both ARM and RISC-V archs.
Add OvmfPkg/Fdt maintainers in Maintainers.txt
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Jiewen Yao
Cc: Jordan Justen
Cc: Gerd Hoffmann
Cc: Daniel Sc
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Jiewen Yao
Cc: Jordan Justen
Cc: Gerd Hoffmann
Cc: Daniel Schaefer
Cc: Sunil V L
---
OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMmio.inf | 2 +-
OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMmio.c | 7 ---
2 files changed, 5 insertions(+), 4
Relocate HighMemDxe to OvmfPkg/Fdt, this library is leverage by
both ARM and RISC-V archs.
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Jiewen Yao
Cc: Jordan Justen
Cc: Gerd Hoffmann
Cc: Daniel Schaefer
Cc: Sunil V L
---
ArmVirtPkg/ArmVirtCloudHv.d
Relocate QemuFwCfgLib to OvmfPkg/Library/QemuFwCfgLib and rename
it to QemuFwCfgLibMmio, this library is leverage by both ARM and
RISC-V archs.
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Jiewen Yao
Cc: Jordan Justen
Cc: Gerd Hoffmann
Cc: Daniel Scha
PcdPciIoTranslation PCD is relocated to MdePkg and leveraged by
both ARM and RISC-V arch. This patch removes the one from ArmPkg
and address the corresponding changes required for other modules
under ArmVirtPkg.
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
C
This is one of the series patches to restructure the location of modules under
ArmVirtPkg for RiscVVirtPkg. RiscVVirtPkg leverage FDT Client protocol to
parse FDT nodes.
Signed-off-by: Abner Chang
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Sami Mujawar
Cc: Gerd Hoffmann
Cc: Daniel Schaefer
Cc:
Add RISC-V VM in the file header.
Add RISC-V to the supported arch.
Signed-off-by: Abner Chang
Cc: Leif Lindholm
Cc: Gerd Hoffmann
Cc: Daniel Schaefer
Cc: Sunil V L
---
OvmfPkg/Fdt/HighMemDxe/HighMemDxe.inf | 3 ++-
OvmfPkg/Fdt/HighMemDxe/HighMemDxe.c | 3 ++-
2 files changed, 4 insertions
Dun:
I understand this test is to make sure Edk2 change doesn't break
WhiskeylakeOpenBoard build. It is still for edk2 change.
Thanks
Liming
> -邮件原件-
> 发件人: devel@edk2.groups.io 代表 duntan
> 发送时间: 2021年9月30日 9:15
> 收件人: Ni, Ray ; devel@edk2.groups.io
> 抄送: Sean Brogan ; Bret Barkelew
>
Hi Ray,
The CI is in edk2. The edk2 pull request which contains changes in specified
folders of edk2 will trigger the ci to pull the latest code of edk2-platforms,
intel/FSP and edk2-non-osi and build WhiskeyLakeOpenBoard.
Thanks,
Dun
-Original Message-
From: Ni, Ray
Sent: Wednesday,
Reviewed-by: Ray Ni
> -Original Message-
> From: Dong, Guo
> Sent: Sunday, September 26, 2021 7:05 AM
> To: devel@edk2.groups.io
> Cc: Ni, Ray ; Ma, Maurice ; You,
> Benjamin ; Dong, Guo
>
> Subject: [`edk2-devel][PATCH 0/8] Add SMM variable support for UEFI payload
>
> From: Guo Dong
Yes. Merged.
499c4608b13df0738213329722cdcb0ac838f3ad..
bd298d75934a7a0cf6b9e2863a5e4975e9714285
> -Original Message-
> From: Stefan Berger
> Sent: Thursday, September 30, 2021 3:15 AM
> To: devel@edk2.groups.io; Yao, Jiewen
> Cc: mhaeu...@posteo.de; spbro...@outlook.com;
> marcandre.lu
Hi ChengChieh,
You could directly build from this branch for the test.
https://github.com/gdong1/edk2/pull/new/for_linux_bootloader_test
Thanks,
Guo
-Original Message-
From: Dong, Guo
Sent: Wednesday, September 29, 2021 12:02 PM
To: devel@edk2.groups.io; chengch...@google.com
Cc: Ni, R
On 9/14/21 6:26 PM, Yao, Jiewen wrote:
Reviewed-by: Jiewen Yao
I will wait for a week, to see if there is any feedback from AMD or Bhyve
reviewer.
I didn't see any more comments to v3
(https://listman.redhat.com/archives/edk2-devel-archive/2021-September/msg00641.html)
So maybe you can a
Hi Ray,
I had updated the patch to update the commit message and fix some coding style.
Please help review V4 patch.
Thanks,
Guo
-Original Message-
From: Ni, Ray
Sent: Tuesday, August 24, 2021 5:35 PM
To: Cheng-Chieh Huang ; devel@edk2.groups.io
Cc: Schaefer, Daniel ; Trammell Hudson
From: Cheng-Chieh Huang
V4: Update commit message and coding style.
Used Linux kernel as a bootloader to boot linux.
To boot windows, UEFI service is required for the bootloader.
So update UEFI payload to support Linux bootloader in order to
boot UEFI OS.
Linux bootloader will provide platfrom i
Hi ChengChieh,
Here is the update to this patch series:
1. UefiPayloadPkg: Add LINUXBOOT payload target
a) Ray asked to update commit message
b) There are still some coding style issues
I know it is a little difficult to fully follow EDKII coding style. I will
help update this
Hi again,
Replies inline,
>
> Hi Joey,
> Thanks for the review, I answered inline:
>
>
> On 9/24/21 9:56 AM, Joey Gouly wrote:
> > Hi,
> >
> > This looks good to me!
> >
> > [...]
> >
> >> +
> >> +/** A parser for EArmObjFixedFeatureFlags.
> >> +*/
> >> +STATIC CONST CM_OBJ_PARSER CmArmFixedFeatu
Hi,
Can you help to explain more why PeiAllocatePages does not have the problem
while PeiAllocatePool has?
Thanks,
Eric
-Original Message-
From: Lou, Yun
Sent: Wednesday, September 29, 2021 7:53 PM
To: devel@edk2.groups.io
Cc: Lou, Yun ; Ni, Ray ; Dong, Eric
; Laszlo Ersek ; Kumar, R
Is this a patch to enable CI in edk2-platforms?
Package maintainers can directly check in changes to edk2-platforms. Then how
the CI is triggered?
Will the CI run because of a edk2 pull request?
Thanks,
Ray
> -Original Message-
> From: Tan, Dun
> Sent: Wednesday, September 29, 2021 2:
Reviewed-by: Ray Ni
> -Original Message-
> From: Dong, Guo
> Sent: Wednesday, September 29, 2021 1:19 AM
> To: devel@edk2.groups.io
> Cc: Dong, Guo ; Ni, Ray ; Ma, Maurice
> ; You, Benjamin
>
> Subject: [`edk2-devel][PATCH V2] UefiPayloadPkg: Build a HOB from bootloader
> ACPI table
>
Thanks for providing a very clear commit message.
Reviewed-by: Ray Ni
> -Original Message-
> From: Lou, Yun
> Sent: Wednesday, September 29, 2021 7:53 PM
> To: devel@edk2.groups.io
> Cc: Lou, Yun ; Ni, Ray ; Dong, Eric
> ; Laszlo Ersek
> ; Kumar, Rahul1
> Subject: [PATCH v1] UefiCpuP
Right. QemuFwCfgLibMmio - per my understanding.
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Abner
> Chang
> Sent: Wednesday, September 29, 2021 8:47 PM
> To: devel@edk2.groups.io; Yao, Jiewen ;
> sami.muja...@arm.com
> Cc: Ard Biesheuvel ; Leif Lindholm
> ; Justen, Jor
Hi Jiewen,
It should be QemuFwCfgLibMmio instead of QemuFwCfgLibMMIO, right?
Abner
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Yao, Jiewen
> Sent: Wednesday, September 29, 2021 8:26 PM
> To: devel@edk2.groups.io; sami.muja...@arm.com
> Cc
Thank you Sami.
We have a clear naming rule for EDKII project during development.
But I don’t know where it is documented. Maybe a good addition to the doc you
point out.
To summarize what I know:
1) Library name: []Lib[]
2) Driver Name:
For the example you point out, I see no problem, becaus
Create a PR https://github.com/tianocore/edk2/pull/2030
-Original Message-
From: devel@edk2.groups.io On Behalf Of Bob Feng
Sent: Wednesday, September 29, 2021 7:44 PM
To: pierre.gond...@arm.com; devel@edk2.groups.io; Liming Gao
; Sami Mujawar
Subject: Re: [edk2-devel] [PATCH v1 0/4] Se
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3634
The memory allocated through "PeiAllocatePool" is located in HOB, and
in DXE phase, the HOB will be migrated to a different location.
After the migration, the data stored in the HOB stays the same, but the
address of pointer to the memory(su
Reviewed-by: Bob Feng
-Original Message-
From: pierre.gond...@arm.com
Sent: Thursday, September 23, 2021 4:59 PM
To: devel@edk2.groups.io; Feng, Bob C ; Liming Gao
; Sami Mujawar
Subject: [PATCH v1 0/4] Set default Makefile name
From: Pierre Gondois
A Makefile name is not set in Ba
Hi Jiewen,
Thank you for clarifying the library naming convention.
I could not find any references/examples as such in
https://edk2-docs.gitbook.io/edk-ii-c-coding-standards-specification/v/release%2F2.20/4_naming_conventions/42_file_names
and therefore had suggested following the file naming as
PR 2020 failed because it has an additional commit "Merge branch 'master' into
BaseTools_Arm"
Create a new PR https://github.com/tianocore/edk2/pull/2029
-Original Message-
From: gaoliming
Sent: Wednesday, September 29, 2021 9:19 AM
To: 'Leif Lindholm' ; devel@edk2.groups.io
Cc: 'Rebe
Acked-by: Bob Feng
Sorry for the late response.
-Original Message-
From: Rebecca Cran
Sent: Monday, August 30, 2021 1:15 PM
To: Feng, Bob C ; Liming Gao ;
Chen, Christine ; Sean Brogan
; Sami Mujawar ; Leif Lindholm
; Ard Biesheuvel
Cc: Rebecca Cran ; devel@edk2.groups.io
Subject:
hi
I think the original name is correct.
The naming convention is : Lib
thank you!
Yao, Jiewen
> 在 2021年9月29日，下午5:45，Sami Mujawar 写道：
>
> Hi Abner,
>
> Thank you for this patch.
>
> I have a minor suggestion marked inline as [SAMI].
>
> Regards,
>
> Sami Mujawar
>
>
>> On 28/09/2021 09
Hi Abner,
Thank you for this patch.
I have a minor suggestion marked inline as [SAMI].
Regards,
Sami Mujawar
On 28/09/2021 09:31 AM, Abner Chang wrote:
Relocate QemuFwCfgLib to OvmfPkg/Library/QemuFwCfgLib and rename
it to QemuFwCfgLibMMIO, this library is leverage by both ARM and
RISC-V ar
Currently the realpath is used when parse modules, which shows the
path with a drive letter in build log. In Windows 'subst' comand is
used to associates a path with a drive letter, when use the mapped
drive letter for build, with realpath function the build log will
have different disk letter info
Hi,
Looks good to me...
Reviewed-by: Grzegorz Bernacki
pon., 27 wrz 2021 o 09:48 Nhi Pham via groups.io
napisał(a):
>
> The dbt and dbx keys are optional, the driver entry should return
> EFI_SUCCESS to start if they are not found in the firmware flash. This
> patch is to fix it and update the
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3666
Currently, CoreLocateHandleBuffer() follows three steps:
1) get the size of protocol database firstly
2) allocate the buffer based on the size
3) get the protocol database into the buffer
There is no lock protection for the whole three steps
46 matches
Mail list logo
