Re: [edk2] [Patch 0/3] Enable Pyrite 2.0 for opal driver.

2018-05-07 Thread Dong, Eric
Hi Jiewen, I do the basic functionality test base on the opal UI. Also I request the QA to do the full regression test and no issue found. Also will include unit test info in later patches. Thanks for your comments. Thanks, Eric -Original Message- From: Yao, Jiewen Sent: Tuesday,

Re: [edk2] [Patch 0/3] Enable Pyrite 2.0 for opal driver.

2018-05-07 Thread Yao, Jiewen
Hi Eric Would you please add more detail on what unit test has been done for this patch? Thank you Yao Jiewen > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Eric > Dong > Sent: Wednesday, May 2, 2018 8:17 PM > To: edk2-devel@lists.01.org;

Re: [edk2] [PATCH v2] MdeModulePkg/AcpiPlatformDxe: Unload after execution.

2018-05-07 Thread Zeng, Star
Thanks for the update. Reviewed-by: Star Zeng and pushed at d9476a7ef1476621c14a7e632b8b8fc3ca419f61. Thanks, Star -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Marvin H?user Sent: Monday, May 7, 2018 5:08 PM To:

[edk2] [Patch 2/2] SecurityPkg/TcgSTorage*Lib.h: Add referenced spec information.

2018-05-07 Thread Eric Dong
Add link and version info for the referenced spec. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Eric Dong Cc: Hao Wu Cc: Jiewen yao --- SecurityPkg/Include/Library/TcgStorageCoreLib.h | 7 ++-

[edk2] [Patch 2/2] SecurityPkg/TcgSTorage*Lib.h: Add referenced spec information.

2018-05-07 Thread Eric Dong
Add link and version info for the referenced spec. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Eric Dong Cc: Hao Wu Cc: Jiewen yao --- SecurityPkg/Include/Library/TcgStorageCoreLib.h | 7 ++-

[edk2] [Patch 1/2] MdePkg/TcgStorage*.h: Add referenced spec information.

2018-05-07 Thread Eric Dong
Add link and version info for the referenced spec. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Eric Dong Cc: Hao Wu Cc: Jiewen yao --- MdePkg/Include/IndustryStandard/TcgStorageCore.h | 7 ++-

[edk2] [Patch 0/2] Add referenced spec information.

2018-05-07 Thread Eric Dong
Add link and version info for the referenced spec. Cc: Hao Wu Cc: Jiewen yao Eric Dong (2): MdePkg/TcgStorage*.h: Add referenced spec information. SecurityPkg/TcgSTorage*Lib.h: Add referenced spec information.

[edk2] error while removing a file

2018-05-07 Thread Carsey, Jaben
Does anyone else see this issue? I see this, but cannot guarantee to see it so I cannot be sure about fixing it. I have a patch that I would like to work with others seeing the issue on... -Jaben My error looks like this, but I think the exact text is dependent on python version:

[edk2] [PATCH] CryptoPkg/CrtLibSupport: add secure_getenv() stub function

2018-05-07 Thread Laszlo Ersek
The Fedora distro ships a modified OpenSSL 1.1.0 package stream. One of their patches calls the secure_getenv() C library function. We already have a stub for getenv(); it applies trivially to secure_getenv() as well. Add the secure_getenv() stub so that edk2 can be built with Fedora's OpenSSL

Re: [edk2] [PATCH v2 17/17] BaseTools/AutoGen: Update header file for MM modules.

2018-05-07 Thread Supreeth Venkatesh
Thanks Liming. With new changes in this file and three-way merge, I have messed this up. I will resend v3 of this again. Supreeth -Original Message- From: Gao, Liming Sent: Monday, May 7, 2018 10:11 AM To: Supreeth Venkatesh ;

Re: [edk2] [PATCH] ShellPkg/UefiShellCommandLib: Do not error-exit when PlatformLang is missing.

2018-05-07 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] > Sent: Saturday, May 05, 2018 5:07 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu > Subject:

Re: [edk2] [PATCH] ShellPkg/UefiShellLevel2CommandsLib: Fix ls on entirely empty directories.

2018-05-07 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Marvin Häuser > Sent: Saturday, May 05, 2018 5:11 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu

Re: [edk2] [PATCH] ShellPkg/UefiShellLevel2CommandsLib: Support unspecified local Timezone.

2018-05-07 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Marvin Häuser > Sent: Saturday, May 05, 2018 1:23 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ni, Ruiyu

[edk2] OVMF Logo

2018-05-07 Thread Дмитрий Степанов
Hello! Is it possible to tune Logo's behavior in OVMF? Now it overlaps Windows Logo during system boot ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel

Re: [edk2] [PATCH v2 17/17] BaseTools/AutoGen: Update header file for MM modules.

2018-05-07 Thread Gao, Liming
Hi, This change should map to PiMm.h. I see the version 1 patch uses PiMm.h. +SUP_MODULE_MM_STANDALONE : ["PiSmm.h", "Library/BaseLib.h", "Library/DebugLib.h", "Library/StandaloneMmDriverEntryPoint.h"], +SUP_MODULE_MM_CORE_STANDALONE : ["PiSmm.h", "Library/BaseLib.h",

Re: [edk2] [PATCH] MdePkg/BaseLib: Globally include ChkStk.c

2018-05-07 Thread Gao, Liming
Marvin: In VS compiler, what case will trig this intrinsic function? Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Marvin > H?user > Sent: Saturday, May 5, 2018 10:25 PM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D

Re: [edk2] [PATCH v2 1/2] MdePkg/UefiLib: Add 'OUT' decorator where necessary.

2018-05-07 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Marvin > H?user > Sent: Saturday, May 5, 2018 10:24 PM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Gao,

Re: [edk2] [PATCH v2 2/2] IntelFrameworkPkg/FrameworkUefiLib: Add 'OUT' decorator where necessary.

2018-05-07 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Marvin > H?user > Sent: Saturday, May 5, 2018 10:24 PM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Gao,

Re: [edk2] [PATCH] MdePkg/PeiServicesLib: Decorate 'PpiDescriptor' as OPTIONAL for LocatePpi().

2018-05-07 Thread Gao, Liming
Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Marvin > H?user > Sent: Saturday, May 5, 2018 10:25 PM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Gao,

[edk2] [PATCH v2] MdeModulePkg/AcpiPlatformDxe: Unload after execution.

2018-05-07 Thread Marvin Häuser
AcpiPlatformDxe solely performs one-time tasks and does not expose any services or create any events. Hence it can safely be unloaded after the Entry Point execution by returning an error code. V2: Return EFI_REQUEST_UNLOAD_IMAGE. Contributed-under: TianoCore Contribution Agreement 1.1

Re: [edk2] [PATCH] MdeModulePkg/AcpiPlatformDxe: Unload after execution.

2018-05-07 Thread Marvin Häuser
Sorry, I did not realize that error code was a thing. Sure, I will submit a V2 soon, thanks! Best regards, Marvin > -Ursprüngliche Nachricht- > Von: Zeng, Star > Gesendet: Montag, 7. Mai 2018 03:52 > An: marvin.haeu...@outlook.com; edk2-devel@lists.01.org > Cc:

Re: [edk2] [PATCH] UefiCpuPkg/SecMain: Add NORETURN decorator to SecStartup().

2018-05-07 Thread Dong, Eric
Hi Marvin, Sorry I missed this change. I will follow up to check in the changes. Thanks, Eric -Original Message- From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] Sent: Saturday, May 5, 2018 9:18 PM To: edk2-devel@lists.01.org Cc: Dong, Eric ; Laszlo Ersek

Re: [edk2] [Patch] BaseTools: Correct the variable name

2018-05-07 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: Monday, May 07, 2018 2:04 PM >To: edk2-devel@lists.01.org >Subject: [edk2] [Patch] BaseTools: Correct the variable name > >the

Re: [edk2] [Patch] NetworkPkg/NetworkPkg.dsc: Add the instance of library class [SafeIntLib].

2018-05-07 Thread Ye, Ting
Reviewed-by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Friday, May 4, 2018 11:53 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Long, Qin ; Bi, Dandan ; Wu, Jiaxin

Re: [edk2] [Patch] NetworkPkg/NetworkPkg.dsc: Add the instance of library class [SafeIntLib].

2018-05-07 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Friday, May 4, 2018 11:53 AM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Long, > Qin ; Bi, Dandan ;