Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

2016-03-10 Thread Long, Qin
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of David > Woodhouse > Sent: Thursday, March 10, 2016 10:01 AM > To: Long, Qin ; Wu, Jiaxin ; > edk2-de...@ml01.01.org > Cc: Ye, Ting ; Fu, Siyuan > Subject: Re: [edk2] [P

Re: [edk2] [PATCH] CryptoPkg/OpensslLib: Convert saved opensslconf.h to DOS line endings

2016-03-10 Thread Long, Qin
Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: David Woodhouse [mailto:david.woodho...@intel.com] > Sent: Thursday, March 10, 2016 3:15 AM > To: edk2-de...@ml01.01.org > Cc: Long, Qin > Subject: [PATCH] CryptoPkg/Ope

Re: [edk2] [Patch 2/6] CryptoPkg: Add OpensslTlsLib module to enable 'openssl\ssl'

2016-03-10 Thread Long, Qin
David, This patch series should be based on the old version, before the back-porting of upstreaming patch was done. We should have no need to add the extra patches on OpenSSL now for OpensslTlsLib build now. Best Regards & Thanks, LONG, Qin > -Original Message- > F

Re: [edk2] [GIT PULL] CryptoPkg/OpensslLib: Fix CRLF breakage in process_files.sh

2016-03-05 Thread Long, Qin
> -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Sunday, March 6, 2016 3:06 AM > To: David Woodhouse > Cc: Long, Qin ; edk2-devel@lists.01.org > > Subject: Re: [edk2] [GIT PULL] CryptoPkg/OpensslLib: Fix CRLF breakage in >

Re: [edk2] [Patch 6/6] Nt32Pkg: Enable Nt32Pkg platform HTTPS boot feature.

2016-02-25 Thread Long, Qin
I cannot recall the history about this flag. Then that will be better. I think it's good to remove the flag for those library, and put them there directly, if no any build performance impacts. Best Regards & Thanks, LONG, Qin From: Ni, Ruiyu Sent: Friday, February 26, 2016 11:29 AM

Re: [edk2] [Patch 6/6] Nt32Pkg: Enable Nt32Pkg platform HTTPS boot feature.

2016-02-25 Thread Long, Qin
One consideration before should be build performance, to use the flag to turn on / off those modules, since openssl build need quite a long times. Best Regards & Thanks, LONG, Qin From: Ni, Ruiyu Sent: Friday, February 26, 2016 11:17 AM To: Wu, Jiaxin; Long, Qin; El-Haj-Mahmoud, Samer

Re: [edk2] [Patch 6/6] Nt32Pkg: Enable Nt32Pkg platform HTTPS boot feature.

2016-02-25 Thread Long, Qin
ould be tolerant to produce the correct makefile even if we enabled both two flags. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Wu, Jiaxin > Sent: Friday, February 26, 2016 9:59 AM > To: El-Haj-Mahmoud, Samer; Ye, Ting; edk2-devel@lists.01.org > Cc: Ni,

Re: [edk2] [PATCH 0/4] free(NULL) and realloc(NULL, size) conformance improvements

2016-02-24 Thread Long, Qin
Great. I will sync-up these changes and follow the windows / VS toolchain validations. Best Regards & Thanks, LONG, Qin > -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Thursday, February 25, 2016 10:05 AM > To: Laszlo Ersek; edk2-dev

Re: [edk2] [PATCH 0/4] free(NULL) and realloc(NULL, size) conformance improvements

2016-02-24 Thread Long, Qin
Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, February 25, 2016 5:14 AM > To: edk2-devel-01 > Cc: Cecil Sheng; Cinnamon Shia; Daryl McDaniel; David Woodhouse; Dong, &g

Re: [edk2] [Patch] CryptoPkg/OpensslLib: Upgrade OpenSSL version to 1.0.2f

2016-02-24 Thread Long, Qin
Laszlo, this is really cool. Thanks for the analysis and root-cause this. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, February 25, 2016 3:39 AM > To: David Woodhouse; Long, Qin > Cc

Re: [edk2] [Patch] CryptoPkg/OpensslLib: Upgrade OpenSSL version to 1.0.2f

2016-02-19 Thread Long, Qin
> -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Friday, February 19, 2016 5:00 PM > To: Long, Qin; Laszlo Ersek > Cc: Ye, Ting; edk2-de...@ml01.01.org > Subject: Re: [edk2] [Patch] CryptoPkg/OpensslLib: Upgrade OpenSSL version >

Re: [edk2] [Patch] CryptoPkg/OpensslLib: Upgrade OpenSSL version to 1.0.2f

2016-02-18 Thread Long, Qin
-by-step. Yeah, also will do more follow-ups about the remaining opens... Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > David Woodhouse > Sent: Friday, February 19, 2016 7:01 AM > To

Re: [edk2] [Patch] CryptoPkg/OpensslLib: Upgrade OpenSSL version to 1.0.2f

2016-02-18 Thread Long, Qin
eave me in the rooms. :-) Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo > Ersek > Sent: Friday, February 19, 2016 1:08 AM > To: David Woodhouse ; Long, Qin > Cc: Ye, Ting ;

