Re: [edk2] [PATCH v2] MdeModulePkg/MemoryStatusCode: Expose the DXE memory status code table.

2016-06-27 Thread Gao, Liming
Cinnamon: I have two minor comments on usage. Install System table means PRODUCES. Others are good to me. Reviewed-by: Liming Gao 1. StatusCodeHandlerRuntimeDxe.inf gMemoryStatusCodeRecordGuid ## SOMETIMES_CONSUMES ## HOB and ConfigurationTable

Re: [edk2] SecurityPkg/Tcg: Fix bug that prevented SubmitCommand buffers from being Max size.

2016-06-27 Thread Yao, Jiewen
Good catch. Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Bret Barkelew > Sent: Tuesday, June 28, 2016 10:29 AM > To: edk2-devel@lists.01.org > Subject: [edk2] SecurityPkg/Tcg: Fix bug that prevented

[edk2] SecurityPkg/Tcg: Fix bug that prevented SubmitCommand buffers from being Max size.

2016-06-27 Thread Bret Barkelew
SecurityPkg/Tcg: Fix bug that prevented SubmitCommand buffers from being Max size. SubmitCommand() was checking the buffer size for ">=" Max size. This would cause code to fail with "EFI_INVALID_PARAMETER" if a buffer was passed that was the "max" size as indicated by the GetCapability()

[edk2] [PATCH v2] MdeModulePkg: Skip registering BootManagerMenu if its FFS is not found

2016-06-27 Thread Sunny Wang
This is a enhancement to support the case when platform firmware doesn’t support Boot Manager Menu. For now, if BootManagerMenu FFS can not be retrieved from FV, BDS core code will still register a boot option for it. Then, this non-functional boot option will still be booted by user's request

Re: [edk2] [Patch 000/351] Convert EDK II core packages to NASM for IA32/X64

2016-06-27 Thread Gao, Liming
Hi, all I pushed all patches. If you meet with any issues, please raise it to me. Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Gao, Liming > Sent: Thursday, June 23, 2016 2:23 PM > To: Laszlo Ersek ;

Re: [edk2] Driver Health HII menu missing in OVMF master

2016-06-27 Thread Gao, Liming
MdeModulePkg\Universal\DriverHealthManagerDxe\DriverHealthManagerDxe.inf is missing after new MdeModulePkg/Universal/BdsDxe/BdsDxe.inf is applied. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Bruce Cran > Sent: Tuesday, June 28, 2016

Re: [edk2] [PATCH] [edk2-staging/HTTPS-TLS][PATCH]: Centralize TLS var cert name and guid

2016-06-27 Thread Wu, Jiaxin
That's fine. I will create one patch directly:). Thanks. Jiaxin > -Original Message- > From: Palmer, Thomas [mailto:thomas.pal...@hpe.com] > Sent: Tuesday, June 28, 2016 12:51 AM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Zimmer, Vincent

Re: [edk2] [RFC V3.1] Proposal to organize packages into directories

2016-06-27 Thread Leif Lindholm
On Mon, Jun 27, 2016 at 03:50:02PM +0100, Ryan Harkin wrote: > >> Silicon > >> Arm > >> ArmPkg > > > > Much of ArmPkg should probably move across into UefiCpuPkg and > > MdeModulePkg over time, but this is probably the best thing to do for > > the restructuring. > > > >>

[edk2] Driver Health HII menu missing in OVMF master

2016-06-27 Thread Bruce Cran
I just noticed that between UDK2015 and master branches in https://github.com/tianocore/edk2 the Driver Health display in "Device Manager" has disappeared. Was its removal deliberate - has it been moved elsewhere? -- Bruce ___ edk2-devel mailing

Re: [edk2] [PATCH] [edk2-staging/HTTPS-TLS][PATCH]: Centralize TLS var cert name and guid

2016-06-27 Thread Palmer, Thomas
I can create the patch if you tell me where to put everything. Or if you are like me, may be easier for you to just code it up. Either way is fine Thomas -Original Message- From: Wu, Jiaxin [mailto:jiaxin...@intel.com] Sent: Sunday, June 26, 2016 8:38 PM To: Palmer, Thomas

Re: [edk2] [RFC V3.1] Proposal to organize packages into directories

2016-06-27 Thread Ryan Harkin
On 26 June 2016 at 16:44, Leif Lindholm wrote: > So, I managed to miss this thread for the longest time ... sorry about > the late comments. > > On Fri, Jun 17, 2016 at 10:51:47PM +, Kinney, Michael D wrote: >> Hello, >> >> Please review this one. I missed a few

Re: [edk2] Adding a memory region to GCD on AARCH64?

