[edk2] [PATCH] MdeModulePkg RamDiskDxe: Fix wrong HII behavior for more than 8 RAM disks

2016-04-19 Thread Hao Wu
The RamDiskDxe driver originally uses a variable-length HII varstore to retrieve the HII checkbox status of each registered RAM disk. However, HII does not support the variable-length varstore feature. Therefore, only the checkbox status for the first 8 RAM disks are tracked for the following defi

Re: [edk2] [staging/HTTPS-TLS][PATCH 0/3] Provide an UI to support TLS authentication

2016-04-19 Thread Wu, Jiaxin
Hi Samer, I also provide a UI config driver to support TLS certificate configuration, this may be helpful to you. Please help to review it. Thanks Jiaxin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Jiaxin Wu > Sent: Wednesday, April 20

[edk2] [staging/HTTPS-TLS][PATCH 0/3] Provide an UI to support TLS authentication

2016-04-19 Thread Jiaxin Wu
Cc: Ye Ting Cc: Fu Siyuan Cc: Long Qin Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu Jiaxin Wu (3): NetworkPkg: Provide an UI to support tls authentication. Nt32Pkg: Add TlsAuthConfigDxe module Readme.MD: Add content for TlsAuthConfigDxe NetworkPkg/In

[edk2] [staging/HTTPS-TLS][PATCH 2/3] Nt32Pkg: Add TlsAuthConfigDxe module

2016-04-19 Thread Jiaxin Wu
This patch is used to add TlsAuthConfigDxe module. Cc: Ye Ting Cc: Fu Siyuan Cc: Long Qin Cc: El-Haj-Mahmoud Samer Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu --- Nt32Pkg/Nt32Pkg.dsc | 1 + Nt32Pkg/Nt32Pkg.fdf | 1 + 2 files changed, 2 insertions(+) diff

[edk2] [staging/HTTPS-TLS][PATCH 1/3] NetworkPkg: Provide an UI to support TLS authentication

2016-04-19 Thread Jiaxin Wu
This patch provides an UI to support TLS authentication. EFI_SIGNATURE_LIST format is used for 'TlsCaCertificate' variable. So, TLS supports multiple certificate configuration. Cc: Ye Ting Cc: Fu Siyuan Cc: Long Qin Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin

[edk2] [staging/HTTPS-TLS][PATCH 3/3] Readme.MD: Add content for TlsAuthConfigDxe

2016-04-19 Thread Jiaxin Wu
Cc: Ye Ting Cc: Fu Siyuan Cc: Long Qin Cc: El-Haj-Mahmoud Samer Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu --- Readme.MD | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Readme.MD b/Readme.MD index 573593e..64c696d 100644 --- a/Readme.MD +++ b/Readme

Re: [edk2] [staging/HTTPS-TLS][PATCH] Readme.MD: Add Readme for HTTPS-TLS branch

2016-04-19 Thread El-Haj-Mahmoud, Samer
Reviewed-by: Samer El-Haj-Mahmoud -Original Message- From: Jiaxin Wu [jiaxin...@intel.com] Received: Tuesday, 19 Apr 2016, 9:58PM To: edk2-devel@lists.01.org [edk2-devel@lists.01.org] CC: Ye Ting [ting...@intel.com]; Fu Siyuan [siyuan...@intel.com]; Long Qin [qin.l...@intel.com]; El-Ha

[edk2] [staging/HTTPS-TLS][PATCH] Readme.MD: Add Readme for HTTPS-TLS branch

2016-04-19 Thread Jiaxin Wu
Cc: Ye Ting Cc: Fu Siyuan Cc: Long Qin Cc: El-Haj-Mahmoud Samer Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu --- Readme.MD | 44 1 file changed, 44 insertions(+) create mode 100644 Readme.MD diff --git a/Readme

Re: [edk2] [PATCH] ShellPkg: Support connect a device handle recursively with '-r'.

