[edk2] [Patch 6/6] MdeModulePkg: Add UNI file for PlatformBootManagerLibNull library.

2015-05-14 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Liming Gao --- MdeModulePkg/Library/PlatformBootManagerLibNull/PlatformBootManagerLibNull.inf | 1 + MdeModulePkg/Library/PlatformBootManagerLibNull/PlatformBootManagerLibNull.uni | Bin 0 -> 1756 bytes 2 fil

[edk2] [Patch 4/6] MdeModulePkg: Add UNI files for BootManagerMenuApp application.

2015-05-14 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Liming Gao --- MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuApp.inf | 9 - MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuApp.uni | Bin 0 -> 2110 bytes MdeModulePk

[edk2] [Patch 5/6] MdeModulePkg: Add UNI file for UefiBootManagerLib library.

2015-05-14 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Liming Gao --- MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf | 11 --- MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.uni | Bin 0 -> 2378 bytes 2 files changed, 8 insertio

[edk2] [Patch 3/6] MdeModulePkg: Add UNI files for DriverHealthManagerDxe driver.

2015-05-14 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Liming Gao --- MdeModulePkg/Universal/DriverHealthManagerDxe/DriverHealthManagerDxe.inf | 8 +++- MdeModulePkg/Universal/DriverHealthManagerDxe/DriverHealthManagerDxe.uni | Bin 0 -> 2302 bytes

[edk2] [Patch 1/6] MdeModulePkg: Add UNI file for BdsDxe driver.

2015-05-14 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Liming Gao --- MdeModulePkg/Universal/BdsDxe/BdsDxe.inf | 4 MdeModulePkg/Universal/BdsDxe/BdsDxe.uni | Bin 0 -> 2494 bytes MdeModulePkg/Universal/BdsDxe/BdsDxeExtra.uni | Bin 0 -> 1356 bytes 3

[edk2] [Patch 0/6] Add UNI files for BDS core modules/libraries

2015-05-14 Thread Ruiyu Ni
Ruiyu Ni (6): MdeModulePkg: Add UNI file for BdsDxe driver. MdeModulePkg: Add UNI files for BootManagerPolicyDxe driver. MdeModulePkg: Add UNI files for DriverHealthManagerDxe driver. MdeModulePkg: Add UNI files for BootManagerMenuApp application. MdeModulePkg: Add UNI file for UefiBootMa

[edk2] [Patch 2/6] MdeModulePkg: Add UNI files for BootManagerPolicyDxe driver.

2015-05-14 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni CC: Liming Gao --- MdeModulePkg/Universal/BootManagerPolicyDxe/BootManagerPolicyDxe.c| 2 +- MdeModulePkg/Universal/BootManagerPolicyDxe/BootManagerPolicyDxe.inf | 4 MdeModulePkg/Universal/Bo

Re: [edk2] [PATCH] CorebootPayloadPkg: Replace PciHostBridge driver with PciRootBridgeNoEnumeration

2015-05-14 Thread Jordan Justen
On 2015-05-08 15:32:52, Ma, Maurice wrote: > Ideally I don't want CorebootPayloadPkg to depend on DuetPkg. But on > the other side I hesitate to duplicate drivers into the > CorebootPayloadPkg from other packages. If I have to make a choice, > I'd like the former. I understand the struggle. It is

Re: [edk2] [PATCH] CorebootPayloadPkg: Replace PciHostBridge driver with PciRootBridgeNoEnumeration

2015-05-14 Thread Jordan Justen
On 2015-05-14 19:17:26, Ni, Ruiyu wrote: > Maurice, > When PcdPciDisableBusEnumeration is TRUE, PciBus driver assumes the > bus number and IO/MMIO BAR are all initialized before it gets > started, so it doesn't call SetBusNumbers(). It's also true for > PciBusNoEnumeration driver in DuetPkg. > I co