2016-06-27 Thread Achin Gupta
Hi Laszlo, On Thu, Jun 23, 2016 at 04:38:03PM +0200, Laszlo Ersek wrote: > On 06/23/16 16:19, Achin Gupta wrote: > > Hi Laszlo, > > > > On Wed, Jun 22, 2016 at 09:56:11PM +0200, Laszlo Ersek wrote: > >> On 06/22/16 20:53, Achin Gupta wrote: > >>> Hi All, > >>> > >>> I having some trouble trying

Re: [edk2] [PATCH] [MdePkg ] Additional Atapi.h definitions complying Industry Standard specifications.

2016-06-27 Thread Gao, Liming
Got it. We will review this change. > -Original Message- > From: Anandakrishnan Loganathan [mailto:anandakrishn...@ami.com] > Sent: Monday, June 27, 2016 5:12 PM > To: Gao, Liming ; edk2-devel@lists.01.org > Subject: RE: [PATCH] [MdePkg ] Additional Atapi.h

Re: [edk2] [PATCH] [MdePkg ] Additional Atapi.h definitions complying Industry Standard specifications.

2016-06-27 Thread Anandakrishnan Loganathan
Hi Gao, Sorry.. I will check the problem in creating the patch my side. Meanwhile I have attached the Atapi.h for your reference Thanks! -Original Message- From: Gao, Liming [mailto:liming@intel.com] Sent: 27 June 2016 14:33 To: Anandakrishnan Loganathan; edk2-devel@lists.01.org

Re: [edk2] [PATCH] [MdePkg ] Additional Atapi.h definitions complying Industry Standard specifications.

2016-06-27 Thread Gao, Liming
Anandakrishnan: Your patch format is Unicode. I can't apply it. Could you create the normal GIT patch or directly send the updated Atapi.h? > -Original Message- > From: Anandakrishnan Loganathan [mailto:anandakrishn...@ami.com] > Sent: Monday, June 27, 2016 2:02 PM > To: Gao, Liming

Re: [edk2] [PATCH] SecurityPkg: AuthVariableLib: Cache UserPhysicalPresent in AuthVariableLib

2016-06-27 Thread Zeng, Star
Reviewed-by: Star Zeng On 2016/6/27 15:12, Yao, Jiewen wrote: Reviewed-by: jiewen@intel.com -Original Message- From: Zhang, Chao B Sent: Monday, June 27, 2016 3:06 PM To: edk2-devel@lists.01.org Cc: Yao, Jiewen ; Zeng, Star

[edk2] [PATCH v2] MdeModulePkg/MemoryStatusCode: Expose the DXE memory status code table.

2016-06-27 Thread Cinnamon Shia
Let data of DXE memory status code can be used by other modules. 1. Save the address of DXE memory status code table to DxeConfigurationTable. 2. Save the address of SMM memory status code table to SmmConfigurationTable. 3. Move RUNTIME_MEMORY_STATUSCODE_HEADER to its public header file.

Re: [edk2] [PATCH] SecurityPkg: AuthVariableLib: Cache UserPhysicalPresent in AuthVariableLib

2016-06-27 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: Zhang, Chao B > Sent: Monday, June 27, 2016 3:06 PM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen ; Zeng, Star ; > Zhang, Chao B > Subject: [PATCH]

[edk2] [PATCH] SecurityPkg: AuthVariableLib: Cache UserPhysicalPresent in AuthVariableLib

2016-06-27 Thread Zhang, Chao B
AuthVariableLib is updated to cache the UserPhysicalPresent state to global variable. This avoids calling PlatformSecureLib during runtime and makes PhysicalPresent state consistent during one boot. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Chao Zhang

Re: [edk2] [PATCH] MdeModulePkg/UsbBusDxe: Fixed USB descriptor length check

2016-06-27 Thread Evgeny Yakovlev
Great, thanks! > 27 июня 2016 г., в 7:51, Tian, Feng написал(а): > > Thanks, Yakovlev. > > If there is no objection, I will commit and push these two patches to git > repo with your RB and mine. > > Thanks > Feng > > From: Evgeny Yakovlev

Re: [edk2] [PATCH] [MdePkg ] Additional Atapi.h definitions complying Industry Standard specifications.

2016-06-27 Thread Anandakrishnan Loganathan
Attached the Patch with the mail -Original Message- From: Gao, Liming [mailto:liming@intel.com] Sent: 27 June 2016 09:41 To: Anandakrishnan Loganathan; edk2-devel@lists.01.org Subject: RE: [PATCH] [MdePkg ] Additional Atapi.h definitions complying Industry Standard specifications.