[edk2] OVMF cannot boot in QEMU 1.6.2

Hi, I found OVMF built from the latest source tree cannot boot in QEMU 1.6.2 (Ubuntu14.04.1). Is it related to the changes made to OvmfPkg? Boot log as following: -- SecCoreStartupWithStack(0xFFFCC000, 0x818000) Register PPI Notify: DCD0BE23-9586-40F4-B643-06522CED4EDE Install PPI:

Re: [edk2] [Patch] OptionRomPkg: Remove redundant included header file defintion in Ax88772.h and Ax88772b.h files

Reviewed-by: Ye Ting -Original Message- From: jiaxinwu [mailto:jiaxin...@intel.com] Sent: Monday, July 06, 2015 1:37 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch] OptionRomPkg: Remove redundant included header file defintion in Ax88772.h and Ax88772b.h files Contribu

[edk2] [PATCH] UefiCpuPkg S3Resume2Pei: Resolve issue brought by r17744

AsciiStrCpyS (PerfData->Token, PERF_TOKEN_SIZE, Token); The above using of AsciiStrCpyS will cause ASSERT if Token is longer than PerfData->Token. Therefore, AsciiStrnCatS is used here to resolve the issue. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed-by

Re: [edk2] [Patch] MdeModulePkg: Update UefiBootManagerLib to support HTTP boot option creation

Looks good to me. Reviewed-by: Fu Siyuan -Original Message- From: Ni, Ruiyu Sent: Tuesday, July 07, 2015 1:38 PM To: edk2-devel@lists.sourceforge.net Cc: Ni, Ruiyu; Fu, Siyuan Subject: [Patch] MdeModulePkg: Update UefiBootManagerLib to support HTTP boot option creation Contributed-unde

[edk2] [Patch] MdeModulePkg: Update UefiBootManagerLib to support HTTP boot option creation

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Siyuan Fu --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 12 +++- MdeModulePkg/Library/UefiBootManagerLib/InternalBm.h | 1 + 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/Md

[edk2] [PATCH 1/5] IntelFrameworkModulePkg GenericBdsLib: Resolve issue brought by r17733

StringBuffer1 = AllocateCopyPool ( MAX_STRING_LEN * sizeof (CHAR16), L"Configuration changed. Reset to apply it Now." ); The above using of AllocateCopyPool() will read contents out of the scope of the constant string. Potential risk for the co

[edk2] [PATCH 3/5] IntelFrameworkModulePkg BootMaint: Resolve issue brought by r17736

Str = AllocateCopyPool (MaxLen * sizeof (CHAR16), Str1); The above using of AllocateCopyPool() will read contents out of the scope of Str1. Potential risk for Str1 allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Hao Wu Reviewed

[edk2] [PATCH 2/5] IntelFrameworkModulePkg BdsDxe: Resolve issue brought by r17735

StrCatS (StrPercent, sizeof (StrPercent) / sizeof (CHAR16), TmpStr); The above using of StrCatS will cause ASSERT if TmpStr is longer than StrPercent. Therefore, StrnCatS is used here to resolve the issue. Similar scenario is for: StrCatS (StrTotalMemory, StrTotalMemorySize / sizeof (CHAR16), Tmp

[edk2] [PATCH 0/5] Resolve issues brought by r17732-r17739

Commit r17732 to r17739 used safe string functions to replace the old ones in IntelFrameworkModulePkg. However, these changes broght the following two issues: 1. AllocateCopyPool (AllocationSize, Buffer) Some usage of AllocateCopyPool() will read contents out of the scope of 'Buffer'. Potential r

[edk2] [PATCH 5/5] IntelFrameworkModulePkg DeviceMngr: Resolve issue brought by r17738

String = AllocateCopyPool (BufferLen, L"MAC:"); The above using of AllocateCopyPool() will read contents out of the scope of the constant string. Potential risk for the constant string allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-b

[edk2] [PATCH 4/5] IntelFrameworkModulePkg BootMngr: Resolve issue brought by r17737

HelpString = AllocateCopyPool (HelpSize, L"Device Path : "); The above using of AllocateCopyPool() will read contents out of the scope of the constant string. Potential risk for the constant string allocated at the boundary of memory region. Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] [PATCH] Fix type cast of ~0 in CheckSignatureListFormat: SigHeaderSize is UINT32

