From: miaoyubo
Add support for extra roots for arm, in this patch, we
import the scan for extra root buses therefore the uefi
could recognize multiply roots for arm.
The logic keeps the same with the logic in
"OvmfPkg/Library/PciHostBridgeLib/PciHostBridgeLib.c"
Contributed-under: TianoCore
On 4/2/20 1:03 PM, Leif Lindholm wrote:
On Thu, Apr 02, 2020 at 12:29:30 +0200, Ard Biesheuvel wrote:
On 4/2/20 12:28 PM, Leif Lindholm wrote:
On Thu, Apr 02, 2020 at 12:20:54 +0200, Ard Biesheuvel wrote:
On 4/2/20 12:16 PM, Leif Lindholm via groups.io wrote:
On Sat, Mar 28, 2020 at 11:43:17
Quick review comments only, for now:
On 04/02/20 14:08, Yubo Miao wrote:
> From: miaoyubo
>
> Add support for extra roots for arm, in this patch, we
> import the scan for extra root buses therefore the uefi
> could recognize multiply roots for arm.
>
> The logic keeps the same with the logic
Dear Community Members,
Please note the time change on tonight's TianoCore Community meeting,
reflecting the US daylight savings time change as we moved clocks an hour
earlier during March.
The community meeting will start from 6.30pm-7.30pm (PST).
You can view the calendar here -
Rebecca,
Thanks for the reminder.
I have updated the available values for that field.
Mike
> -Original Message-
> From: devel@edk2.groups.io On
> Behalf Of Rebecca Cran
> Sent: Thursday, April 2, 2020 12:53 PM
> To: devel@edk2.groups.io; Kinney, Michael D
>
> Subject: Re:
Sorry, Correction.
The meeting is scheduled from 7.30pm-8.30pm (PST). Please note the time.
_
From: Guptha, Soumya K
Sent: Thursday, April 2, 2020 4:23 PM
To: 'annou...@edk2.groups.io' ; devel@edk2.groups.io
Subject: TianoCore Community Meeting tonight -
Rebecca,
The current use of that field is to identify previous
releases that a bug fix needs to be propagated to.
We do not know the exact release name for future releases
until we make the release.
I recommend you use the "Deadline" field to provide
the date the feature must be implemented,
Okay, but the edk2-stable202002 release is also missing.
--
Rebecca Cran
On 4/2/20 1:24 PM, Michael D Kinney wrote:
Rebecca,
The current use of that field is to identify previous
releases that a bug fix needs to be propagated to.
We do not know the exact release name for future releases
*Reminder:* TianoCore Design Meeting - APAC/NAMO
*When:* Friday, 3 April 2020, 9:30am to 10:30am, (GMT+08:00) Asia/Chongqing
*Where:* https://zoom.com.cn/j/971006237
View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=700770 )
*Organizer:* Ray Ni ray...@intel.com (
"TianoCore Design Meeting - APAC/NAMO"
When: Friday, 3 April 2020
9:30am to 10:30am
(GMT+08:00) Asia/Chongqing
Where: https://zoom.com.cn/j/971006237
Organizer: ray...@intel.com
Description: # TOPIC
* EDK2 Redfish Implementation Review (Abner Chang/HPE)
Slides:
Hi Guomin,
I think you missed one at line 435, function PowerButtonPhaseToString().
Regards,
Jian
From: Jiang, Guomin
Sent: Monday, March 30, 2020 9:32 AM
To: devel@edk2.groups.io; newexplor...@gmail.com; GuoMinJ via Groups.Io
Cc: Wang, Jian J ; Wu, Hao A
Subject: RE: [edk2-devel] [PATCH]
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2654
If FV is placed in FD region, its FV image size is fixed.
When FV image size exceeds it, it will trig the build break.
To alert the developer to adjust FV image size earlier,
I request to add new checker for the the spare FV space.
When the
This change is good to me.
Reviewed-by: Bob C Feng
-Original Message-
From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Michael
Kubacki
Sent: Friday, April 3, 2020 1:05 AM
To: devel@edk2.groups.io
Cc: Feng, Bob C ; Gao, Liming
Subject: [edk2-devel] [PATCH v1 1/1]
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2272
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2287
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2288
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2289
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2290
Index should
I would have much rather seen this implemented independent of the GenFds tool
as this is a nice feature but shouldn't be tightly coupled into the tool that
does building and packaging firmware.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply
Reviewed-by: Liming Gao
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Zhang, Shenglei
> Sent: Wednesday, April 1, 2020 4:25 PM
> To: devel@edk2.groups.io
> Cc: Sean Brogan ; Bret Barkelew
> ; Kinney, Michael D
> ; Gao, Liming
> Subject: [edk2-devel] [PATCH]
Reviewed-by: Liming Gao
> -Original Message-
> From: Zhang, Shenglei
> Sent: Wednesday, April 1, 2020 4:32 PM
> To: devel@edk2.groups.io
> Cc: Gao, Liming ; Wang, Jian J ;
> Wu, Hao A ; Feng, Bob C
>
> Subject: [PATCH v2 0/2] Use submodule way to access brotli
>
> Currently brotli is
Reviewed-by: Liming Gao
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Zhang, Shenglei
> Sent: Thursday, April 2, 2020 3:17 PM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J ; Wu, Hao A ;
> Gao, Liming
> Subject: [edk2-devel] [PATCH v2]
Hi,
It can be implemented independent of the GenFds tool
It accepts the parameters passed by building and GenFds Tool
Best Regards
Fan Zhiju
From: sean.brogan via []
Sent: Friday, April 3, 2020 10:51 AM
To: Fan; Fan, ZhijuX ; devel@edk2.groups.io
Subject: Re: [edk2-devel] [PATCH 1/1]
Reviewed-by: Jian J Wang
Regards,
Jian
> -Original Message-
> From: Jiang, Guomin
> Sent: Friday, April 03, 2020 12:25 PM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J ; Wu, Hao A ;
> Dong, Eric ; Ni, Ray
> Subject: [PATCH v2] MdeModulePkg/SmiHandlerProfileInfo: Overflowed Array
>
For the functionality, it is the same with before for platforms which set
PcdPropertiesTableEnable to false by default.
Reviewed-by: Dandan Bi for patch [PATCH 2/4] [PATCH 3/4]
[PATCH 4/4].
If anyone still has the use case of enabling PcdPropertiesTableEnable, please
comment.
Thanks,
Not sure I follow. The changes are in BaseTools/Source/Python/GenFds/GenFds.py
I am suggesting this functionality to be written outside the GenFds.py files so
that we don't add complexity to the critical tools used for building and
packaging FW binaries.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2272
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2287
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2288
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2289
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2290
Index should
*Reminder:* TianoCore Community Meeting - APAC/NAMO
*When:* Thursday, 2 April 2020, 7:30pm to 8:30pm, (GMT-07:00) America/Los
Angeles
*Where:* https://bluejeans.com/889357567?src=join_info
View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=700768 )
*Organizer:* Brian Richardson
Reviewed-by: Nate DeSimone
-Original Message-
From: Desimone, Ashley E
Sent: Tuesday, March 31, 2020 5:51 PM
To: devel@edk2.groups.io
Cc: Desimone, Nathaniel L ; Pandya, Puja
; Bjorge, Erik C ; Bret
Barkelew ; Agyeman, Prince
Subject: [edk2-staging/EdkRepo] [PATCH] EdkRepo: Allow
Hi Micheal Rothman,
Can you help to review this issue? Because of the uefi spec issue, some
firmware implemented the different behavior.
Should the edk2 code to handle such issue? If yes, here maybe the situation:
1. Change for all apps -> uefi spec update and accept such behavior with
*Reminder:* TianoCore Community Meeting - EMEA/NAMO
*When:* Thursday, 2 April 2020, 9:00am to 10:00am, (GMT-07:00) America/Los
Angeles
*Where:* https://bluejeans.com/889357567?src=join_info
View Event ( https://edk2.groups.io/g/devel/viewevent?eventid=700767 )
*Organizer:* Brian Richardson
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2655
This patch fixes reversed logic of recently added ASSERTs which should
ensure that Ip6IsNDOptionValid() implementation properly reacts to invalid
packets.
Cc: Jiaxin Wu
Cc: Siyuan Fu
Signed-off-by: Maciej Rabeda
Reviewed-by: Siyuan Fu
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2656
1. Add GetPackagesPath.py, it will be used to get package pathes from
special directories. A sub directory is a qualified package path
when an EDKII Package can be found under it.
2. Add SetPackagesPath.bat and SetPackagesPath.sh, these
Summarize the issue, if anything incorrect, please help to correct it. Thanks.
Background:
Uefi spec ambiguous description:
> When interpreting the data from this function, it should be noted that if a
> class
> of printable characters that are normally adjusted by shift modifiers (e.g.
> Shift
Enhance for check input FD if empty file or not.
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: Yunhua Feng
---
BaseTools/Source/C/FMMT/FirmwareModuleManagement.c | 2 +-
BaseTools/Source/C/FMMT/FmmtLib.c | 5 +
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git
Use submodule way to access oniguruma. And upgrade oniguruma
version from v6.9.3 to v6.9.4_mark1.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2073
Cc: Jian J Wang
Cc: Hao A Wu
Cc: Liming Gao
Signed-off-by: Shenglei Zhang
---
v2:1.Update the inf file to make VS2019 build pass.
I am not a fan of this. The community and design meetings have had a few
discussions about tools (PyTools has been presented twice) and a few brief
discussions about common patterns to build platforms but I don't think there is
real alignment. Each "platform" has its own way of doing things.
From: Sean Brogan
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2640
Azure Pipelines agents have 2 threads. This commit has been shown to
reduce the build time in half on those agents.
Cc: Bob C Feng
Cc: Liming Gao
Signed-off-by: Michael Kubacki
---
BaseTools/Edk2ToolsBuild.py | 16
I will fix it next time.
-Erik
-Original Message-
From: Desimone, Nathaniel L
Sent: Wednesday, April 1, 2020 10:59 PM
To: devel@edk2.groups.io; Bjorge, Erik C
Cc: Desimone, Ashley E ; Pandya, Puja
; Bret Barkelew ; Agyeman,
Prince
Subject: RE: [edk2-devel] [edk2-staging/EdkRepo]
On 4/2/20 12:23 PM, Leif Lindholm via groups.io wrote:
On Sat, Mar 28, 2020 at 11:43:21 +0100, Ard Biesheuvel wrote:
TT_ATTR_INDX_INVALID is #define'd but never used so drop it.
Signed-off-by: Ard Biesheuvel
---
ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c | 3 ---
1 file changed, 3
On 4/2/20 12:28 PM, Leif Lindholm wrote:
On Thu, Apr 02, 2020 at 12:20:54 +0200, Ard Biesheuvel wrote:
On 4/2/20 12:16 PM, Leif Lindholm via groups.io wrote:
On Sat, Mar 28, 2020 at 11:43:17 +0100, Ard Biesheuvel wrote:
Before getting rid of GetRootTranslationTableInfo() and the related
On Thu, Apr 02, 2020 at 12:29:30 +0200, Ard Biesheuvel wrote:
> On 4/2/20 12:28 PM, Leif Lindholm wrote:
> > On Thu, Apr 02, 2020 at 12:20:54 +0200, Ard Biesheuvel wrote:
> > > On 4/2/20 12:16 PM, Leif Lindholm via groups.io wrote:
> > > > On Sat, Mar 28, 2020 at 11:43:17 +0100, Ard Biesheuvel
On Sat, Mar 28, 2020 at 11:43:17 +0100, Ard Biesheuvel wrote:
> Before getting rid of GetRootTranslationTableInfo() and the related
> LookupAddresstoRootTable() in AARCH64's version of ArmMmuLib, add a
> version of the former to CpuDxe, which will be its only remaining
> user. While at it,
On 4/2/20 12:16 PM, Leif Lindholm via groups.io wrote:
On Sat, Mar 28, 2020 at 11:43:17 +0100, Ard Biesheuvel wrote:
Before getting rid of GetRootTranslationTableInfo() and the related
LookupAddresstoRootTable() in AARCH64's version of ArmMmuLib, add a
version of the former to CpuDxe, which
On Thu, Apr 02, 2020 at 12:20:54 +0200, Ard Biesheuvel wrote:
> On 4/2/20 12:16 PM, Leif Lindholm via groups.io wrote:
> > On Sat, Mar 28, 2020 at 11:43:17 +0100, Ard Biesheuvel wrote:
> > > Before getting rid of GetRootTranslationTableInfo() and the related
> > > LookupAddresstoRootTable() in
On 4/2/20 12:57 PM, Leif Lindholm wrote:
On Thu, Apr 02, 2020 at 12:29:40 +0200, Ard Biesheuvel wrote:
On 4/2/20 12:23 PM, Leif Lindholm via groups.io wrote:
On Sat, Mar 28, 2020 at 11:43:21 +0100, Ard Biesheuvel wrote:
TT_ATTR_INDX_INVALID is #define'd but never used so drop it.
On Sat, Mar 28, 2020 at 11:43:21 +0100, Ard Biesheuvel wrote:
> TT_ATTR_INDX_INVALID is #define'd but never used so drop it.
>
> Signed-off-by: Ard Biesheuvel
> ---
> ArmPkg/Library/ArmMmuLib/AArch64/ArmMmuLibCore.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git
On Thu, Apr 02, 2020 at 12:29:40 +0200, Ard Biesheuvel wrote:
> On 4/2/20 12:23 PM, Leif Lindholm via groups.io wrote:
> > On Sat, Mar 28, 2020 at 11:43:21 +0100, Ard Biesheuvel wrote:
> > > TT_ATTR_INDX_INVALID is #define'd but never used so drop it.
> > >
> > > Signed-off-by: Ard Biesheuvel
>
It would also be good to clearly document the status of all
tool chain tags in a few places:
https://github.com/tianocore/edk2/blob/master/BaseTools/Conf/tools_def.template
https://github.com/tianocore/edk2/blob/master/Readme.md
The status I can think of are:
1) Actively used by EDK II CI.
On 4/2/20 9:45 AM, devel@edk2.groups.io Calendar wrote:
*Reminder:* TianoCore Community Meeting - EMEA/NAMO
*When:* Thursday, 2 April 2020, 9:00am to 10:00am, (GMT-07:00)
America/Los Angeles
In relation to the discussion during the meeting, I've created
At least for the Feature Requests, the "Release(s) the issue is
observed" and "Release(s) the issues must be fixed" fields are outdated,
and missing newer tags.
I haven't checked if the other areas such as bug reports also need updated.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io
47 matches
Mail list logo