[edk2] [PATCH] MdeModulePkg PiSmmCoreMemoryAllocationLib: Get SMRAM ranges from FullSmramRanges and FullSmramRangeCount in SmmCorePrivate by Constructor.

2015-05-14 Thread Star Zeng
It can avoid potential first call to FreePool() -> BufferInSmram() -> if (mSmramRanges == NULL) { GetSmramRanges();} -> gBS->LocateProtocol() at boottime with >= TPL_NOTIFY or after ReadyToLock or at OS runtime. Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [PATCH v2 2/2] SourceLevelDebugPkg/DebugAgentDxe: Move help info from DxeDebugAgent

2015-05-14 Thread Jeff Fan
Now DxeDebugAgent Library instance will print help information on how to load DebugAgentDxe.efi in UEFI shell. But it is printed after Target connected to Host side. This fix is to move help info print to DebugAgentDxe module before Target tries to connect HOST. It could help developer to get usefu

[edk2] [PATCH v2 1/2] SourceLevelDebugPkg/DxeDebugAgent: Initialize Local APIC Timer early

2015-05-14 Thread Jeff Fan
Now Debug Agent library uses Local APIC Timer to implement time-out mechanism. But it does not initialize Local APIC Timer before transfer data with HOST. This fix is to move Local APIC Timer initialization to SetupDebugAgentEnvironment(). This fix also updates function header and fixed one typo of

[edk2] [PATCH v2 0/2] SourceDebugAgent/DxeDebugAgent: Initialize Local APIC Timer

2015-05-14 Thread Jeff Fan
DxeDebugAgent library instance initialize Local APIC timer is too late to be used for time-out mechanism. Also move help information for serial debug port to DebugAgentDxe module to help developer as early as possible. Changes since v1: a) Update function header comments in SetupDebugAgentEnvironm

Re: [edk2] [Patch 1/2] SourceLevelDebugPkg/DxeDebugAgent: Initialize Local APIC Timer early

2015-05-14 Thread Ni, Ruiyu
No concerns now:) Remember to change the function header when you check in. Reviewed-by: Ruiyu Ni -Original Message- From: Fan, Jeff Sent: Friday, May 15, 2015 10:09 AM To: Ni, Ruiyu; edk2-devel@lists.sourceforge.net Subject: RE: [Patch 1/2] SourceLevelDebugPkg/DxeDebugAgent: Initialize

Re: [edk2] [PATCH] CorebootPayloadPkg: Replace PciHostBridge driver with PciRootBridgeNoEnumeration

2015-05-14 Thread Ni, Ruiyu
Maurice, When PcdPciDisableBusEnumeration is TRUE, PciBus driver assumes the bus number and IO/MMIO BAR are all initialized before it gets started, so it doesn't call SetBusNumbers(). It's also true for PciBusNoEnumeration driver in DuetPkg. I compared the PciBus and PciBusNoEnumeration before, t

Re: [edk2] [Patch 1/2] SourceLevelDebugPkg/DxeDebugAgent: Initialize Local APIC Timer early

2015-05-14 Thread Fan, Jeff
Thanks your comments: 1. I will add some initial Local APIC contents in comments in function header of SetupDebugAgentEnvironment(). 2. Yes. This patch also fixed the similar issue on S3 path. Because original S3 path, local APIC timer needn't to be initialized. But now it is required for time-

Re: [edk2] [RFC 0/4] New terminal type for Linux

2015-05-14 Thread Yao, Jiewen
Good suggestion, Mike. I believe "Tty" is much better than "Linux". -Original Message- From: Kinney, Michael D Sent: Friday, May 15, 2015 3:31 AM To: Yao, Jiewen; edk2-devel@lists.sourceforge.net; linaro-u...@lists.linaro.org; Tian, Feng; Kinney, Michael D; roy.fr...@linaro.org Subject

[edk2] [PATCH 0/2] Header files and device support for Wi-Fi protocol