Bruce: The patch is good Reviewed-by： Chao Zhang Thanks & Best regards Chao Zhang -Original Message- From: Bruce Cran [mailto:br...@cran.org.uk] Sent: Tuesday, July 07, 2015 10:22 AM To: Zhang, Chao B; edk2-devel@lists.sourceforge.net Cc: Bruce Cran Subject: [PATCH] Fix type cas

Re: [edk2] [PATCH] MdeModulePkg: Use StrnCpyS instead of StrCpyS to indicate the copy length from the source.

Reviewed-by: Liming Gao -Original Message- From: Qiu, Shumin Sent: Tuesday, July 07, 2015 10:14 AM To: edk2-devel@lists.sourceforge.net Cc: Qiu, Shumin; Gao, Liming Subject: [edk2][PATCH] MdeModulePkg: Use StrnCpyS instead of StrCpyS to indicate the copy length from the source. Contrib

Re: [edk2] XHCI question

Baranee, I found XhciPei also have same issue and here are the fixes on these two modules. If you are ok with my changes, could you sign Reviewed-by? Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Feng Tian Thanks Feng -Original Message- From: Anbazhagan, Bara

Re: [edk2] [PATCH v2 02/24] PcAtChipsetPkg: remove PciHostBridgeDxe

Jordan, Laszlo, I personally strongly disagree using PCD to split code to make the code "common". PCD is a very fashion word in EDKII world but most of the time it downgrades to C macro. Do you think one piece of "common" code heavily using macro is better than two pieces of code? I doubt that!

Re: [edk2] [PATCH] Fix type cast of ~0 in CheckSignatureListFormat: SigHeaderSize is UINT32

