Re: [edk2] [PATCH V3 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

On 26 January 2017 at 14:36, Leif Lindholm wrote: > On Fri, Jan 20, 2017 at 05:10:45PM +0530, Bhupesh Sharma wrote: >> ARM TZASC-380 IP provides a mechanism to split memory regions being >> protected via it into eight equal-sized sub-regions. A bit-setting >> allows the corresponding subregion to

[edk2] [PATCH v3 1/2] OvmfPkg: dynamic defaults for PcdCpuSmmApSyncTimeout, PcdCpuSmmSyncMode

Move the platform-specific default values for these PCDs from the [PcdsFixedAtBuild] / [PcdsFixedAtBuild.X64] sections to the [PcdsDynamicDefault] section. Cc: Jordan Justen Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=230 Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-

[edk2] [PATCH v3 0/2] OvmfPkg: broadcast SMIs and dynamically revert to traditional AP sync mode

Previous version (v2): . In this version (again), OVMF negotiates the broadcast SMI feature with QEMU dynamically. If the feature is available, OVMF selects it, and also re-sets the UefiCpuPkg PCDs that are related to BSP-AP sync

[edk2] [PATCH v3 2/2] OvmfPkg/SmmControl2Dxe: select broadcast SMI if available

When writing to IO port 0xB2 (ICH9_APM_CNT), QEMU by default injects an SMI only on the VCPU that is writing the port. This has exposed corner cases and strange behavior with edk2 code, which generally expects a software SMI to affect all CPUs at once. We've experienced instability despite the fact

Re: [edk2] [PATCH V3 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

On Tue, Jan 31, 2017 at 09:48:11AM +, Ryan Harkin wrote: > On 26 January 2017 at 14:36, Leif Lindholm wrote: > > On Fri, Jan 20, 2017 at 05:10:45PM +0530, Bhupesh Sharma wrote: > >> ARM TZASC-380 IP provides a mechanism to split memory regions being > >> protected via it into eight equal-sized

Re: [edk2] [PATCH V3 1/1] ArmPlatformPkg/TZASC: Allow specifying subregions to be disabled

On 31 January 2017 at 11:44, Leif Lindholm wrote: > On Tue, Jan 31, 2017 at 09:48:11AM +, Ryan Harkin wrote: >> On 26 January 2017 at 14:36, Leif Lindholm wrote: >> > On Fri, Jan 20, 2017 at 05:10:45PM +0530, Bhupesh Sharma wrote: >> >> ARM TZASC-380 IP provides a mechanism to split memory re

[edk2] [staging/edk2-test PATCH 0/6] Fix XCODE and GCC build issues

This series fixes a number of XCODE and GCC build compatibility issues with the edk2-test branch. It also updates Readme.md with the Linux build instructions and addresses a few typos. Cc: Andrew Fish Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [staging/edk2-test PATCH 5/6] TestCasePkg: Fix XCODE compatibility issues

From: Michael Kinney Cc: Michael Kinney Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Andrew Fish --- TestCasePkg/Timer/ArchTimerBBTestConformance.c | 5 ++--- TestCasePkg/Timer/ArchTimerBBTestFunction.c| 8 2 files changed, 6 inser

[edk2] [staging/edk2-test PATCH 4/6] TestFrameworkPkg: Fix GCC compatibility issues

Cc: Andrew Fish Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Michael Kinney --- TestFrameworkPkg/Eftp/ComponentName.c | 4 + TestFrameworkPkg/Eftp/EftpDriver.h | 5 + TestFrameworkPkg/Eftp/EftpMain.c

[edk2] [staging/edk2-test PATCH 2/6] TestFrameworkPkg: Fix XCODE compatibility issues

Cc: Michael Kinney Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Andrew Fish --- TestFrameworkPkg/Eftp/EftpCommon.h | 2 +- TestFrameworkPkg/Eftp/EftpMain.c | 4 +- TestFrameworkPkg/Eftp/EftpMain.h

[edk2] [staging/edk2-test PATCH 3/6] TestFrameworkPkg: Fix VS2015 compatibility issues

Cc: Andrew Fish Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Michael Kinney --- TestFrameworkPkg/Sct/Execute/ExecuteSupport.c | 17 - TestFrameworkPkg/Sct/UI/KeyFunction.c | 14 +++--- 2 files changed, 7 insertions

[edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

* Add Supreeth Venkatesh to list of branch owners * Clean up environment variable for Windows builds * Add Linux Build Instructions Cc: Andrew Fish Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Michael Kinney --- Readme.md | 50 ++

[edk2] [staging/edk2-test PATCH 6/6] TestCasePkg: Fix GCC compatibility issues

Cc: Andrew Fish Cc: Supreeth Venkatesh Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Michael Kinney --- TestCasePkg/Timer/ArchTimerBBTestConformance.c | 2 ++ TestCasePkg/Timer/ArchTimerBBTestFunction.c| 4 +++ TestCasePkg/Timer/ArchTimerBBTestMain.c| 4 +

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

nothing of importance(and most likely not related to this commit because it has been there before), but you misspelled staging as 'stagaing' :) On Tue, Jan 31, 2017 at 7:45 PM, Michael Kinney wrote: > * Add Supreeth Venkatesh to list of branch owners > * Clean up environment variable for Windows

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

On Tue, 2017-01-31 at 20:29 +0100, Michael Zimmermann wrote: > nothing of importance(and most likely not related to this commit > because it has been there before), but you misspelled staging as > 'stagaing' :) > 1. Yes. I noticed it too. Will fix this with a new patch. 2. I see that there are ad

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

> On Jan 31, 2017, at 2:41 PM, Supreeth Venkatesh > wrote: > > On Tue, 2017-01-31 at 20:29 +0100, Michael Zimmermann wrote: >> nothing of importance(and most likely not related to this commit >> because it has been there before), but you misspelled staging as >> 'stagaing' :) >> > 1. Yes. I no

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