2015-05-14 Thread Hao Wu
UEFI 2.5 introduces EFI wireless MAC connection protocol. The header file will be added into MdePkg. Wi-Fi device path definition and its node/text conversion are added as well. Hao Wu (2): MdePkg: Add EFI Wireless MAC Connection Protocol definitions MdePkg: Add WiFi device path definition and

[edk2] [PATCH 2/2] MdePkg: Add WiFi device path definition and its node/text conversion

2015-05-14 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Ye Ting --- MdePkg/Include/Protocol/DevicePath.h | 14 ++ .../Library/UefiDevicePathLib/DevicePathFromText.c | 31 ++ .../Library/UefiDevicePathLib/DevicePathToTex

[edk2] [PATCH 1/2] MdePkg: Add EFI Wireless MAC Connection Protocol definitions

2015-05-14 Thread Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by: Ye Ting --- MdePkg/Include/Protocol/WiFi.h | 1144 MdePkg/MdePkg.dec |3 + 2 files changed, 1147 insertions(+) create mode 100644 MdePkg/Include/P

Re: [edk2] [PATCH] MdeModulePkg: remove unused variable DevicePathSize

2015-05-14 Thread Dong, Eric
Thanks for the patch. Reviewed-by: Eric Dong Also checked in the code in r17448. Thanks, Eric -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Thursday, May 14, 2015 3:47 PM To: ler...@redhat.com; edk2-devel@lists.sourceforge.net; Dong, Eric Cc: Ard Bie

Re: [edk2] [RFC 0/4] New terminal type for Linux

2015-05-14 Thread Kinney, Michael D
Roy, 1) New GUID should be defined in MdeModulePkg/MdeModulePkg.dec 2) New include file for new GUID in MdeModulePkg/Include/Guid 3) Update TerminalDxe to use this new GUID. I do not recommend creating a new terminal driver. Please continue as you have which is adding support for this one new t

Re: [edk2] [RFC 0/4] New terminal type for Linux

2015-05-14 Thread Roy Franz
On Thu, May 14, 2015 at 12:31 PM, Kinney, Michael D wrote: > Jiewen, > > I think most of the terminal emulators being discussed here layer on top of a > TTY device. > > How about TtyTerm instead? I think this is a more appropriate name. Michael - I'll also try to address feedback on the new GUI

Re: [edk2] [RFC 0/4] New terminal type for Linux

2015-05-14 Thread Kinney, Michael D
Jiewen, I think most of the terminal emulators being discussed here layer on top of a TTY device. How about TtyTerm instead? Mike -Original Message- From: Yao, Jiewen Sent: Wednesday, May 13, 2015 7:13 PM To: Kinney, Michael D; edk2-devel@lists.sourceforge.net; linaro-u...@lists.lina

[edk2] EDK II BaseTools Binaries VS2013

2015-05-14 Thread Hauch, Larry
Hi Folks, We have updated automated build of the EDK II BaseTools Win32 binaries to be built using Microsoft* Visual Studio 2013. Refer to e-mail from Liming Gao of April 23, 2015, that states: "new binary BaseTools will not work in the old windows OS (XP and Server 2003). This is a known issue

Re: [edk2] [Patch 2/2] SourceLevelDebugPkg/DebugAgentDxe: Move help info from DxeDebugAgent

2015-05-14 Thread Carsey, Jaben
Isn't having a library print information to console out a pretty significant violation of the principles of the UEFI spec? -Jaben > -Original Message- > From: Jeff Fan [mailto:jeff@intel.com] > Sent: Thursday, May 14, 2015 1:42 AM > To: edk2-devel@lists.sourceforge.net > Subject: [ed

Re: [edk2] [patch] ShellPkg: Add DiskInfo GUID matching to the Shell