On Mon, Jul 06, 2015 at 08:11:50PM -0600, Bruce Cran wrote: > -if (mSupportSigItem[Index].SigHeaderSize != ((UINTN) ~0) && > +if (mSupportSigItem[Index].SigHeaderSize != ((UINT32) ~0) && Sorry, please ignore this - I forgot to add the Contributed-under and Signed-off-by headers.

[edk2] [PATCH] Fix type cast of ~0 in CheckSignatureListFormat: SigHeaderSize is UINT32

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bruce Cran --- SecurityPkg/VariableAuthenticated/RuntimeDxe/AuthService.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SecurityPkg/VariableAuthenticated/RuntimeDxe/AuthService.c b/SecurityPkg/VariableAu

[edk2] [PATCH] Fix typo of 'determines' in ShellPkg

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bruce Cran --- ShellPkg/Application/Shell/Shell.c | 4 ++-- ShellPkg/Application/Shell/Shell.h | 2 +- ShellPkg/Include/Protocol/EfiShellEnvironment2.h | 6 +++--- Shel

[edk2] [PATCH] MdeModulePkg: Use StrnCpyS instead of StrCpyS to indicate the copy length from the source.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Qiu Shumin Reviewed-by: Hao Wu CC: Liming Gao --- MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.c | 4 ++-- MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c| 2 +- 2 files changed, 3 ins

[edk2] [PATCH] Fix type cast of ~0 in CheckSignatureListFormat: SigHeaderSize is UINT32

--- SecurityPkg/VariableAuthenticated/RuntimeDxe/AuthService.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SecurityPkg/VariableAuthenticated/RuntimeDxe/AuthService.c b/SecurityPkg/VariableAuthenticated/RuntimeDxe/AuthService.c index 566c5e2..885b916 100644 --- a/SecurityPk

Re: [edk2] XHCI question

Do you have any other feedback on these code changes? If not, could you please commit those changes. Thanks. > > -Original Message- > > From: Tian, Feng [mailto:feng.t...@intel.com] > > Sent: Friday, June 26, 2015 1:28 AM > > To: Anbazhagan, Baraneedharan; Eric Wittmayer; > > edk2-devel@l

Re: [edk2] [PATCH] MdeModulePkg/PartitionDxe: Fix media probe

Martin, Please go ahead to check in the patch first. I will send new patch for review later when I have bandwidth to look into this further. Reviewed-by: Ruiyu Ni Thanks, Ray From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: Monday, July 6, 2015 10:12 PM To: Ni, Ruiyu; edk2-devel@lists

Re: [edk2] [Patch 0/7] Remove Ip4ConfigDxe module and related guid definitions from related packages.

Looks good. Reviewed-by: Ye Ting -Original Message- From: jiaxinwu [mailto:jiaxin...@intel.com] Sent: Tuesday, July 07, 2015 9:28 AM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [Patch 0/7] Remove Ip4ConfigDxe module and related guid definitions from related packages. Those p

Re: [edk2] [PATCH 0/6] fix runtime relocation of disjoint PE/COFF images

Ard: So, in this case, PE image space will not be continuous. Its code and data section are in the different region. I know RelocatImage is for the absolute address data access. But, I am still curious how PE image handles the relative address data access? For patch 1 & 2, I agree those cha

Re: [edk2] [Patch 0/7] Remove Ip4ConfigDxe module and related guid definitions from related packages.

Hi all, Ip4ConfigDxe driver is deprecated in UEFI 2.5, so we will not support original Ip4Config Protocol, which is replace by Ip4Config2 Protocol integrated in Ip4Dxe driver. Related pkgs(MdeModulePkg, Nt32Pkg, ArmPlatformPkg, ArmVirtPkg, EmulatorPkg, OvmfPkg, Vlv2TbltDevicePkg) need to remo

[edk2] [Patch 1/7] MdeModulePkg: Remove Ip4ConfigDxe module and related guid definition from MdeModulePkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- MdeModulePkg/Include/Guid/Ip4ConfigHii.h | 25 - MdeModulePkg/Include/Guid/NicIp4ConfigNvData.h | 70 -- MdeModulePkg/MdeModulePkg.dec | 6 - MdeModulePkg/MdeModulePkg.dsc

[edk2] [Patch 3/7] OvmfPkg: Remove Ip4ConfigDxe module from OvmfPkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- OvmfPkg/OvmfPkgIa32.dsc| 1 - OvmfPkg/OvmfPkgIa32.fdf| 1 - OvmfPkg/OvmfPkgIa32X64.dsc | 1 - OvmfPkg/OvmfPkgIa32X64.fdf | 1 - OvmfPkg/OvmfPkgX64.dsc | 1 - OvmfPkg/OvmfPkgX64.fdf | 1 - 6 files c

[edk2] [Patch 7/7] Vlv2TbltDevicePkg: Remove Ip4ConfigDxe module from Vlv2TbltDevicePkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- Vlv2TbltDevicePkg/PlatformPkg.fdf | 1 - Vlv2TbltDevicePkg/PlatformPkgGcc.fdf| 1 - Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc | 1 - Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 1 - Vlv2TbltDevicePkg/Platfor

[edk2] [Patch 2/7] Nt32Pkg: Remove Ip4ConfigDxe module from Nt32Pkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- Nt32Pkg/Nt32Pkg.dsc | 1 - Nt32Pkg/Nt32Pkg.fdf | 1 - 2 files changed, 2 deletions(-) diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc index a8e5590..330c25b 100644 --- a/Nt32Pkg/Nt32Pkg.dsc +++ b/Nt32Pkg/N

[edk2] [Patch 6/7] EmulatorPkg: Remove Ip4ConfigDxe module from EmulatorPkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- EmulatorPkg/EmulatorPkg.dsc | 1 - EmulatorPkg/EmulatorPkg.fdf | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/EmulatorPkg/EmulatorPkg.dsc b/EmulatorPkg/EmulatorPkg.dsc index b16fcac..e0c616

[edk2] [Patch 0/7] Remove Ip4ConfigDxe module and related guid definitions from related packages.

Those patches are used to remove Ip4ConfigDxe module and related guid definitions from related packages. jiaxinwu (7): MdeModulePkg: Remove Ip4ConfigDxe module and related guid definition from MdeModulePkg. Nt32Pkg: Remove Ip4ConfigDxe module from Nt32Pkg. OvmfPkg: Remove Ip4ConfigDxe m

[edk2] [Patch 4/7] ArmPlatformPkg: Remove Ip4ConfigDxe module from ArmPlatformPkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- ArmPlatformPkg/ArmJunoPkg/ArmJuno.fdf | 1 - ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-CTA15-A7.fdf| 1 - ArmPlatformPkg/ArmVExpressPkg/ArmVExpress-CTA9x4.fdf | 1 - ArmPla

[edk2] [Patch 5/7] ArmVirtPkg: Remove Ip4ConfigDxe module from ArmVirtPkg.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: jiaxinwu --- ArmVirtPkg/ArmVirt.dsc.inc | 1 - ArmVirtPkg/ArmVirtQemu.fdf | 1 - 2 files changed, 2 deletions(-) diff --git a/ArmVirtPkg/ArmVirt.dsc.inc b/ArmVirtPkg/ArmVirt.dsc.inc index 7ec0de4..6f47d8e 100644 --- a/ArmVirt

Re: [edk2] XHCI with ARM-v8 based platform

> On Jul 6, 2015, at 4:54 PM, Tian, Feng wrote: > > You can refer to Omap35xxPkg\PciEmulation which produces PCI_IO protocol on > arm board and try to build this driver and EDKII xHCI driver in your firmware > to see if it works. > > DMA issue mentioned by Andrew is the most likely to bring b

Re: [edk2] [PATCH v2 02/24] PcAtChipsetPkg: remove PciHostBridgeDxe

On 2015-07-06 15:10:45, Laszlo Ersek wrote: > On 06/28/15 01:36, Jordan Justen wrote: > > On 2015-06-25 21:26:20, Ni, Ruiyu wrote: > >> Jordan, > >> Simply using PCD to split the code is very straightforward. Another > >> approach is to introduce a new library class with carefully defined > >> libr

Re: [edk2] [Patch] PcRtc: Fix PcRtcInit() to not clear RegisterB.AIE always

Reviewed-by: Feng Tian -Original Message- From: Ni, Ruiyu Sent: Monday, July 6, 2015 2:50 PM To: edk2-devel@lists.sourceforge.net Cc: Tian, Feng Subject: RE: [Patch] PcRtc: Fix PcRtcInit() to not clear RegisterB.AIE always Fix comments as below: PcRtc: Fix PcRtcInit() to use correct val

Re: [edk2] XHCI with ARM-v8 based platform

You can refer to Omap35xxPkg\PciEmulation which produces PCI_IO protocol on arm board and try to build this driver and EDKII xHCI driver in your firmware to see if it works. DMA issue mentioned by Andrew is the most likely to bring bugs as in most cases IA platforms are 1:1 mapping(identical) f

[edk2] [PATCH V2 1/4] Add "TtyTerm" terminal type to TerminalDxe

This patch a adds new terminal type, TtyTerm, to TerminalDxe. This terminal type provides a place to add support for various *nix terminals that don't behave like standard VT terminals. The goal is to 'just work' with as many terminals as possible, rather than properly emulating any one specific

[edk2] [PATCH V2 0/4] Add TtyTerm terminal type

This patchset adds a new terminal type "TtyTerm", that better handles the typical Linux terminal (xterm/rxvt/gnome terminal/tmux/screen/etc.) These often treat backspace/delete differently than the existing EDK2 terminal types expect, particularly in emulated environments where the emulated serial

[edk2] [PATCH V2 2/4] Treat ASCII 0x7F as backspace for TtyTerm terminals

Treat ASCII 0x7F as backspace, rather than delete, for TTY terminals. This better matches the default Linux terminal settings that are used when connecting to a simulated platform using xterm or a similar terminal program. Signed-off-by: Roy Franz Contributed-under: TianoCore Contribution Agreem

[edk2] [PATCH V2 4/4] Add PCD for selecting terminal type at build time

From: Laszlo Ersek Add a fixed pointer PCD to allow build-time selection of VT100 or TTY terminal type. The default remains VT100 emulation. Add support for building the ARM QEMU platforms with the TTY terminal with the "-D TTY_TERMINAL" build option. Contributed-under: TianoCore Contribution A

[edk2] [PATCH V2 3/4] Accept VT220 DEL and function keys for TTY terminal type

Accept the VT220 escape code [3~ as backspace for TtyTerm terminals. This is sent by many Linux terminals by default. Also accept VT220 function keys F1-F12, and VT100 F1-F4 keys as these are commonly sent by Linux terminals. The VT220 escape codes are longer, and variable length so a new state i

Re: [edk2] [PATCH 0/9] multiple packages: save S3 state at EndOfDxe

All, On 06/27/15 03:08, Laszlo Ersek wrote: > In > , > Jiewen proposed: > >> And in the future, I believe we could remove EFI_ACPI_S3_SAVE_PROTOCOL >> protocol completely. (This is Framework 0.9 defined API, not PI >> defi

Re: [edk2] [PATCH v2 02/24] PcAtChipsetPkg: remove PciHostBridgeDxe

On 06/28/15 01:36, Jordan Justen wrote: > On 2015-06-25 21:26:20, Ni, Ruiyu wrote: >> Jordan, >> Simply using PCD to split the code is very straightforward. Another >> approach is to introduce a new library class with carefully defined >> library APIs so that different platforms can use one common

Re: [edk2] XHCI with ARM-v8 based platform

> On Jul 6, 2015, at 11:47 AM, Thomas Jacob > wrote: > > Hi All, > > I am working on bringing up EDK-II on a ARM-v8 based platform. I was > wondering if somebody has already brought up XHCI controllers on such similar > platforms. If so, did you face any problems ? > Thomas, There are two

Re: [edk2] [PATCH] ArmVirtPkg: adapt ArmVirtXen build to system memory end global variable

On 07/06/15 22:25, Ard Biesheuvel wrote: > This fixes the ArmVirtXen build that was broken by r17835, which adds > a global variable mSystemMemoryEnd which is shared between a module > and a library it depends on. > > Add the same global variable to the relocatable PrePi used by ArmVirtXen. > > C

Re: [edk2] Qestion about how to debug OVMF on QEMU

On 07/06/15 20:25, B Cran wrote: >> On Jul 6, 2015, at 11:33 AM, Laszlo Ersek > > wrote: >> >> Hm... memories memories memories... Yes. Are you aware of this: >> >> http://people.freebsd.org/~wpaul/edk2/README.txt > > I’d read bits of it a few months ago, but it seemed ra

Re: [edk2] Qestion about how to debug OVMF on QEMU

Of all the gin joints in all the towns in all the world, B Cran had to walk into mine at 11:27:01 on Monday 06 July 2015 and say: > (re-sending since I sent it from an address that’s not subscribed to > edk2-devel) > > On Jul 6, 2015, at 12:25 PM, B Cran wrote: > > On Jul 6, 2015, at 11:33 AM,

[edk2] [PATCH] ArmVirtPkg: adapt ArmVirtXen build to system memory end global variable

This fixes the ArmVirtXen build that was broken by r17835, which adds a global variable mSystemMemoryEnd which is shared between a module and a library it depends on. Add the same global variable to the relocatable PrePi used by ArmVirtXen. Contributed-under: TianoCore Contribution Agreement 1.0

Re: [edk2] [PATCH V2 2/2] Describe submission of a patch authored by someone else

On 07/01/15 10:22, Ard Biesheuvel wrote: > On 30 June 2015 at 06:12, Roy Franz wrote: >> Add a description of how to describe the authorship of a patch that >> is submitted by someone other than the original author. >> Add mention of git format-patch options for generating >> more useful patch nam

Re: [edk2] URGENT SVN screwup [Re: [PATCH 0/5] MdeModulePkg: clean up Properties Table implementation and adapt it to AArch64]

On 07/02/15 09:30, Ard Biesheuvel wrote: > On 2 July 2015 at 09:29, Jordan Justen wrote: >> On 2015-07-02 00:10:40, Jordan Justen wrote: >>> On 2015-07-01 23:51:19, Ard Biesheuvel wrote: On 2 July 2015 at 08:38, Ard Biesheuvel wrote: > Liming, Jordan, Jiewen. > > I appeared to ha

Re: [edk2] Replacement EDK2 email list coming soon

On 2015-07-04 15:56:05, B Cran wrote: > > On May 4, 2015, at 2:21 PM, Peterson, Joe > > wrote: > > Due to community feedback, a new mailing list is being set up to replace > > this one. The new list will be hosted on Lists.01.org and should be more > > stable and consistent than this one. > > >

[edk2] XHCI with ARM-v8 based platform

Hi All, I am working on bringing up EDK-II on a ARM-v8 based platform. I was wondering if somebody has already brought up XHCI controllers on such similar platforms. If so, did you face any problems ? Regards, Thomas -- D

Re: [edk2] Qestion about how to debug OVMF on QEMU

(re-sending since I sent it from an address that’s not subscribed to edk2-devel) On Jul 6, 2015, at 12:25 PM, B Cran wrote: > On Jul 6, 2015, at 11:33 AM, Laszlo Ersek > wrote: > > Hm... memories memories memories... Yes. Are you aware of this: > > http://people.free

[edk2] [PATCH 3/3] ArmVExpressPkg: use PSCI for system reset only on AARCH64 platforms

The PSCI specification covers both ARM and AARCH64, however, the ARM Trusted Firmware (ATF) reference implementation is only available for AARCH64, and PSCI firmware is not widely available for ARM platforms. So use the EfiResetSystemLib implementation that uses PSCI calls only on AARCH64, and rev

[edk2] [PATCH 2/3] ArmVExpressPkg: use ArmVExpressSysConfigRuntimeLib by default

Instead of using a NULL implementation of ArmPlatformSysConfigLib for DXE_RUNTIME_DRIVER modules, which prevents them from accessing system control registers even at boot time, use the new implementation that only switches into a non-functional mode at runtime, and operates as before otherwise. Co

[edk2] [PATCH 0/3] ArmVExpressPkg: avoid PSCI reboot/poweroff on 32-bit platforms

This fixes an issue reported by Ryan Harkin where 32-bit ARM platforms without PSCI compliant firmware cannot be rebooted or powered off successfully at boot time. This is caused by a change which aimed to prevent system register accesses at runtime. These registers are not virtually remapped, so

[edk2] [PATCH 1/3] ArmPlatformPkg/ArmVExpressPkg: add ArmPlatformSysConfigLib for runtime

This adds a ArmPlatformSysConfigLib implementation that is usable by DXE_RUNTIME_DRIVER modules. Since the system registers that this library encapsulates are not usable at runtime, this driver allows access to those registers only at boot time. Contributed-under: TianoCore Contribution Agreement

Re: [edk2] [PATCH 1/2] BaseTools: AArch64: use explicit linker scripts

On 29 June 2015 at 23:22, Cohen, Eugene wrote: > Reviewed-by: Eugene Cohen > Hello Eugene, These patches have now been merged. Unfortunately, I failed to add your R-b: this was unintentional, but there is little I can about it now. Thanks again for the review, Regards, Ard. > -Original

Re: [edk2] Qestion about how to debug OVMF on QEMU

On 07/05/15 01:20, B Cran wrote: > On Tue, May 26, 2015 at 3:38 AM, Laszlo Ersek > wrote: > > Bruce, what happened to the nice summary you had put up under: > > http://edk2.bluestop.org/w/tianocore/debugging-with-gdb/ > > (announced in > >

Re: [edk2] ArmPlatformPkg: TC2 reset/shutdown broken in Tianocore EDK2 tree

On 6 July 2015 at 17:32, Ard Biesheuvel wrote: > On 6 July 2015 at 18:11, Ryan Harkin wrote: > > The following 3 patches break reboot/shutdown from within UEFI on TC2 and > > probably all 32-bit ARM Ltd platforms: > > > > ee171cc 2015-05-08 ArmVExpressPkg: restrict ArmVExpressSysConfigLib to >

Re: [edk2] [PATCH] ArmPlatformPkg/ArmVExpressPkg: use 64 KB section alignment for runtime drivers

Reviewed-by: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 06 July 2015 17:40 To: edk2-devel@lists.sourceforge.net; Olivier Martin Cc: leif.lindh...@linaro.org; Ard Biesheuvel Subject: [PATCH] ArmPlatformPkg/ArmVExpressPkg: use 64 KB secti

[edk2] [PATCH] ArmPlatformPkg/ArmVExpressPkg: use 64 KB section alignment for runtime drivers

This adds the 64 KB alignment overlay linker script to the linker command line of DXE_RUNTIME_DRIVER modules built for AARCH64. This makes these modules compatible with the new Properties Table feature by aligning the .text and .data sections to 64 KB. Contributed-under: TianoCore Contribution Agr

Re: [edk2] ArmPlatformPkg: TC2 reset/shutdown broken in Tianocore EDK2 tree

On 6 July 2015 at 18:11, Ryan Harkin wrote: > The following 3 patches break reboot/shutdown from within UEFI on TC2 and > probably all 32-bit ARM Ltd platforms: > > ee171cc 2015-05-08 ArmVExpressPkg: restrict ArmVExpressSysConfigLib to SEC > and DXE_DRIVER [Ard Biesheuvel] > c889bf2 2015-05

[edk2] [PATCH] MdeModulePkg/FvSimpleFileSystemDxe: Support file opening with no '.efi'

FvSimpleFileSystem adds '.efi' to the EFI application and drivers filenames even through this extension is not present in the real filename of the EFI module. In the current behaviour, it would not be possible to open an EFI application using FvSimpleFileSystem if the extension has been omitted in

[edk2] ArmPlatformPkg: TC2 reset/shutdown broken in Tianocore EDK2 tree

The following 3 patches break reboot/shutdown from within UEFI on TC2 and probably all 32-bit ARM Ltd platforms: ee171cc 2015-05-08 ArmVExpressPkg: restrict ArmVExpressSysConfigLib to SEC and DXE_DRIVER [Ard Biesheuvel] c889bf2 2015-05-08 ArmVExpressPkg: avoid the use of ArmVExpressSysConf

[edk2] [PATCH] MdePkg/ImageAuthentication.h: Fixed ARM toolchain error

ARM Toolchain raised the error: "last line of file ends without a newline" Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Olivier Martin --- MdePkg/Include/Guid/ImageAuthentication.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MdePkg/Include/Guid

Re: [edk2] [PATCH 2/2] ArmVirtPkg: build runtime drivers with 64 KB section alignment

Reviewed-By: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 29 June 2015 19:37 To: edk2-devel@lists.sourceforge.net; ler...@redhat.com; Olivier Martin; af...@apple.com; eug...@hp.com; jiewen@intel.com Cc: yingke.d@intel.com; matt.f

Re: [edk2] [PATCH 1/2] BaseTools: AArch64: use explicit linker scripts

Reviewed-By: Olivier Martin -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 29 June 2015 19:37 To: edk2-devel@lists.sourceforge.net; ler...@redhat.com; Olivier Martin; af...@apple.com; eug...@hp.com; jiewen@intel.com Cc: yingke.d@intel.com; matt.f

Re: [edk2] [PATCH] MdeModulePkg/PartitionDxe: Fix media probe

Are you still debating on my simple patch I sent 3 months ago that definitely fixes an issue? From: Ni, Ruiyu [mailto:ruiyu...@intel.com] Sent: 06 May 2015 03:37 To: Olivier Martin; edk2-devel@lists.sourceforge.net Cc: Ronald Cron Subject: RE: [PATCH] MdeModulePkg/PartitionDxe: Fix media probe M

[edk2] [PATCH v2] MdeModulePkg:Modify the incorrect DestStr length in safe string functions

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong Reviewed-by: Liming Gao --- MdeModulePkg/Universal/HiiDatabaseDxe/Database.c | 2 +- MdeModulePkg/Universal/HiiDatabaseDxe/String.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)

Re: [edk2] [PATCH 1/6] MdePkg/BasePeCoffLib: remove redundant PeCoffLoaderEx.c for AARCH64

Reviewed-By: Olivier Martin Harry had to do a similar clean-up when he added support for LLVM: https://github.com/ARM-software/edk2/commit/54cb35d53e691a856381efa2c0756d7bd7e2347a -Original Message- From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: 03 July 2015 10:40 To: ed

[edk2] [Patch] UefiCpuPkg/Library/CpuExceptionHandlerLib: Add exception type decoder

Add exception type decoder to print exception name string beside print exception type value. The exception names are from IA32 SDM. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jeff Fan Reviewed-by: Feng Tian --- .../CpuExceptionHandlerLib/CpuExceptionCommon.c| 48

Re: [edk2] [PATCH 0/6] fix runtime relocation of disjoint PE/COFF images

On 6 July 2015 at 08:25, Gao, Liming wrote: > Ard: > Have you GIT branch for those changes? Then, I can easily review those > changes. > Sure I pushed it here: https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/pecoff-runtime-relocation Thanks, Ard. > -Origin

Re: [edk2] [PATCH v2] ShellPkg: Remove UefiHandleParsingLib and Ip4Config Protocol dependency.

Reviewed-by: Qiu Shumin -Original Message- From: jiaxinwu [mailto:jiaxin...@intel.com] Sent: Monday, July 06, 2015 1:12 PM To: edk2-devel@lists.sourceforge.net Subject: [edk2] [PATCH v2] ShellPkg: Remove UefiHandleParsingLib and Ip4Config Protocol dependency. Contributed-under: TianoC