Re: [edk2-devel] [PATCH] BaseTools: Add RISCV64 binding

2020-09-28 Thread Nikita
On 29 Sep 2020, at 06:15, gaoliming wrote: > > Pull request is created. https://github.com/tianocore/edk2/pull/969 > Thanks! >> -邮件原件- >> 发件人: bounce+27952+65571+4905953+8761...@groups.io >> 代表 Abner Chang >> 发送时间: 2020年9月24日 22:14 >> 收件人: devel@edk2.groups.io;

Re: [edk2-devel] more development process failure [was: UefiPayloadPkg: Runtime MMCONF]

2020-09-28 Thread Guo Dong
> On 09/26/20 02:34, Dong, Guo wrote: > > > > Sorry to have a long email thread since my merge and thanks all for > > the comments. > > In general, I still feel current process is a little complicated for > > the maintainers who don't daily work on EDK2 like me. I have less > > than %5 of time

回复: [edk2-devel] TianoCore Bug Triage - APAC / NAMO - Tue, 09/29/2020 6:30pm-7:30pm #cal-reminder

2020-09-28 Thread gaoliming
Hi, all Below BZs will be reviewed this week Bug Triage meeting. This week bug triage meeting still uses WebEx. 2989 EDK2 Code unassig...@tianocore.org UNCO In PciBusDxe\PciOptionRomSupport.c increase the width if the data read

回复: [edk2-devel] [PATCH] BaseTools: Add RISCV64 binding

2020-09-28 Thread gaoliming
Pull request is created. https://github.com/tianocore/edk2/pull/969 > -邮件原件- > 发件人: bounce+27952+65571+4905953+8761...@groups.io > 代表 Abner Chang > 发送时间: 2020年9月24日 22:14 > 收件人: devel@edk2.groups.io; sh1r...@mail.si-head.nl > 主题: Re: [edk2-devel] [PATCH] BaseTools: Add RISCV64 binding >

Re: [edk2-devel] [PATCH 1/1] BaseTools: Update license of UserManuals files

2020-09-28 Thread Yuwei Chen
Hi, Liming Agree with you. I will remove the BaseTools\UserManuals folder as developers have the way to get the help message. Thanks Yuwei > -Original Message- > From: devel@edk2.groups.io On Behalf Of > gaoliming > Sent: Monday, September 28, 2020 10:37 PM > To: devel@edk2.groups.io;

Re: [edk2-devel] [PATCH 1/1] BaseTools: Move C tool flags before the common flags

2020-09-28 Thread Yuwei Chen
Reviewed-by: Yuwei Chen > -Original Message- > From: devel@edk2.groups.io On Behalf Of > gaoliming > Sent: Wednesday, September 16, 2020 9:03 AM > To: devel@edk2.groups.io > Cc: Chen, Christine ; Feng, Bob C > > Subject: [edk2-devel] [PATCH 1/1] BaseTools: Move C tool flags before the >

回复: [edk2-devel] [Patch V2 1/1] Tools\FitGen: Add extra parameter to input the Higher Flash Address

2020-09-28 Thread gaoliming
Chinni: Can you introduce the real usage case with this enhancement? After this case, how will you use FitGen tool in your case? Thanks Liming > -邮件原件- > 发件人: bounce+27952+65439+4905953+8761...@groups.io > 代表 cbduggap > 发送时间: 2020年9月22日 18:36 > 收件人: devel@edk2.groups.io > 抄送: cbduggap

Re: [edk2-devel] [PATCH v13 1/1] ShellPkg/DynamicCommand: add HttpDynamicCommand

2020-09-28 Thread Gao, Zhichao
Hi, I have checked in openCI and it is passed. Reviewed-by: Zhichao Gao Hi Laszlo, Do you have a plan to test this patch? Thanks, Zhichao > -Original Message- > From: Vladimir Olovyannikov > Sent: Friday, September 25, 2020 4:41 AM > To: devel@edk2.groups.io > Cc: Vladimir

