Re: [edk2-devel] [PATCH 24/25] MdeModulePkg/ResetUtilityLib: Add a new API ResetSystemWithSubtype

2019-04-16 Thread Wu, Hao A
> -Original Message- > From: Gao, Zhichao > Sent: Monday, April 15, 2019 11:06 AM > To: devel@edk2.groups.io > Cc: Bret Barkelew; Wang, Jian J; Wu, Hao A; Ni, Ray; Zeng, Star; Gao, Liming; > Sean Brogan; Michael Turner > Subject: [PATCH 24/25] MdeModulePkg/ResetUtilityLib: Add a new API >

Re: [edk2-devel] [PATCH 00/25] Add the new APIs ResetSystem and ResetSystemWithSubtype

2019-04-16 Thread Wu, Hao A
> -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Gao, > Zhichao > Sent: Monday, April 15, 2019 11:06 AM > To: devel@edk2.groups.io > Cc: Kinney, Michael D; Gao, Liming; Zhang, Chao B; Yao, Jiewen; Wang, Jian J; > Wu, Hao A; Ni, Ray; Zeng, Star;

Re: [edk2-devel] [PATCH] MdeModulePkg/UefiBootManagerLib: fix crash on uninitialized ExitData

2019-04-16 Thread Gary Lin
On Tue, Apr 16, 2019 at 01:29:35PM -0700, Ard Biesheuvel wrote: > As reported by Gary, the recent LoadImage/StartImage changes to > accommodate dispatching PE/COFF images built for foreign architectures > may result in a crash when loading an IA32 option ROM into a X64 VM > running OVMF: > >

[edk2-devel] [PATCH] BaseTools:Makefiles cause data errors on some platforms

2019-04-16 Thread Fan, ZhijuX
As the Dict is unordered, an error occurs when using the Dict while creating a new item.So for now, use OrdereDict instead of Dict. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Zhiju.Fan --- BaseTools/Source/Python/AutoGen/GenMake.py | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-)

Re: [edk2-devel] [PATCH V2 0/5] Make some DebugLib instance runtime safe

2019-04-16 Thread Liming Gao
Reviewed-by: Liming Gao >-Original Message- >From: Gao, Zhichao >Sent: Friday, April 12, 2019 8:42 AM >To: devel@edk2.groups.io >Cc: Yao, Jiewen ; Zhang, Chao B >; Wang, Jian J ; Wu, Hao A >; Ni, Ray ; Zeng, Star >; Kinney, Michael D ; >Gao, Liming ; Sean Brogan >; Michael Turner >; Bret

[edk2-devel] [PATCH] BaseTools: Hash false success.. minor change in hash invalidation

2019-04-16 Thread Christian Rodriguez
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1692 Change part of the hash error handling to invalidate hashes in the cache destination not the cache source. Signed-off-by: Christian Rodriguez Cc: Bob Feng Cc: Liming Gao Cc: Yonghong Zhu --- BaseTools/Source/Python/build/build.py | 4

Re: [edk2-devel] [edk2-platforms: PATCH v2 0/6] Armada7k8k misc improvements

2019-04-16 Thread Ard Biesheuvel
On Tue, 16 Apr 2019 at 02:23, Marcin Wojtas wrote: > > Hi, > > A second version of the patchset brings one, though significant > change. An idea of using custom DtLoaderLib for the PMU IRQ handling > was dropped. Instead use existing PlatInitDxe. The EL3 service > is still switched off in the

Re: [edk2-devel] [edk2-platforms: PATCH v2 3/6] Marvell/Armada7k8k: Implement PMU interrupt handling

2019-04-16 Thread Ard Biesheuvel
On Tue, 16 Apr 2019 at 02:23, Marcin Wojtas wrote: > > Generic handling of the PMU interrupts in UEFI and Linux with > ACPI require enabling a dedicated handler in the EL3. > Extend the PlatInitDxe with enabler code. > > Because for DT world the EL3 service must remain disabled, > switch it off

Re: [edk2-devel] [PATCH 02/10] MdePkg/PiFirmwareFile: fix undefined behavior in SECTION_SIZE

2019-04-16 Thread Jordan Justen
On 2019-04-16 03:59:48, Laszlo Ersek wrote: > On 04/16/19 11:04, Jordan Justen wrote: > > On 2019-04-15 09:15:31, Laszlo Ersek wrote: > >> On 04/14/19 09:19, Jordan Justen wrote: > >>> On 2019-04-12 16:31:20, Laszlo Ersek wrote: > RH covscan justifiedly reports that accessing >

[edk2-devel] Remove debug strings from OVMF.fd

2019-04-16 Thread Pedro Barbosa
Hi EDK2 developers, I'm trying to build OVMF without debug symbols but even building as RELEASE, I still can see some strings on the PE files, such as: /Build/OvmfX64/RELEASE_GCC5/X64/MdeModulePkg/Application/UiApp/UiApp/DEBUG/UiApp.dll and many others in the format */DEBUG/*.dll Is there a way

[edk2-devel] [edk2-platforms: PATCH v2 1/6] Marvell/Armada7k8k: RealTimeClockLib: Update bus parameters

2019-04-16 Thread Marcin Wojtas
From: Mark Kettenis Adjust bus timing parameters to make reading and updating the RTC reliable. This patch aligns the bus configuration to the one used by Linux. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Marcin Wojtas ---

[edk2-devel] [PATCH] Edk2\NetworkPkg\UefiPxeBcDxe : Open a method to Enabled/Disabled IPV6/IPV6 PXE Support.

2019-04-16 Thread Xue, ShengfengX
Please refer this link for defail. https://bugzilla.tianocore.org/show_bug.cgi?id=1695 Some Platform setup need provide a item for user to Enabled/Disabled IPV4/IPV6 PXE boot, but Origin UefiPxeBcDxe driver doesn't have such interface. So attached code review added two PCD to control IPV4/IPV6

Re: [edk2-devel] [RFC] Propose update of security bug handling process

2019-04-16 Thread Wang, Jian J
Laszlo, I got your point, and understood the requirements for source released platforms. I think it's something needing all vendors who have concerns to get agreement with. It's part of 'optimization' Vincent mentioned in another email. Stephano will set up meetings to drive related discussions.