[PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s


Yao, Jiewen
 

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao,
Jiewen
Sent: Friday, November 18, 2022 4:11 PM
To: Chen, Gang C <gang.c.chen@...>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@...>
Subject: Re: [edk2-devel] [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL
to 1.1.1s

Sounds good. No size different is good news.

Reviewed-by: Jiewen Yao <Jiewen.yao@...>

We are in hard-freeze now. This will be merged after freeze.

Thank you
Yao Jiewen

-----Original Message-----
From: Chen, Gang C <gang.c.chen@...>
Sent: Friday, November 18, 2022 4:08 PM
To: Yao, Jiewen <jiewen.yao@...>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@...>
Subject: RE: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Hi Jiewen,

Thanks for your review.

Yes, I did finish the tiano CI:
https://github.com/tianocore/edk2/pull/3648.

Regarding the size, I'm not sure which lib you'd like to know. So I list the 3
CryptLib size here(compiler: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0):

PeiCryptLib.lib BaseCryptLib.lib
SmmCryptLib.lib
1.1.1n 645426 973132
908298
1.1.1s 645402 973140 908298

Thanks.

Best Regards
Gang

-----Original Message-----
From: Yao, Jiewen <jiewen.yao@...>
Sent: Thursday, November 17, 2022 5:32 PM
To: Chen, Gang C <gang.c.chen@...>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@...>; Yao, Jiewen
<jiewen.yao@...>
Subject: RE: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Hi
Have you run the tiano CI ?

Would you please share the size data before and after?

Thank you
Yao, Jiewen

-----Original Message-----
From: Chen, Gang C <gang.c.chen@...>
Sent: Thursday, November 17, 2022 4:40 PM
To: devel@edk2.groups.io
Cc: Yao, Jiewen <jiewen.yao@...>; Wang, Jian J
<jian.j.wang@...>; Chen, Gang C <gang.c.chen@...>
Subject: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Upgrade openssl to 1.1.1s.

Pick up bugfixes from the latest openssl release, which include some
GCC build warnings cleaning up.

Cc: Jian J Wang <jian.j.wang@...>
Cc: Jiewen Yao <jiewen.yao@...>
Signed-off-by: Gang Chen <gang.c.chen@...>
---
CryptoPkg/Library/OpensslLib/openssl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CryptoPkg/Library/OpensslLib/openssl
b/CryptoPkg/Library/OpensslLib/openssl
index d82e959e62..129058165d 160000
--- a/CryptoPkg/Library/OpensslLib/openssl
+++ b/CryptoPkg/Library/OpensslLib/openssl
@@ -1 +1 @@
-Subproject commit d82e959e621a3d597f1e0d50ff8c2d8b96915fd7
+Subproject commit 129058165d195e43a0ad10111b0c2e29bdf65980
--
2.38.0




Yao, Jiewen
 

Sounds good. No size different is good news.

Reviewed-by: Jiewen Yao <Jiewen.yao@...>

We are in hard-freeze now. This will be merged after freeze.

Thank you
Yao Jiewen

-----Original Message-----
From: Chen, Gang C <gang.c.chen@...>
Sent: Friday, November 18, 2022 4:08 PM
To: Yao, Jiewen <jiewen.yao@...>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@...>
Subject: RE: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Hi Jiewen,

Thanks for your review.

Yes, I did finish the tiano CI: https://github.com/tianocore/edk2/pull/3648.

Regarding the size, I'm not sure which lib you'd like to know. So I list the 3
CryptLib size here(compiler: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0):

PeiCryptLib.lib BaseCryptLib.lib
SmmCryptLib.lib
1.1.1n 645426 973132
908298
1.1.1s 645402 973140 908298

Thanks.

Best Regards
Gang

-----Original Message-----
From: Yao, Jiewen <jiewen.yao@...>
Sent: Thursday, November 17, 2022 5:32 PM
To: Chen, Gang C <gang.c.chen@...>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@...>; Yao, Jiewen
<jiewen.yao@...>
Subject: RE: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Hi
Have you run the tiano CI ?

Would you please share the size data before and after?

Thank you
Yao, Jiewen

-----Original Message-----
From: Chen, Gang C <gang.c.chen@...>
Sent: Thursday, November 17, 2022 4:40 PM
To: devel@edk2.groups.io
Cc: Yao, Jiewen <jiewen.yao@...>; Wang, Jian J
<jian.j.wang@...>; Chen, Gang C <gang.c.chen@...>
Subject: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Upgrade openssl to 1.1.1s.

Pick up bugfixes from the latest openssl release, which include some
GCC build warnings cleaning up.

Cc: Jian J Wang <jian.j.wang@...>
Cc: Jiewen Yao <jiewen.yao@...>
Signed-off-by: Gang Chen <gang.c.chen@...>
---
CryptoPkg/Library/OpensslLib/openssl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CryptoPkg/Library/OpensslLib/openssl
b/CryptoPkg/Library/OpensslLib/openssl
index d82e959e62..129058165d 160000
--- a/CryptoPkg/Library/OpensslLib/openssl
+++ b/CryptoPkg/Library/OpensslLib/openssl
@@ -1 +1 @@
-Subproject commit d82e959e621a3d597f1e0d50ff8c2d8b96915fd7
+Subproject commit 129058165d195e43a0ad10111b0c2e29bdf65980
--
2.38.0


Chen, Gang C
 

Hi Jiewen,

Thanks for your review.

Yes, I did finish the tiano CI: https://github.com/tianocore/edk2/pull/3648.

Regarding the size, I'm not sure which lib you'd like to know. So I list the 3 CryptLib size here(compiler: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0):

PeiCryptLib.lib BaseCryptLib.lib SmmCryptLib.lib
1.1.1n 645426 973132 908298
1.1.1s 645402 973140 908298

Thanks.

Best Regards
Gang

-----Original Message-----
From: Yao, Jiewen <jiewen.yao@...>
Sent: Thursday, November 17, 2022 5:32 PM
To: Chen, Gang C <gang.c.chen@...>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@...>; Yao, Jiewen <jiewen.yao@...>
Subject: RE: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Hi
Have you run the tiano CI ?

Would you please share the size data before and after?

Thank you
Yao, Jiewen

-----Original Message-----
From: Chen, Gang C <gang.c.chen@...>
Sent: Thursday, November 17, 2022 4:40 PM
To: devel@edk2.groups.io
Cc: Yao, Jiewen <jiewen.yao@...>; Wang, Jian J
<jian.j.wang@...>; Chen, Gang C <gang.c.chen@...>
Subject: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Upgrade openssl to 1.1.1s.

Pick up bugfixes from the latest openssl release, which include some
GCC build warnings cleaning up.

Cc: Jian J Wang <jian.j.wang@...>
Cc: Jiewen Yao <jiewen.yao@...>
Signed-off-by: Gang Chen <gang.c.chen@...>
---
CryptoPkg/Library/OpensslLib/openssl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CryptoPkg/Library/OpensslLib/openssl
b/CryptoPkg/Library/OpensslLib/openssl
index d82e959e62..129058165d 160000
--- a/CryptoPkg/Library/OpensslLib/openssl
+++ b/CryptoPkg/Library/OpensslLib/openssl
@@ -1 +1 @@
-Subproject commit d82e959e621a3d597f1e0d50ff8c2d8b96915fd7
+Subproject commit 129058165d195e43a0ad10111b0c2e29bdf65980
--
2.38.0


Yao, Jiewen
 

Hi
Have you run the tiano CI ?

Would you please share the size data before and after?

Thank you
Yao, Jiewen

-----Original Message-----
From: Chen, Gang C <gang.c.chen@...>
Sent: Thursday, November 17, 2022 4:40 PM
To: devel@edk2.groups.io
Cc: Yao, Jiewen <jiewen.yao@...>; Wang, Jian J
<jian.j.wang@...>; Chen, Gang C <gang.c.chen@...>
Subject: [PATCH] CryptoPkg/OpensslLib: Upgrade OpenSSL to 1.1.1s

Upgrade openssl to 1.1.1s.

Pick up bugfixes from the latest openssl release, which include
some GCC build warnings cleaning up.

Cc: Jian J Wang <jian.j.wang@...>
Cc: Jiewen Yao <jiewen.yao@...>
Signed-off-by: Gang Chen <gang.c.chen@...>
---
CryptoPkg/Library/OpensslLib/openssl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CryptoPkg/Library/OpensslLib/openssl
b/CryptoPkg/Library/OpensslLib/openssl
index d82e959e62..129058165d 160000
--- a/CryptoPkg/Library/OpensslLib/openssl
+++ b/CryptoPkg/Library/OpensslLib/openssl
@@ -1 +1 @@
-Subproject commit d82e959e621a3d597f1e0d50ff8c2d8b96915fd7
+Subproject commit 129058165d195e43a0ad10111b0c2e29bdf65980
--
2.38.0


Chen, Gang C
 

Upgrade openssl to 1.1.1s.

Pick up bugfixes from the latest openssl release, which include
some GCC build warnings cleaning up.

Cc: Jian J Wang <jian.j.wang@...>
Cc: Jiewen Yao <jiewen.yao@...>
Signed-off-by: Gang Chen <gang.c.chen@...>
---
CryptoPkg/Library/OpensslLib/openssl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CryptoPkg/Library/OpensslLib/openssl b/CryptoPkg/Library/OpensslLib/openssl
index d82e959e62..129058165d 160000
--- a/CryptoPkg/Library/OpensslLib/openssl
+++ b/CryptoPkg/Library/OpensslLib/openssl
@@ -1 +1 @@
-Subproject commit d82e959e621a3d597f1e0d50ff8c2d8b96915fd7
+Subproject commit 129058165d195e43a0ad10111b0c2e29bdf65980
--
2.38.0