2016-04-19 Thread Qiu, Shumin
Hi Jaben, 'ShellCommandLineGetCount' return the count of value parameters including the command name and excluding the flags. So if we execute the command 'connect 1E -r' in Shell. The count return from 'ShellCommandLineGetCount' is 2 The first one is 'connect' and the second one is '1E'. The 1E

[edk2] [PATCH v2 0/2] Report ACPI NFIT for reserved memory RAM disks

2016-04-19 Thread Hao Wu
Changes compared with V1: 1. Instead of creating a new NFIT for each registered reserved memory RAM disk, new cotent of the NFIT is appended to the existing one. 2. Report an NVDIMM root device in the \SB scope if there is no NFIT in the ACPI table. 3. Modify FDF files in OvmfPkg to make su

[edk2] [PATCH v2 1/2] MdeModulePkg RamDiskDxe: Report ACPI NFIT for reserved memory RAM disks

2016-04-19 Thread Hao Wu
The RamDiskDxe now will report RAM disks with reserved memory type to NFIT in the ACPI table. This commit will also make sure that an NVDIMM root device exists in the \SB scope before reporting any RAM disk to NFIT. To properly report the NVDIMM root device, one will modify the RamDiskDxe item li

[edk2] [PATCH v2 2/2] OvmfPkg: Modify fdf files due to RamDiskDxe driver's adding ASL code

2016-04-19 Thread Hao Wu
The RamDiskDxe driver in MdeModulePkg now will sometimes report a NVDIMM Root Device using ASL code which is put in a Secondary System Description Table (SSDT) according to the ACPI 6.1 spec. Locating the SSDT requires modifying the fdf files in OvmfPkg. Cc: Laszlo Ersek Cc: Jordan Justen Contr

Re: [edk2] OVMF: hang when booting Linux via GRUB after driver allocates 2GB above 4GB

2016-04-19 Thread Bruce Cran
On 4/19/16 6:04 AM, Laszlo Ersek wrote: Bruce, could you please test this? If 742563777e8d is indeed the solution, I will file an RHBZ (for the RHEL kernel) to backport it. Laszlo, I've just verified that 742563777e8d does indeed fix the problem: that kernel successfully boots after the memo

Re: [edk2] [PATCH 06/12] CorebootPayloadPkg: Make debug serial PCDs patchable

2016-04-19 Thread Ma, Maurice
This patch looks good to me. Reviewed-by: Maurice Ma Thanks Maurice -Original Message- From: Leahy, Leroy P Sent: Tuesday, April 19, 2016 1:31 PM To: edk2-devel@lists.01.org; Ma, Maurice; Agyeman, Prince; Leahy, Leroy P Subject: [PATCH 06/12] CorebootPayloadPkg: Make debug serial PCDs

Re: [edk2] [PATCH 05/12] CorebootPayloadPkg: Make shell selectable

2016-04-19 Thread Ma, Maurice
Hi, Lee, Did you try 64 bit build with this change?It might have issue. For 32 bit build, the ARCH variable will be set to "IA32" in DSC.It will have no issue to use: INF RuleOverride = BINARY USE = $(ARCH) ShellBinPkg/UefiShell/UefiShell.inf However for 64 bit payload, the ARCH v

Re: [edk2] [PATCH 04/12] CorebootPayloadPkg/PciBusNoEnumerationDxe: Skip disabled devices

2016-04-19 Thread Leahy, Leroy P
HI Maurice, Coreboot only assigns resources to devices which are enabled. If the device is set to OFF in the mainboard/.../devicetree.cb file then no resources are assigned and the device is left disabled. This is the issue that I am running into which is causing PciBusNoEnumerationDxe to fai

Re: [edk2] [PATCH 03/12] CorebootModulePkg/SerialDxe: Use PlatformHookLib

2016-04-19 Thread Leahy, Leroy P
Hi Maurice, This driver is being built with PlatformHelperLib to pass the serial port configuration from coreboot. The .inf file is modified and the driver is being rebuilt. Lee Leahy (425) 881-4919 Intel Corporation Suite 125 2700 - 156th Ave NE Bellevue, WA 98007-6554 -Original Message

