Re: [edk2] [PATCH] ArmPkg/CpuDxe: Disable interrupt before restoring context

2015-08-23 Thread Heyi Guo
On 08/17/2015 05:52 PM, Ard Biesheuvel wrote: On 13 August 2015 at 05:10, Heyi Guoheyi@linaro.org wrote: Interrupt must be disabled before we storing ELR and other system registers, or else ELR will be overridden by interrupt reentrance. This bug is critical as we may get occasional

Re: [edk2] [Patch 6/8] OptionRomPkg/OvmfPkg: Remove BltLib::BltConfigure API

2015-08-23 Thread Jordan Justen
On 2015-08-23 20:09:23, Ni, Ruiyu wrote: Jordan, reply embedded in the mail. Mike, Jordan proposed to add a new parameter Configuration in BltConfigure, instead of removing this API, to eliminate the global variables in library implementation. I think it's also a good solution and can

Re: [edk2] [patch] MdeModulePkg:Refine the code in HiiDatabaseDxe

2015-08-23 Thread Dong, Eric
Reviewed-by: Eric Dong eric.d...@intel.com -Original Message- From: Bi, Dandan Sent: Monday, August 24, 2015 1:28 PM To: Dong, Eric; Qiu, Shumin; edk2-devel@lists.01.org Subject: [patch] MdeModulePkg:Refine the code in HiiDatabaseDxe Check the local variable 'NameSpace' before use it.

[edk2] [patch] MdeModulePkg:Refine the code in HiiDatabaseDxe

2015-08-23 Thread Dandan Bi
Check the local variable 'NameSpace' before use it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi dandan...@intel.com --- MdeModulePkg/Universal/HiiDatabaseDxe/ConfigKeywordHandler.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git