On Tue, 2017-01-31 at 15:02 -0800, Andrew Fish wrote: > > > On Jan 31, 2017, at 2:41 PM, Supreeth Venkatesh > > @arm.com> wrote: > > > > On Tue, 2017-01-31 at 20:29 +0100, Michael Zimmermann wrote: > > > nothing of importance(and most likely not related to this commit > > > because it has been t

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

Supreeth, Thanks for the feedback. Not sure why I did not see those errors. I am using GCC49 on Fedora. I will make sure that warning is enabled and will fix all the instances. Mike > -Original Message- > From: Supreeth Venkatesh [mailto:supreeth.venkat...@arm.com] > Sent: Tuesday, Jan

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

The command I used is this: build -a IA32 -a X64 -t GCC49 -p TestFrameworkPkg/TestFrameworkPkg.dsc However, I noticed that my compiler is newer (on Ubuntu) gcc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609 Perhaps, these errors are not visible in GCC4.9. Thanks, Supreeth -Original Message--

Re: [edk2] [staging/edk2-test PATCH 1/6] edk2-test: Update Readme.md

Michael, I will fix Readme.md and the few additional GCC issues that Supreeth found. Thanks, Mike > -Original Message- > From: Michael Zimmermann [mailto:sigmaepsilo...@gmail.com] > Sent: Tuesday, January 31, 2017 11:29 AM > To: Kinney, Michael D > Cc: edk2-devel@lists.01.org; Andrew

[edk2] [PATCH 1/1] ShellPkg/UefiShellLevel2CommandsLib: Fix "cp" for GCC systems

On GCC built UEFIs, the "cp" command fails with "The source and destination are the same". The root cause is that StrniCmp failed to detect a case- insensitive string difference due to the InternalCharToUpper function being improperly declared. Fix is to add missing EFIAPI token to match the orig