Re: [edk2] [PATCH 02/12] CorebootModulePkg/PciSioSerialDxe: Use PlatformHookLib

2016-04-19 Thread Leahy, Leroy P
Hi Maurice, Quark uses a PCI serial port, not the legacy serial port. As such, I need to pass parameters from coreboot to the PCI serial driver. This requires the use of PlatformHelperLib which requires that I modify the .inf file and rebuild the serial driver. Lee Leahy (425) 881-4919 Intel

Re: [edk2] [PATCH 04/12] CorebootPayloadPkg/PciBusNoEnumerationDxe: Skip disabled devices

2016-04-19 Thread Ma, Maurice
Hi, Lee, Do we assume all PCI devices will be enabled in the command registers in coreboot before transitioning to the UEFI payload ? How about those PCI devices with command register disabled but with PCI BARs programmed ? If we skip them, then the resource will not be collected by the PciB

Re: [edk2] [PATCH 03/12] CorebootModulePkg/SerialDxe: Use PlatformHookLib

2016-04-19 Thread Ma, Maurice
Hi, Lee, I have the same question as previous patch. Please let us know why the driver in MdeModulePkg does not fit your needs. If it is only for PCD initialization, we might consider using PcdsDynamic type and set them in platform driver instead. Thanks Maurice -Original Message

Re: [edk2] [PATCH 02/12] CorebootModulePkg/PciSioSerialDxe: Use PlatformHookLib

2016-04-19 Thread Ma, Maurice
Hi, Lee, For this change, could you let us know why you decided to copy the PciSioSerialDxe over to the CorebootModulePkg ? I am wondering why the current driver in MdeModulePkg cannot be reused. Thanks Maurice -Original Message- From: Leahy, Leroy P Sent: Tuesday, April 19, 2016 1:29

Re: [edk2] [PATCH] ShellPkg: Support connect a device handle recursively with '-r'.

2016-04-19 Thread Carsey, Jaben
I am confused. It looks like the code change is that "-r" requires ...GetCount() >2 where it used to be >1. Why? I didn't think that recursive searching required both DeviceHandle and DriverHandle to be on the command line... -Jaben > -Original Message- > From: edk2-devel [mailto:edk

[edk2] [PATCH] QuarkPlatformPkg: Move baud rate setting to top of .dsc file

2016-04-19 Thread Lee Leahy
Move the baud rate setting to the top of the .dsc file. Use a single setting for each board. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- QuarkPlatformPkg/Quark.dsc| 25 - QuarkPlatformPkg/QuarkMin.dsc | 25

[edk2] [PATCH] QuarkSocPkg/SDControllerDxe: Add EFIAPI to SetHighSpeedMode

2016-04-19 Thread Lee Leahy
Fix 64-bit build error detected with GCC4.8 due to inconsistent routine declaration and implementation. Add EFIAPI to fix the build error. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- QuarkSocPkg/QuarkSouthCluster/Sdio/Dxe/SDControllerDxe/SDController.c |

[edk2] [PATCH 10/12] CorebootPayloadPkg: Allow MaxLogicalProcessorNumber to be changed

2016-04-19 Thread Lee Leahy
Add a define and use it with MaxLogicalProcessorNumber to enable this PCD to be changed via the command line. Quark needs to set this value to one during the builds. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootPayloadPkg/CorebootPayloadPkgIa32.ds

[edk2] [PATCH 09/12] CorebootPayloadPkg/PlatformBdsLib: Pass more serial parameters

2016-04-19 Thread Lee Leahy
Pass the serial port baudrate, register stride, input clock rate and ID from coreboot to CorebootPayloadPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootModulePkg/Include/Coreboot.h | 3 ++ CorebootModulePkg/Include/Library/CbParseL

[edk2] [PATCH 11/12] CorebootPayloadPkg/PlatformHelperLib: Remove unreferenced function

