Re: [edk2-devel] [PATCH v5 0/9] Crypto: Upgrade OpenSSL to 1.1.1b

2019-06-03 Thread Laszlo Ersek 
On 06/03/19 02:48, Wang, Jian J wrote:
> Pushed 
> 6fcc3d68eb02aa798ef572c94867a03d0c1fedf2...acfb90911840c38a0beb9bcfe0065668244d2b4d

Thank you all.

The proper commit range to mention here is: 371e7001e8d5..acfb90911840.

(371e7001e8d5 is the parent of 6fcc3d68eb02.)

Because, the LHS of such a git commit range is exclusive. Please refer
to

for the explanation.

Thanks
Laszlo

>> -Original Message-
>> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>> Xiaoyu Lu
>> Sent: Wednesday, May 29, 2019 6:40 PM
>> To: devel@edk2.groups.io
>> Cc: Lu, XiaoyuX ; Wang, Jian J ;
>> Ye, Ting ; Laszlo Ersek 
>> Subject: [edk2-devel] [PATCH v5 0/9] Crypto: Upgrade OpenSSL to 1.1.1b
>>
>> This series is also avaiable at:
>> https://github.com/xiaoyuxlu/edk2/commits/bz_1089_upgrade_to_openssl_1_
>> 1_1b_v5
>>
>> Change since v4:
>>
>> 1. Split "[PATCH v4 6/7] CryptoPkg: Upgrade OpenSSL to 1.1.1b" patch into 
>> two.
>>a. CryptoPkg/OpensslLib: Add functions for upgrading OpenSSL1_1_1b
>>b. CryptoPkg: Upgrade OpenSSL to 1.1.1b
>>
>> 2. Add "CryptoPkg: Fix possible build problem with Clang" patch.
>>Blow is a link to the problem.
>>Ref: https://edk2.groups.io/g/devel/message/41424
>>OpenSSL_1_1_1b may build failed with Clang38. So
>>add additional build flag to work around it.
>>
>> Cc: Jian J Wang 
>> Cc: Ting Ye 
>> Cc: Laszlo Ersek 
>>
>> Laszlo Ersek (1):
>>   CryptoPkg/OpensslLib: Fix cross-build problem for AARCH64
>>
>> Xiaoyu Lu (8):
>>   CryptoPkg/OpensslLib: Modify process_files.pl for upgrading OpenSSL
>>   CryptoPkg/OpensslLib: Exclude unnecessary files in process_files.pl
>>   CryptoPkg/IntrinsicLib: Fix possible unresolved external symbol issue
>>   CryptoPkg/OpensslLib: Prepare for upgrading OpenSSL
>>   CryptoPkg: Fix possible build problem with Clang
>>   CryptoPkg/OpensslLib: Add functions for upgrading OpenSSL1_1_1b
>>   CryptoPkg: Upgrade OpenSSL to 1.1.1b
>>   CryptoPkg/BaseCryptLib: Make HMAC_CTX size backward compatible
>>
>>  CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf|   2 +
>>  CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf |   2 +
>>  CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf |   2 +
>>  CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf |   2 +
>>  CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf|   4 +-
>>  CryptoPkg/Library/OpensslLib/OpensslLib.inf|  77 -
>>  CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf  |  68 -
>>  CryptoPkg/Library/Include/CrtLibSupport.h  |  13 +-
>>  CryptoPkg/Library/Include/openssl/opensslconf.h|  54 +++-
>>  CryptoPkg/Library/Include/sys/syscall.h|  11 +
>>  CryptoPkg/Library/OpensslLib/buildinf.h|   2 +
>>  CryptoPkg/Library/OpensslLib/rand_pool_noise.h |  29 ++
>>  CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacMd5.c |   8 +-
>>  .../Library/BaseCryptLib/Hmac/CryptHmacSha1.c  |   9 +-
>>  .../Library/BaseCryptLib/Hmac/CryptHmacSha256.c|   8 +-
>>  CryptoPkg/Library/IntrinsicLib/Ia32/MathFtol.c |  22 ++
>>  CryptoPkg/Library/OpensslLib/ossl_store.c  |  17 ++
>>  CryptoPkg/Library/OpensslLib/rand_pool.c   | 316
>> +
>>  CryptoPkg/Library/OpensslLib/rand_pool_noise.c |  29 ++
>>  CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c |  43 +++
>>  CryptoPkg/Library/OpensslLib/openssl   |   2 +-
>>  CryptoPkg/Library/OpensslLib/process_files.pl  |  11 +-
>>  22 files changed, 679 insertions(+), 52 deletions(-)
>>  create mode 100644 CryptoPkg/Library/Include/sys/syscall.h
>>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.h
>>  create mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathFtol.c
>>  create mode 100644 CryptoPkg/Library/OpensslLib/ossl_store.c
>>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool.c
>>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.c
>>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
>>
>> --
>> 2.7.4
>>
>>
>> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41811): https://edk2.groups.io/g/devel/message/41811
Mute This Topic: https://groups.io/mt/31832279/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v5 0/9] Crypto: Upgrade OpenSSL to 1.1.1b