Re: [edk2-devel] [PATCH v3 1/1] MdeModulePkg/XhciDxe: Fix Broken Timeouts

2020-09-28 Thread Wu, Hao A
Merged via Pull request: https://github.com/tianocore/edk2/pull/968 Commit: 71dd80f14f2724ccc99dd7d349b7392adf114019 Best Regards, Hao Wu > -Original Message- > From: devel@edk2.groups.io On Behalf Of Wu, Hao > A > Sent: Thursday, September 24, 2020 9:22 AM > To: devel@edk2.groups.io;

[edk2-devel] TianoCore Bug Triage - APAC / NAMO - Tue, 09/29/2020 6:30pm-7:30pm #cal-reminder

2020-09-28 Thread devel@edk2.groups.io Calendar
*Reminder:* TianoCore Bug Triage - APAC / NAMO *When:* Tuesday, 29 September 2020, 6:30pm to 7:30pm, (GMT-07:00) America/Los Angeles *Where:* https://meetingsamer34.webex.com/meetingsamer34/j.php?MTID=m623b1dceab0c6dc62d56b2508fa50f39 View Event (

回复: [edk2-devel] [PATCH v2 1/2] MdePkg: Definitions for Extended Interrupt Flags

2020-09-28 Thread gaoliming
Reviewed-by: Liming Gao > -邮件原件- > 发件人: bounce+27952+65628+4905953+8761...@groups.io > 代表 Sami Mujawar > 发送时间: 2020年9月25日 21:18 > 收件人: devel@edk2.groups.io > 抄送: Sami Mujawar ; michael.d.kin...@intel.com; > gaolim...@byosoft.com.cn; zhiguang@intel.com; alexei.fedo...@arm.com; >

回复: [edk2-devel] [PATCH v2 0/2] UEFI memmap workaround for hiding page-access caps from OSes hides SP and CRYPTO caps too

2020-09-28 Thread gaoliming
Meg: What real problem do you meet with? What purpose is for this change? And, I also include UEFI Arch Rothman. Rothman: Can you help clarify what OS (Windows or Linux) behavior is expected for UEFI SP and CRYPTO memory attribute? Thanks Liming > -邮件原件- > 发件人:

回复: [edk2-devel] [PATCH 1/1] BaseTools: Move C tool flags before the common flags

2020-09-28 Thread gaoliming
Bob & Yuwei: Can you help review this patch? Thanks Liming > -邮件原件- > 发件人: bounce+27952+65297+4905953+8761...@groups.io > 代表 gaoliming > 发送时间: 2020年9月16日 9:03 > 收件人: devel@edk2.groups.io > 抄送: Yuwei Chen ; Bob Feng > 主题: [edk2-devel] [PATCH 1/1] BaseTools: Move C tool flags before the

回复: 回复: [edk2-devel] Tianocore community page on who we are - please review

2020-09-28 Thread gaoliming
Soumya: Do you mean to directly modify wiki page? Or, send the update content as the patch for your integration? Thanks Liming > -邮件原件- > 发件人: Guptha, Soumya K > 发送时间: 2020年9月29日 0:20 > 收件人: Leif Lindholm ; devel@edk2.groups.io; > gaolim...@byosoft.com.cn > 抄送: Yao, Jiewen ;

回复: 回复: [edk2-devel] Tianocore community page on who we are - please review

2020-09-28 Thread gaoliming
Laszlo: I agree this point. The guideline is to collect the feedback in one week, not the deadline to finish the code review. Thanks Liming > -邮件原件- > 发件人: Laszlo Ersek > 发送时间: 2020年9月29日 1:15 > 收件人: gaoliming ; devel@edk2.groups.io; > jiewen@intel.com; 'Guptha, Soumya K' ; >

Re: [edk2-devel] [edk2-staging/EdkRepo] [PATCH V3] EdkRepo: Return exit codes in edkrepo_entry_point.py