Re: [edk2] [Patch] CryptoPkg/OpensslLib: Upgrade OpenSSL version to 1.0.2f

2016-02-18 Thread Long, Qin
ed on my last patch, which will not change any build process, and just to catch the latest release for some requirements. (David, apology for my late feedback to your patch post.) Let me know if any concerns. Best Regards & Thanks, LONG, Qin > -Original Message- > Fro

Re: [edk2] Linking Problem when using OpenSslLib for 32 bits build

2016-01-24 Thread Long, Qin
;cc = RShiftU64 (aa, (UINTN) xxx)" Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Rodrigo Caropreso > Sent: Sunday, January 24, 2016 12:25 AM > To: edk2-devel@lists.01.org >

Re: [edk2] [PATCH] CryptoPkg: Fix function qsort for non 32-bit machines

2016-01-19 Thread Long, Qin
Authur, the patch is OK for me to use "int" here, since CrtWrapper.c was already using the mixed type style to meet the compiler. Thanks for catching this. Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Burigo, Arthur Cripp

Re: [edk2] [PATCH] CryptoPkg: Fix function qsort for non 32-bit machines

2016-01-19 Thread Long, Qin
Thanks for the detailed analysis. The fix makes sense to me. And yes, I also prefer to use INT32 as Laszlo's comment (since we don't consider ILP64 data model), to keep the consistent style in this function declaration. Best Regards & Thanks, LONG, Qin > -Original Me

Re: [edk2] [Patch] edk2: Update the maintainer list.

2016-01-18 Thread Long, Qin
It's good for me. Reviewed-by: Qin Long > -Original Message- > From: Fu, Siyuan > Sent: Tuesday, January 19, 2016 9:46 AM > To: Wu, Jiaxin; edk2-devel@lists.01.org > Cc: Tian, Hot; Li, Ruth; Long, Qin > Subject: RE: [edk2] [Patch] edk2: Update the maint

Re: [edk2] [PATCH] SecurityPkg: SecureBootConfigDxe: Fix string typo

2016-01-14 Thread Long, Qin
Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Zhang, Chao B > Sent: Friday, January 15, 2016 9:39 AM > To: edk2-de...@ml01.01.org > Cc: Long, Qin; Fu, Siyuan; Zhang, Chao B > Subject: [PATCH] SecurityPkg: SecureBootConf

Re: [edk2] [PATCH] SecurityPkg: SecureBootConfigDxe: Change KEY_TRANS_SECURE_BOOT_MODE value

2016-01-13 Thread Long, Qin
Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Zhang, Chao B > Sent: Thursday, January 14, 2016 9:01 AM > To: edk2-de...@ml01.01.org > Cc: Fu, Siyuan; Zh

Re: [edk2] [PATCH] SecurityPkg: TcgDxe, Tcg2Dxe, TrEEDxe: New PCD for TCG event log and TCG2 final event log area

2016-01-08 Thread Long, Qin
Looks good to me. Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Zhang, Chao B > Sent: Friday, January 08, 2016 3:34 PM > To: edk2-de...@ml01.01.org > Cc: Yao,

Re: [edk2] [PATCH] CryptoPkg: fix build support under RVCT

2015-12-17 Thread Long, Qin
Looks I missed this one. It's OK for me. Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Friday, December 18, 2015 12:11 AM > To: edk2-devel@lists.01.org; Long, Qin

Re: [edk2] [PATCH v2 04/20] CryptoPkg: Convert all .uni files to utf-8

2015-12-07 Thread Long, Qin
Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Justen, Jordan L > Sent: Saturday, December 5, 2015 4:13 AM > To: edk2-devel@lists.01.org > Cc: Justen, Jordan L; Long, Qin > Subject: [PATCH v2 04/20] CryptoPkg: Convert a

Re: [edk2] [PATCH] CryptoPkg: remove global variable 'timeval' from OpenSslSupport.h

2015-12-04 Thread Long, Qin
Sorry, I missed this patch. Yes, this looks good. Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Saturday, December 5, 2015 12:45 AM > To: edk2-devel@lists.01.org; Lon

Re: [edk2] [PATCH 0/4] CryptoPkg: fix ARM and RVCT issues