2015-05-14 Thread Carsey, Jaben
Ok. Committed. 17447 From: El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hp.com] Sent: Thursday, May 14, 2015 9:02 AM To: edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [patch] ShellPkg: Add DiskInfo GUID matching to the Shell Importance: High UNI file attached From: Carsey, Jaben [m

Re: [edk2] [patch] ShellPkg: Add DiskInfo GUID matching to the Shell

2015-05-14 Thread El-Haj-Mahmoud, Samer
UNI file attached From: Carsey, Jaben [mailto:jaben.car...@intel.com] Sent: Wednesday, May 13, 2015 6:00 PM To: edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [patch] ShellPkg: Add DiskInfo GUID matching to the Shell Looks good, but patch files don't work for the UNI file it seems. Can you

Re: [edk2] [PATCH] Fix: MdeModulePkg: Implement UEFI25 HII Config keyword handler protocol.

2015-05-14 Thread Ryan Harkin
OK, so it seems Ard beat me to it and posted a patch already. On 14 May 2015 at 10:58, Ryan Harkin wrote: > This patch removes the unused DevicePathSize variable. > > The following commit introduces the DevicePathSize variable in function > GetStringIdFromDatabase: > > commit 87bfeb11f84

[edk2] [PATCH] Fix: MdeModulePkg: Implement UEFI25 HII Config keyword handler protocol.

2015-05-14 Thread Ryan Harkin
This patch removes the unused DevicePathSize variable. The following commit introduces the DevicePathSize variable in function GetStringIdFromDatabase: commit 87bfeb11f84dd1d369bb2e195ef83d20b0d80d61 Author: Eric Dong Date: Wed May 13 08:35:29 2015 + Md

Re: [edk2] [PATCH 2/4] MdeModulePkg DxeCore: Add OEM reserved memory type support.

2015-05-14 Thread Gao, Liming
Star: Could you help add more comments to show the valid memory type range? Other change looks good to me. -Original Message- From: Zeng, Star Sent: Thursday, May 14, 2015 5:22 PM To: edk2-devel@lists.sourceforge.net Cc: Yao, Jiewen; Gao, Liming Subject: [PATCH 2/4] MdeModulePkg Dx

[edk2] [PATCH 2/4] MdeModulePkg DxeCore: Add OEM reserved memory type support.

2015-05-14 Thread Star Zeng
Cc: Jiewen Yao Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- MdeModulePkg/Core/Dxe/Mem/Imem.h | 7 ++- MdeModulePkg/Core/Dxe/Mem/Page.c | 4 ++-- MdeModulePkg/Core/Dxe/Mem/Pool.c | 9 + 3 files changed, 13 insertions(+), 7 deleti

[edk2] [PATCH 4/4] MdeModulePkg: Update memory profile for OEM reserved memory type.

2015-05-14 Thread Star Zeng
Cc: Jiewen Yao Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- .../MemoryProfileInfo/MemoryProfileInfo.c | 39 +++-- MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c| 15 +--- MdeModulePkg/Include/Guid/MemoryP

[edk2] [PATCH 1/4] MdePkg UefiSpec.h: Update comments for OEM reserved memory type.

2015-05-14 Thread Star Zeng
UEFI 2.5 Spec: MemoryType values in the range 0x7000..0x7FFF are reserved for OEM use. Cc: Jiewen Yao Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- MdePkg/Include/Uefi/UefiSpec.h | 16 +--- 1 file changed, 13 insertions(

[edk2] [PATCH 0/4] Add OEM reserved memory type support

2015-05-14 Thread Star Zeng
Cc: Jiewen Yao Cc: Liming Gao The patches follow UEFI 2.5 Spec to add OEM reserved memory type support, and update memory profile accordingly. Star Zeng (4): MdePkg UefiSpec.h: Update comments for OEM reserved memory type. MdeModulePkg DxeCore: Add OEM reserved memory type support. MdeMo

[edk2] [PATCH 3/4] MdeModulePkg PiSmmCore: Care runtime code/data only for SMRAM profile.

2015-05-14 Thread Star Zeng
Cc: Jiewen Yao Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- MdeModulePkg/Core/PiSmmCore/SmramProfileRecord.c | 82 +--- 1 file changed, 46 insertions(+), 36 deletions(-) diff --git a/MdeModulePkg/Core/PiSmmCore/SmramPro

Re: [edk2] [Patch 1/2] SourceLevelDebugPkg/DxeDebugAgent: Initialize Local APIC Timer early

2015-05-14 Thread Ni, Ruiyu
Jeff, SetupDebugAgentEnvironment() behavior was changed but the function header wasn't updated. If the Local APIC Timer initialization is moved to SetupDebugAgentEnvironment(), does it impact S3 debugging? Because SetupDebugAgentEnvironment() is also called when DEBUG_AGENT_INIT_S3. Thanks, Ray

[edk2] [Patch 2/2] SourceLevelDebugPkg/DebugAgentDxe: Move help info from DxeDebugAgent

2015-05-14 Thread Jeff Fan
Now DxeDebugAgent Library instance will print help information on how to load DebugAgentDxe.efi in UEFI shell. But it is printed after Target connected to Host side. This fix is to move help info print to DebugAgentDxe module before Target tries to connect HOST. It could help developer to get usefu

[edk2] [Patch 0/2] SourceDebugAgent/DxeDebugAgent: Initialize Local APIC Timer

2015-05-14 Thread Jeff Fan
DxeDebugAgent library instance initialize Local APIC timer is too late to be used for time-out mechanism. Also move help information for serial debug port to DebugAgentDxe module to help developer as early as possible. Jeff Fan (2): SourceLevelDebugPkg/DxeDebugAgent: Initialize Local APIC Timer

[edk2] [Patch 1/2] SourceLevelDebugPkg/DxeDebugAgent: Initialize Local APIC Timer early

2015-05-14 Thread Jeff Fan
Now Debug Agent library uses Local APIC Timer to implement time-out mechanism. But it does not initialize Local APIC Timer before transfer data with HOST. This fix is to move Local APIC Timer initialization to SetupDebugAgentEnviroment(). Contributed-under: TianoCore Contribution Agreement 1.0 Sig

Re: [edk2] [PATCH] MdeModulePkg: remove unused variable DevicePathSize

2015-05-14 Thread Laszlo Ersek
On 05/14/15 09:47, Ard Biesheuvel wrote: > Remove the unused DevicePathSize variable from EnumerateAllKeywords(). > Due to our use of -Werror=unused-but-set-variable when building under > GCC, the presence of DevicePathSize is breaking the build. > > Contributed-under: TianoCore Contribution Agree

Re: [edk2] [RFC 4/4] Change QEMU terminal type to LinuxTerm

2015-05-14 Thread Laszlo Ersek
On 05/14/15 01:54, Roy Franz wrote: > This changes the QEMU terminal type to LinuxTerm, which > handles the terminals commonly used to connect to the > emulated serial port. > > Signed-off-by: Roy Franz > Contributed-under: TianoCore Contribution Agreement 1.0 > --- > ArmPlatformPkg/ArmVirtualiz

[edk2] [PATCH] MdeModulePkg: remove unused variable DevicePathSize

2015-05-14 Thread Ard Biesheuvel
Remove the unused DevicePathSize variable from EnumerateAllKeywords(). Due to our use of -Werror=unused-but-set-variable when building under GCC, the presence of DevicePathSize is breaking the build. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- MdeMod

Re: [edk2] commit 87bfeb11 (svn 17428) breaks OVMF x86_64 linux build

2015-05-14 Thread Laszlo Ersek
On 05/13/15 22:46, Gabriel L. Somlo wrote: > Hi, > > I build OVMF using > > build -a X64 -t GCC49 -p OvmfPkg/OvmfPkgX64.dsc > > As of commit 87bfeb11f84dd1d369bb2e195ef83d20b0d80d61 (svn 17428), I > started getting the error below. Apparently GCC cares about the > difference between UINT8*