2019-06-02 Thread Wang, Jian J 
Pushed 
6fcc3d68eb02aa798ef572c94867a03d0c1fedf2...acfb90911840c38a0beb9bcfe0065668244d2b4d

Regards,
Jian


> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Xiaoyu Lu
> Sent: Wednesday, May 29, 2019 6:40 PM
> To: devel@edk2.groups.io
> Cc: Lu, XiaoyuX ; Wang, Jian J ;
> Ye, Ting ; Laszlo Ersek 
> Subject: [edk2-devel] [PATCH v5 0/9] Crypto: Upgrade OpenSSL to 1.1.1b
> 
> This series is also avaiable at:
> https://github.com/xiaoyuxlu/edk2/commits/bz_1089_upgrade_to_openssl_1_
> 1_1b_v5
> 
> Change since v4:
> 
> 1. Split "[PATCH v4 6/7] CryptoPkg: Upgrade OpenSSL to 1.1.1b" patch into two.
>a. CryptoPkg/OpensslLib: Add functions for upgrading OpenSSL1_1_1b
>b. CryptoPkg: Upgrade OpenSSL to 1.1.1b
> 
> 2. Add "CryptoPkg: Fix possible build problem with Clang" patch.
>Blow is a link to the problem.
>Ref: https://edk2.groups.io/g/devel/message/41424
>OpenSSL_1_1_1b may build failed with Clang38. So
>add additional build flag to work around it.
> 
> Cc: Jian J Wang 
> Cc: Ting Ye 
> Cc: Laszlo Ersek 
> 
> Laszlo Ersek (1):
>   CryptoPkg/OpensslLib: Fix cross-build problem for AARCH64
> 
> Xiaoyu Lu (8):
>   CryptoPkg/OpensslLib: Modify process_files.pl for upgrading OpenSSL
>   CryptoPkg/OpensslLib: Exclude unnecessary files in process_files.pl
>   CryptoPkg/IntrinsicLib: Fix possible unresolved external symbol issue
>   CryptoPkg/OpensslLib: Prepare for upgrading OpenSSL
>   CryptoPkg: Fix possible build problem with Clang
>   CryptoPkg/OpensslLib: Add functions for upgrading OpenSSL1_1_1b
>   CryptoPkg: Upgrade OpenSSL to 1.1.1b
>   CryptoPkg/BaseCryptLib: Make HMAC_CTX size backward compatible
> 
>  CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf|   2 +
>  CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf |   2 +
>  CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf |   2 +
>  CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf |   2 +
>  CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf|   4 +-
>  CryptoPkg/Library/OpensslLib/OpensslLib.inf|  77 -
>  CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf  |  68 -
>  CryptoPkg/Library/Include/CrtLibSupport.h  |  13 +-
>  CryptoPkg/Library/Include/openssl/opensslconf.h|  54 +++-
>  CryptoPkg/Library/Include/sys/syscall.h|  11 +
>  CryptoPkg/Library/OpensslLib/buildinf.h|   2 +
>  CryptoPkg/Library/OpensslLib/rand_pool_noise.h |  29 ++
>  CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacMd5.c |   8 +-
>  .../Library/BaseCryptLib/Hmac/CryptHmacSha1.c  |   9 +-
>  .../Library/BaseCryptLib/Hmac/CryptHmacSha256.c|   8 +-
>  CryptoPkg/Library/IntrinsicLib/Ia32/MathFtol.c |  22 ++
>  CryptoPkg/Library/OpensslLib/ossl_store.c  |  17 ++
>  CryptoPkg/Library/OpensslLib/rand_pool.c   | 316
> +
>  CryptoPkg/Library/OpensslLib/rand_pool_noise.c |  29 ++
>  CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c |  43 +++
>  CryptoPkg/Library/OpensslLib/openssl   |   2 +-
>  CryptoPkg/Library/OpensslLib/process_files.pl  |  11 +-
>  22 files changed, 679 insertions(+), 52 deletions(-)
>  create mode 100644 CryptoPkg/Library/Include/sys/syscall.h
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.h
>  create mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathFtol.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/ossl_store.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
> 
> --
> 2.7.4
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41783): https://edk2.groups.io/g/devel/message/41783
Mute This Topic: https://groups.io/mt/31832279/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v5 0/9] Crypto: Upgrade OpenSSL to 1.1.1b