2015-12-04 Thread Long, Qin
For the series, reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Thursday, December 3, 2015 6:51 PM > To: edk2-devel@lists.01.org; eug...@hp.com; Long, Qin; dw...@infrade

Re: [edk2] [PATCH] CryptoPkg/OpensslLib: upgrade to openssl-1.0.2e

2015-12-04 Thread Long, Qin
ase. We may be able to remove whole EDKII-openssl-xxx.patch, and have some native EFI configuration, INF generation, etc, if all patches were integrated into openssl head. :-) Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun

Re: [edk2] [PATCH] CryptoPkg/OpensslLib: upgrade to openssl-1.0.2e

2015-12-04 Thread Long, Qin
David, do you think it's valuable to feedback to OpenSSL community? Personally, I think the old behavior (before 1.0.2e) can be acceptable when two data (embedded and detached data) were supplied. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Ard Biesheuv

Re: [edk2] [PATCH 0/4] CryptoPkg: fix ARM and RVCT issues

2015-12-03 Thread Long, Qin
Eugene, As Ard's said, this patch series should cover / supersede your old fixes for RVCT build, which was pended by me. :-) Could you please help to validate them? Since I have no RVCT environment. Best Regards & Thanks, LONG, Qin > -Original Message- > From:

Re: [edk2] [PATCH 4/4] CryptoPkg/OpensslLib: ignore more false positive warnings for RVCT

2015-12-03 Thread Long, Qin
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard > Biesheuvel > Sent: Thursday, December 3, 2015 11:13 PM > To: David Woodhouse > Cc: edk2-devel@lists.01.org; Long, Qin; Cohen, Eugene > Subject: Re: [edk2] [PATCH 4/4]

Re: [edk2] [PATCH 3/4] CryptoPkg/OpensslLib: comment out unused code

2015-12-03 Thread Long, Qin
> -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Thursday, December 3, 2015 8:52 PM > To: David Woodhouse > Cc: edk2-devel@lists.01.org; Cohen, Eugene; Long, Qin > Subject: Re: [PATCH 3/4] CryptoPkg/OpensslLib: comment out unu

Re: [edk2] [Patch 5/6] CryptoPkg: Disable VS2015 warning C4311 in OpensslLib

2015-12-02 Thread Long, Qin
Looks good. Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Gao, Liming > Sent: Thursday, December 03, 2015 1:27 PM > To: edk2-devel@lists.01.org > Cc: Long, Qin > Subject: [Patch 5/6] CryptoPkg: Disable VS2015 warni

Re: [edk2] [PATCH 0/5] Enable ECR1263 Customized Secure Boot

2015-12-01 Thread Long, Qin
The patch series is good to me. Please make the patch subjects and description more clear, as Laszlo's comments. And also need to check the RetVal descriptions in your new-added functions. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Laszlo Er

Re: [edk2] [PATCH v2 4/4] CryptoPkg: add softfloat dependency for ARM

2015-11-29 Thread Long, Qin
Reviewed-by: Qin Long Thanks, Ard. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Friday, November 27, 2015 11:22 PM > To: edk2-devel@lists.01.org; leif.lindh...@linaro.org; ler...@redhat.com;

Re: [edk2] [PATCH] CryptoPkg: Fix ARM RVCT 4 build

2015-11-25 Thread Long, Qin
> -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Wednesday, November 25, 2015 4:40 PM > To: Long, Qin; Cohen, Eugene; Ard Biesheuvel; Leif Lindholm; Leif Lindholm > Cc: edk2-devel@lists.01.org > Subject: Re: [PATCH] CryptoPkg: Fix ARM R

Re: [edk2] CryptoPkg: OpenSSL build issue with RVCT

2015-11-24 Thread Long, Qin
> -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Wednesday, November 25, 2015 12:56 AM > To: Cohen, Eugene; Leif Lindholm; Leif Lindholm > Cc: David Woodhouse; Long, Qin; edk2-devel@lists.01.org > Subject: Re: [edk2] CryptoPkg: O

Re: [edk2] [PATCH] CryptoPkg: Fix ARM RVCT 4 build