2016-04-19 Thread Lee Leahy
Remove the PlatformFlashEraseWrite function which is not used within CorebootPayloadPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- .../Include/Library/PlatformHelperLib.h| 38 +- .../Library/PlatformHelperLib/PlatformHelperDxe.c | 141

[edk2] [PATCH 07/12] CorebootPayloadPkg: Make serial I/O configurable

2016-04-19 Thread Lee Leahy
Allow the serial port configuration to be overriden from the command line. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc| 59 +++- CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc | 58 ++

[edk2] [PATCH 08/12] CorebootPayloadPkg: Use serial drivers with PlatformHookLib

2016-04-19 Thread Lee Leahy
Use the serial drivers which update the serial PCDs from PlatformHookLib. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootPayloadPkg/CorebootPayloadPkg.fdf| 3 ++- CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc| 11 ++- CorebootPay

[edk2] [PATCH 06/12] CorebootPayloadPkg: Make debug serial PCDs patchable

2016-04-19 Thread Lee Leahy
Make the debug serial PCDs patchable in module. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc| 16 +--- CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc | 16 +--- 2 files changed, 18

[edk2] [PATCH 05/12] CorebootPayloadPkg: Make shell selectable

2016-04-19 Thread Lee Leahy
Add all of the shell options from ShellBinPkg including building the shell from source. Test: Use -DSHELL_TYPE=BUILD_SHELL command line options to build the shell from source. Run the result on Galileo Gen2. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- C

[edk2] [PATCH 04/12] CorebootPayloadPkg/PciBusNoEnumerationDxe: Skip disabled devices

2016-04-19 Thread Lee Leahy
Skip non-bridge devices which are not enabled either for memory or I/O access. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootModulePkg/PciBusNoEnumerationDxe/PciEnumeratorSupport.c | 9 + 1 file changed, 9 insertions(+) diff --git a/Coreboo

[edk2] [PATCH 03/12] CorebootModulePkg/SerialDxe: Use PlatformHookLib

2016-04-19 Thread Lee Leahy
Copy the driver from MdeModulePkg/Universal/SerialDxe. Add PlatformHookLib to the Library section of the .inf file to adjust the PCDs for the UART. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootModulePkg/SerialDxe/SerialDxe.inf | 55 +++ Core

[edk2] [PATCH 02/12] CorebootModulePkg/PciSioSerialDxe: Use PlatformHookLib

2016-04-19 Thread Lee Leahy
Copy the driver from MdeModulePkg/Bus/Pci/PciSioSerialDxe. Add PlatformHookLib to the Library section of the .inf file to adjust the PCDs for the UART. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Lee Leahy --- CorebootModulePkg/PciSioSerialDxe/ComponentName.c | 288

Re: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: API BmIsValidLoadOptionVariableName

2016-04-19 Thread El-Haj-Mahmoud, Samer
Never mind, I just saw this was committed in 3dc5c1ae5c7575ba2d1714b40692b36fca56c120 Thanks! --Samer -Original Message- From: El-Haj-Mahmoud, Samer Sent: Tuesday, April 19, 2016 2:39 PM To: Palmer, Thomas ; edk2-devel@lists.01.org Cc: feng.t...@intel.com; star.z...@intel.com; ruiyu...

Re: [edk2] [PATCH] MdeModulePkg/UefiBootManagerLib: API BmIsValidLoadOptionVariableName

2016-04-19 Thread El-Haj-Mahmoud, Samer
Reviewed-by: Samer El-Haj-Mahmoud Ray, Feng, Star, Can you please help review and commit this change? -Original Message- From: Palmer, Thomas Sent: Monday, April 4, 2016 2:52 PM To: edk2-devel@lists.01.org Cc: El-Haj-Mahmoud, Samer ; feng.t...@intel.com; star.z...@intel.com; ruiyu...@

Re: [edk2] OVMF: hang when booting Linux via GRUB after driver allocates 2GB above 4GB