2020-09-28 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge -Original Message- From: Desimone, Nathaniel L Sent: Friday, September 25, 2020 6:35 PM To: devel@edk2.groups.io; Desimone, Ashley E Cc: Pandya, Puja ; Bret Barkelew ; Agyeman, Prince ; Bjorge, Erik C Subject: RE: [edk2-devel] [edk2-staging/EdkRepo] [PATCH

Re: [edk2-devel] [edk2-staging/EdkRepo] [PATCH V3 2/2] EdkRepo: Add support for subst drives

2020-09-28 Thread Bjorge, Erik C
Reviewed-by: Erik Bjorge -Original Message- From: Nate DeSimone Sent: Friday, September 25, 2020 6:28 PM To: devel@edk2.groups.io Cc: Desimone, Ashley E ; Pandya, Puja ; Bret Barkelew ; Agyeman, Prince ; Bjorge, Erik C Subject: [edk2-staging/EdkRepo] [PATCH V3 2/2] EdkRepo: Add

Re: [edk2-devel] [edk2-platforms] [PATCH V1] LogoFeaturePkg: Update Intel Logo

2020-09-28 Thread Nate DeSimone
Pushed: https://github.com/tianocore/edk2-platforms/commit/23f993d1913ffb421a00206d6f1b31079a1f41a1 -Original Message- From: devel@edk2.groups.io On Behalf Of Nate DeSimone Sent: Wednesday, September 2, 2020 9:55 AM To: devel@edk2.groups.io Cc: Chaganty, Rangasai V ; Bi, Dandan ;

Re: [EXTERNAL] [edk2-devel] ECC: Won't somebody PLEASE think of the... test structures.

2020-09-28 Thread Laszlo Ersek
On 09/26/20 04:52, Andrew Fish via groups.io wrote: > Bret, > > Details aside this ECC issue got me thinking it might be useful to some how > tag code that follows different, subsetted, or alternate rules, or uses > different build environments. I was kind of thinking of doing something like >

Re: [edk2-devel] who can help restore the iBFT spec link to working state?

2020-09-28 Thread Laszlo Ersek
On 09/25/20 18:02, Sean Brogan wrote: > I see what i can find. > > Thanks for pointing this out. Thank you Sean! Laszlo > On 9/25/2020 7:20 AM, Laszlo Ersek wrote: >> Hi, >> >> the iBFT spec link >> >>    http://www.microsoft.com/whdc/system/platform/firmware/ibft.mspx >> >> as shown at >> >>   

Re: [edk2-devel] more development process failure [was: UefiPayloadPkg: Runtime MMCONF]

2020-09-28 Thread Laszlo Ersek
On 09/26/20 02:34, Dong, Guo wrote: > > Sorry to have a long email thread since my merge and thanks all for > the comments. > In general, I still feel current process is a little complicated for > the maintainers who don't daily work on EDK2 like me. I have less > than %5 of time spent on open

Re: 回复: [edk2-devel] Tianocore community page on who we are - please review

2020-09-28 Thread Laszlo Ersek
Hi Liming, On 09/27/20 04:32, gaoliming wrote: > Guidelines for a Maintainer. Never let a pending request get older than a > calendar week. This requirement is too strict to the maintainer or reviewer. > The maintainer or reviewer should try to give the response in one week. But, > they may

Re: 回复: [edk2-devel] Tianocore community page on who we are - please review

2020-09-28 Thread Soumya Guptha
Jiewen, Good point on the release maintainer definition - we need to add this. Yes I think we need to add those guidelines for maintainer. (Please keep in mind to separate the role vs process, process will stay in maintainers process document and we link to it. Please hash out contributor role.

Re: [edk2-devel] [PATCH v2 0/2] UEFI memmap workaround for hiding page-access caps from OSes hides SP and CRYPTO caps too

2020-09-28 Thread Malgorzata Kukiello
Liming, As for mktme there is a change commited: https://patchwork.kernel.org/patch/10935909/ As for SP I can't find anything specific. Thanks Meg -Original Message- From: devel@edk2.groups.io On Behalf Of gaoliming Sent: Friday, September 25, 2020 10:55 AM To: devel@edk2.groups.io;

Re: [edk2-devel] [PATCH v1 1/1] UefiPayloadPkg: Set default PciBaseSize on Ia32

2020-09-28 Thread Ma, Maurice
Reviewed-by: Maurice Ma > -Original Message- > From: devel@edk2.groups.io On Behalf Of Marcello > Sylvester Bauer > Sent: Thursday, September 24, 2020 4:49 > To: devel@edk2.groups.io > Cc: Marcello Sylvester Bauer ; > patrick.rudo...@9elements.com; Ma, Maurice ; Dong, > Guo ; You,

回复: [edk2-devel] [PATCH 1/1] BaseTools: Update license of UserManuals files

2020-09-28 Thread gaoliming
Yuwei: Another option is to remove those manual files. Those manual files are very old. They are out of maintain. I suggest to remove them all. The developer can type --help to find tool help message. Thanks Liming > -邮件原件- > 发件人: bounce+27952+65678+4905953+8761...@groups.io > 代表

Re: [EXTERNAL] RE: [edk2-announce] 回复: [edk2-devel] Tianocore community page on who we are - please review

2020-09-28 Thread Leif Lindholm
Hi Jiewen, On Sun, Sep 27, 2020 at 03:25:24 +, Yao, Jiewen wrote: > Thanks Liming. > > It seems I have some misunderstanding here. > > According to current process, I feel that only maintainer has right to > *approve* the patch. > The reviewer cannot approve the patch. > Do you mean the

Re: 回复: [edk2-devel] Tianocore community page on who we are - please review

2020-09-28 Thread Leif Lindholm
Hi Liming, On Sun, Sep 27, 2020 at 10:32:55 +0800, gaoliming wrote: > Jiewen: > > Now, we have reviewer and maintainer role. Reviewer takes role 1~4. > Maintainer takes role 1~7. If the people know edk2 process well, they mostly > know edk2 one or more packages (modules). So, they can take

[edk2-devel] [PATCH 1/1] BaseTools: Update license of UserManuals files

2020-09-28 Thread Yuwei Chen
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2890 All files in BaseTools\UserManuals directory still use old BSD license. Update to new SPDX-License-Identifier: BSD-2-Clause-Patent license. Signed-off-by: Yuwei Chen Cc: Bob Feng Cc: Liming Gao ---

Re: [edk2-devel] [PATCH] NetworkPkg/HttpDxe: TlsChildHandle is not cleared when reset

2020-09-28 Thread Wu, Jiaxin
Reviewed-by: Wu Jiaxin Thanks, Jiaxin > -Original Message- > From: Abner Chang > Sent: Monday, September 28, 2020 2:30 PM > To: devel@edk2.groups.io > Cc: abner.ch...@hpe.com; Maciej Rabeda ; > Wu, Jiaxin ; Fu, Siyuan ; Nickle > Wang > Subject: [PATCH] NetworkPkg/HttpDxe:

[edk2-devel] [PATCH] NetworkPkg/HttpDxe: TlsChildHandle is not cleared when reset

2020-09-28 Thread Abner Chang
BZ #2909, https://bugzilla.tianocore.org/show_bug.cgi?id=2909 When Http->Configure() is invoked with HttpConfigData set to NULL to reset the EFI HTTP instance, TLS child instance is destroyed but HttpInstance->TlsChildHandle is not set to NULL. After reconfiguring HTTP through Http->Configure()

[edk2-devel] [PATCH] MdePkg/Include: Add HTTP definitions

2020-09-28 Thread Abner Chang
BZ #2915, https://bugzilla.tianocore.org/show_bug.cgi?id=2915 Add definitions for HTTP chunk transfer. HTTP chunk would be used in UEFI Redfish solution for provisioning or patch BIOS Redfish properties. Signed-off-by: Abner Chang Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Wu