2015-11-24 Thread Long, Qin
meet UEFI TS scenario. In theory, there should have no this kind of link issues (duplicated macro definitions) based on the current implementation. OK, give me more times for digging. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Cohen, Eugene [mailto:eug

Re: [edk2] CryptoPkg: OpenSSL build issue with RVCT

2015-11-23 Thread Long, Qin
(e.g. applying OPENSSL_SYS_UEFI) to silence it in our package, and then submit the ticket in OpenSSL RT for the native fix. Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Cohen, > Eugene > Sent:

Re: [edk2] [PATCH] CryptoPkg: fix AARCH64 build under CLANG35

2015-11-05 Thread Long, Qin
Thanks for the fix, Ard. Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Thursday, November 5, 2015 10:30 PM > To: edk2-devel@lists.01.org; Long, Qin > Cc: Ye, Ting;

Re: [edk2] [PATCH 1/3] MdePkg: Create GetRandomNumber128 in RngLib

2015-10-08 Thread Long, Qin
Looks good. Series Reviewed-by: Qin Long Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Thomas Palmer > Sent: Friday, October 9, 2015 1:13 AM > To: edk2-devel@lists.01.org > Cc: Z

Re: [edk2] [PATCH v3] Pkg-Module: Integrate new RngLib into RngDxe

2015-10-08 Thread Long, Qin
_final_Aug7.pdf"; Best Regards & Thanks, LONG, Qin > -Original Message- > From: Kinney, Michael D > Sent: Thursday, October 8, 2015 11:12 PM > To: Long, Qin; Justen, Jordan L; Thomas Palmer; edk2-devel@lists.01.org; > Kinney, Michael D > Subject: RE: [edk2] [

Re: [edk2] [PATCH v3] Pkg-Module: Integrate new RngLib into RngDxe

2015-10-07 Thread Long, Qin
Thomas, Thanks for doing this. I think it's better to separate your patch into two parts (one for MdePkg, and the other for SecurityPkg). Then you can CC me as Jordan's suggestion. Best Regards & Thanks, LONG, Qin > -Original Message- > From: edk2-devel [ma

Re: [edk2] [patch] SecurityPkg: Fix one returned status code issue in P7Verify Protocol.

2015-08-25 Thread Long, Qin
Exactly. Thanks for catching this. Will send the new patch later. Best Regards & Thanks, LONG, Qin > -Original Message- > From: Zhang, Chao B > Sent: Tuesday, August 25, 2015 8:17 PM > To: Long, Qin; edk2-devel@lists.01.org > Subject: RE: [patch] SecurityPkg: Fix

Re: [edk2] [PATCH] SecurityPkg: Update Package version to 0.96

2015-08-19 Thread Long, Qin
Reviewed-by: Qin Long > -Original Message- > From: Zhang, Chao B > Sent: Thursday, August 20, 2015 10:11 AM > To: edk2-devel@lists.01.org > Cc: Long, Qin; Zhang, Chao B > Subject: [PATCH] SecurityPkg: Update Package version to 0.96 > > Update Package version

Re: [edk2] [PATCH] SecurityPkg: Update SignatureSize to comply UEFI spec

2015-08-16 Thread Long, Qin
Looks good. Reviewed-by: Qin Long < qin.l...@intel.com> Best Regards & Thanks, LONG, Qin > -Original Message- > From: Zhang, Chao B > Sent: Monday, August 17, 2015 10:01 AM > To: edk2-devel@lists.01.org > Cc: Long, Qin; Zhang, Chao B > Subject: [PATCH] Secur

Re: [edk2] [RFC] OpenSSL HEAD WIP

2015-07-28 Thread Long, Qin
> -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Tuesday, July 28, 2015 4:23 PM > To: Long, Qin; Laszlo Ersek; edk2-de...@ml01.01.org > Subject: Re: [edk2] [RFC] OpenSSL HEAD WIP > > On Tue, 2015-07-28 at 08:08 +, Long, Qin w

Re: [edk2] [RFC] OpenSSL HEAD WIP

2015-07-28 Thread Long, Qin
> -Original Message- > From: David Woodhouse [mailto:dw...@infradead.org] > Sent: Tuesday, July 28, 2015 3:42 PM > To: Long, Qin; Laszlo Ersek; edk2-de...@ml01.01.org > Subject: Re: [edk2] [RFC] OpenSSL HEAD WIP > > On Tue, 2015-07-28 at 01:44 +

Re: [edk2] [PATCH 2/2] CryptoPkg/OpensslLib: Undefine NO_BUILTIN_VA_FUNCS to fix varargs breakage

2015-07-27 Thread Long, Qin
Reviewed-by: Qin Long And Ersek, could you kindly help to double-check it will not break any shim scenario? Best Regards & Thanks, LONG, Qin > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Monday, July 27, 2015 8:17 PM > To: David Woodhou

Re: [edk2] [RFC] OpenSSL HEAD WIP

2015-07-27 Thread Long, Qin
> -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Tuesday, July 28, 2015 6:41 AM > To: David Woodhouse; edk2-de...@ml01.01.org > Cc: Long, Qin > Subject: Re: [edk2] [RFC] OpenSSL HEAD WIP > > On 07/27/15 22:13, David Woodhouse wrote:

<    1   2   3