2016-04-19 Thread Bruce Cran
On 4/19/2016 6:04 AM, Laszlo Ersek wrote: Bruce, could you please test this? If 742563777e8d is indeed the solution, I will file an RHBZ (for the RHEL kernel) to backport it. Will do. Thanks for all your work on this! -- Bruce ___ edk2-devel mailing

Re: [edk2] [PATCH] ArmPlatformPkg: fixups for 64-bit mailbox pointers

2016-04-19 Thread Duran, Leo
Leif, Please see my reply below. Leo. > -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Monday, April 18, 2016 5:39 AM > To: Ard Biesheuvel > Cc: Duran, Leo ; edk2-devel@lists.01.org > Subject: Re: [PATCH] ArmPlatformPkg: fixups for 64-bit mailbox pointe

Re: [edk2] [PATCH] StdLib: add __noreturn attribute to __assert

2016-04-19 Thread Michael Zimmermann
Hi it doesn't break building with VC++ because __noreturn isn't from GCC, it's from edk2. https://github.com/tianocore/edk2/blob/master/StdLib/Include/sys/EfiCdefs.h#L211 Michael On Tue, Apr 19, 2016 at 4:21 PM, Daryl McDaniel wrote: > Michael, > > In your patch, you use GCC's __noreturn attri

Re: [edk2] [Patch] ShellPkg: Enhance ping to select the interface automatically

2016-04-19 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Wu, Jiaxin > Sent: Monday, April 18, 2016 6:52 PM > To: edk2-devel@lists.01.org > Cc: David Van Arnem ; Bhupesh Sharma > ; Carsey, Jaben ; Ye, > Ting ; Fu, Siyuan > Subject: [Patch] ShellPkg: Enhance ping to select the interface au

[edk2] .bss section is being combined with the .data section in GCC linker script