2019-05-29 Thread Wang, Jian J 
For this patch series,
Reviewed-by: Jian J Wang 

> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Xiaoyu Lu
> Sent: Wednesday, May 29, 2019 6:40 PM
> To: devel@edk2.groups.io
> Cc: Lu, XiaoyuX ; Wang, Jian J ;
> Ye, Ting ; Laszlo Ersek 
> Subject: [edk2-devel] [PATCH v5 0/9] Crypto: Upgrade OpenSSL to 1.1.1b
> 
> This series is also avaiable at:
> https://github.com/xiaoyuxlu/edk2/commits/bz_1089_upgrade_to_openssl_1_
> 1_1b_v5
> 
> Change since v4:
> 
> 1. Split "[PATCH v4 6/7] CryptoPkg: Upgrade OpenSSL to 1.1.1b" patch into two.
>a. CryptoPkg/OpensslLib: Add functions for upgrading OpenSSL1_1_1b
>b. CryptoPkg: Upgrade OpenSSL to 1.1.1b
> 
> 2. Add "CryptoPkg: Fix possible build problem with Clang" patch.
>Blow is a link to the problem.
>Ref: https://edk2.groups.io/g/devel/message/41424
>OpenSSL_1_1_1b may build failed with Clang38. So
>add additional build flag to work around it.
> 
> Cc: Jian J Wang 
> Cc: Ting Ye 
> Cc: Laszlo Ersek 
> 
> Laszlo Ersek (1):
>   CryptoPkg/OpensslLib: Fix cross-build problem for AARCH64
> 
> Xiaoyu Lu (8):
>   CryptoPkg/OpensslLib: Modify process_files.pl for upgrading OpenSSL
>   CryptoPkg/OpensslLib: Exclude unnecessary files in process_files.pl
>   CryptoPkg/IntrinsicLib: Fix possible unresolved external symbol issue
>   CryptoPkg/OpensslLib: Prepare for upgrading OpenSSL
>   CryptoPkg: Fix possible build problem with Clang
>   CryptoPkg/OpensslLib: Add functions for upgrading OpenSSL1_1_1b
>   CryptoPkg: Upgrade OpenSSL to 1.1.1b
>   CryptoPkg/BaseCryptLib: Make HMAC_CTX size backward compatible
> 
>  CryptoPkg/Library/BaseCryptLib/BaseCryptLib.inf|   2 +
>  CryptoPkg/Library/BaseCryptLib/PeiCryptLib.inf |   2 +
>  CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf |   2 +
>  CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf |   2 +
>  CryptoPkg/Library/IntrinsicLib/IntrinsicLib.inf|   4 +-
>  CryptoPkg/Library/OpensslLib/OpensslLib.inf|  77 -
>  CryptoPkg/Library/OpensslLib/OpensslLibCrypto.inf  |  68 -
>  CryptoPkg/Library/Include/CrtLibSupport.h  |  13 +-
>  CryptoPkg/Library/Include/openssl/opensslconf.h|  54 +++-
>  CryptoPkg/Library/Include/sys/syscall.h|  11 +
>  CryptoPkg/Library/OpensslLib/buildinf.h|   2 +
>  CryptoPkg/Library/OpensslLib/rand_pool_noise.h |  29 ++
>  CryptoPkg/Library/BaseCryptLib/Hmac/CryptHmacMd5.c |   8 +-
>  .../Library/BaseCryptLib/Hmac/CryptHmacSha1.c  |   9 +-
>  .../Library/BaseCryptLib/Hmac/CryptHmacSha256.c|   8 +-
>  CryptoPkg/Library/IntrinsicLib/Ia32/MathFtol.c |  22 ++
>  CryptoPkg/Library/OpensslLib/ossl_store.c  |  17 ++
>  CryptoPkg/Library/OpensslLib/rand_pool.c   | 316
> +
>  CryptoPkg/Library/OpensslLib/rand_pool_noise.c |  29 ++
>  CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c |  43 +++
>  CryptoPkg/Library/OpensslLib/openssl   |   2 +-
>  CryptoPkg/Library/OpensslLib/process_files.pl  |  11 +-
>  22 files changed, 679 insertions(+), 52 deletions(-)
>  create mode 100644 CryptoPkg/Library/Include/sys/syscall.h
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.h
>  create mode 100644 CryptoPkg/Library/IntrinsicLib/Ia32/MathFtol.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/ossl_store.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise.c
>  create mode 100644 CryptoPkg/Library/OpensslLib/rand_pool_noise_tsc.c
> 
> --
> 2.7.4
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41638): https://edk2.groups.io/g/devel/message/41638
Mute This Topic: https://groups.io/mt/31832279/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-