2016-04-19 Thread Stephen Polkowski
Hello, Could someone explain why the .bss section is being combined with the .data section in the GCC linker script in the EDK2? This will force all uninitialized variables to take space in the output .efi file. This seems really wasteful. GccBase.lds /* * The alignment of the .data section

Re: [edk2] [PATCH 5/5] ArmPkg/ArmDmaLib: do not remap arbitrary memory regions as uncached

2016-04-19 Thread Ard Biesheuvel
On 19 April 2016 at 16:55, Ard Biesheuvel wrote: > In the DmaMap () operation, if the region to be mapped happens to be > aligned to the Cache Writeback Granule (CWG) (whose value is typically > 64 or 128 bytes and 2 KB maximum), we remap the memory as uncached. > > Since remapping memory occurs a

[edk2] [PATCH 3/3] Platforms/ARM/Juno: Add support for ACPI 6.0 LPI(Low Power Idle) states

2016-04-19 Thread Sudeep Holla
ACPI 6.0 introduced LPI(Low Power Idle) states that provides an alternate method to describe processor idle states. LPI extensions leverages the processor container device(again introduced in ACPI 6.0) allowing to express which parts of the system are affected by a given LPI state. It defines the

[edk2] [PATCH 1/3] Platforms/ARM/Juno: Fix IO window translate offset in PCIe root complex

2016-04-19 Thread Sudeep Holla
XPress-RICH3 PCIe driver initialises the root complex with the same source and target address for IO window which makes translate offset 0. This patch fixes the translate offset for the IO window in Juno PCIe root complex ACPI table. Contributed-under: TianoCore Contribution Agreement 1.0 Cc: Ard

[edk2] [PATCH 2/3] Platforms/ARM/Juno: limit ACPI support to v5.0 and higher

2016-04-19 Thread Sudeep Holla
The ACPI spec predates the AARCH64 architecture by 5 versions, so there is no point in supporting anything below v5.0. So set the PCD that controls the ACPI table generation to the appropriate value. Based on the commit e0692789058e ("ArmVirtPkg/ArmVirtQemu: limit ACPI support to v5.0 and higher")

[edk2] [PATCH 4/5] ArmPkg/ArmDmaLib: reject consistent DMA mappings of cached memory

2016-04-19 Thread Ard Biesheuvel
DmaMap () operations of type MapOperationBusMasterCommonBuffer should return a mapping that is coherent between the CPU and the device. For this reason, the API only allows DmaMap () to be called with this operation type if the memory to be mapped was allocated by DmaAllocateBuffer (), which in thi

[edk2] [PATCH 2/5] ArmPkg/ArmDmaLib: consistently use 'gCacheAlignment - 1' as alignment mask

2016-04-19 Thread Ard Biesheuvel
We manage to use both an AND operation with 'gCacheAlignment - 1' and a modulo operation with 'gCacheAlignment' in the same compound if statement. Since gCacheAlignment is a global of which the compiler cannot guarantee that it is a power of two, simply use the AND version, and use it against the b

[edk2] [PATCH 3/5] ArmPkg/ArmDmaLib: interpret GCD attributes as a bit field

2016-04-19 Thread Ard Biesheuvel
Comparing a GCD attribute field directly against EFI_MEMORY_UC and EFI_MEMORY_WT is incorrect, since it may have other bits set as well which are not related to the cacheability of the region. So instead, test explicitly against the flags EFI_MEMORY_WB and EFI_MEMORY_WT, which must be set if the re

[edk2] [PATCH 1/5] ArmPkg/ArmDmaLib: deal with NULL return value of UncachedAllocatePages ()

2016-04-19 Thread Ard Biesheuvel
The allocation function UncachedAllocatePages () may return NULL, in which case our implementation of DmaAllocateBuffer () should return EFI_OUT_OF_RESOURCES rather than silently ignoring the NULL value and returning EFI_SUCCESS. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-b

[edk2] [PATCH 5/5] ArmPkg/ArmDmaLib: do not remap arbitrary memory regions as uncached

2016-04-19 Thread Ard Biesheuvel
In the DmaMap () operation, if the region to be mapped happens to be aligned to the Cache Writeback Granule (CWG) (whose value is typically 64 or 128 bytes and 2 KB maximum), we remap the memory as uncached. Since remapping memory occurs at page granularity, while the buffer and the CWG may be muc

Re: [edk2] [PATCH] StdLib: add __noreturn attribute to __assert

2016-04-19 Thread Daryl McDaniel
Michael, In your patch, you use GCC's __noreturn attribute at the end of the line. Does GCC have a version of this that precedes the declaration? Since use of noreturn would apply in a significant number of cases, I was hoping to be able to produce a NORETURN macro to be used at the beginning

Re: [edk2] Request HTTPS branch in edk2-staging

2016-04-19 Thread El-Haj-Mahmoud, Samer
Looks good. No objection here Reviewed-by: Samer El-Haj-Mahmoud -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Wu, Jiaxin Sent: Tuesday, April 19, 2016 3:55 AM To: El-Haj-Mahmoud, Samer ; edk2-devel@lists.01.org Subject: Re: [edk2] Request H

Re: [edk2] OVMF: hang when booting Linux via GRUB after driver allocates 2GB above 4GB

2016-04-19 Thread Laszlo Ersek
On 04/18/16 23:23, Matt Fleming wrote: > On Mon, 18 Apr, at 04:33:42PM, Laszlo Ersek wrote: >> >> I think at this point I'll copy Matt :) , and ask you to reproduce >> the issue with a fresh upstream kernel (most recent Linux release, >> or even fresh git). If it reproduces, then it's an upstream k

Re: [edk2] [PATCH v2] OvmfPkg: AcpiPlatformDxe: Don't enable unsupported PCI attributes

2016-04-19 Thread Laszlo Ersek
On 04/18/16 22:17, Jordan Justen wrote: > Reviewed-by: Jordan Justen Reviewed-by: Laszlo Ersek Tested-by: Laszlo Ersek Commit 90bb4c577d055e7ac6f0488b21885f65617eec82. Thank you for the contribution, Volker! Laszlo > On 2016-04-18 12:51:30, Volker RĂ¼melin wrote: >> Current code in PciEnableD

[edk2] UEFI HTTP Boot Device Path and DNS

2016-04-19 Thread Michael Chang
Background (And also Question): The PXE Base Code Protocol is no longer produced if selecting to boot from UEFI HTTP device from boot menu. Does anyone here know why or is it's just a problem of my environment (I'm using OVMF from openSUSE) ? That made the work to support it in grub2 more complic

Re: [edk2] [PATCH v2 0/2] Get default value from CallBack function for OrderedList

2016-04-19 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: Bi, Dandan > Sent: Monday, April 18, 2016 3:47 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming; Dong, Eric > Subject: [PATCH v2 0/2] Get default value from CallBack function for > OrderedList > > For a question, it can retrive defau

Re: [edk2] [Patch] BaseTools: Add mixed PCD support feature

2016-04-19 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Yonghong Zhu > Sent: Tuesday, April 12, 2016 11:10 AM > To: edk2-devel@lists.01.org > Subject: [edk2] [Patch] BaseTools: Add mixed PCD support feature > > Problem state

Re: [edk2] [patch] MdeModulePkg: refine codes of iSCSI driver.

2016-04-19 Thread Zhang, Lubo
Thank you for your series comments, will update them while checking in. Thanks Lubo -Original Message- From: Ye, Ting Sent: Tuesday, April 19, 2016 5:02 PM To: Zhang, Lubo ; edk2-devel@lists.01.org Cc: Fu, Siyuan ; Wu, Jiaxin Subject: RE: [edk2] [patch] MdeModulePkg: refine codes of i

Re: [edk2] [patch] MdeModulePkg: refine codes of iSCSI driver.

2016-04-19 Thread Ye, Ting
Same comment for this patch for MdeModulePkg: Suggest to update below comment for EFI_ACCESS_DENIED from + @retval EFI_ACCESS_DENIED The interface was not removed because the interface is +still being used by a driver. To: : The protocol could not be removed

Re: [edk2] [patch] NetworkPkg: refine codes of iSCSI driver.

2016-04-19 Thread Ye, Ting
Suggest to update below comment for EFI_ACCESS_DENIED from + @retval EFI_ACCESS_DENIED The interface was not removed because the interface is +still being used by a driver. To: : The protocol could not be removed from the handle since its interfaces are being

Re: [edk2] Request HTTPS branch in edk2-staging

2016-04-19 Thread Wu, Jiaxin
I would like to attach this feature branch "Readme.MD" file first before the new branch created. And also update one of the request patch since the latest version commit in CryptoPkg is in conflict with this fix. The patch subject is <[Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable '

[edk2] [PATCH v2 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

2016-04-19 Thread Jiaxin Wu
v2: * The latest version commit in CryptoPkg is in conflict with this fix. So, this update resolve the conflict issue. * Remove NULL 'time' parameter fix and make it as a standalone commit. * Update OpensslTlsLib implementation. * convert OpensslTlsLib.uni from UTF-16 to UTF-8. This patch is used

Re: [edk2] GenFw error 3000 when using prebuilt *.a lib

2016-04-19 Thread Gao, Liming
Michael: I check GenFw logic. If no symbols, it will impact the relocation in the converted PE COFF image. I suggest you continue to check why the generated ELF image misses symbols. Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Beh

[edk2] [PATCH] ArmPlatformPkg/PrePi: allow unicore version to be used on MP hardware

2016-04-19 Thread Ard Biesheuvel
When combining UEFI firmware built from Tianocore with ARM Trusted Firmware running in EL3, it is the responsibility of ATF that only a single core enters the UEFI firmware in EL2, and the remaining cores are released directly to the OS via PSCI SMC calls. In this case, we don't need the MpCore fl

[edk2] KeyEnroll Utility source code

2016-04-19 Thread Vinod Mudaliar
HI , I am looking for KeyEnroll utiltiy source code. I tried searching for it in edk2 github, but could not find it. Thank you, Vinod ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel