Hi Jiewen, Yes it is, based on latest commit 3c9e2f239a38590b4e3a8c1ec2304227f2af0103. I applied them to EDKII master branch successfully, not sure why. Attachments are the successful patch files for reference, But they are same as the patches in the mail. Let me know if there are still conflicts. Thanks, Yi -----Original Message----- From: Yao, Jiewen Sent: Monday, October 10, 2022 12:46 PM To: devel@edk2.groups.io; Yao, Jiewen ; Li, Yi1 Cc: Wang, Jian J ; Lu, Xiaoyu1 ; Jiang, Guomin ; Kinney, Michael D ; Gao, Liming Subject: RE: [edk2-devel] [PATCH V3 0/3] CryptoPkg: Extend Tls library Hi Yi Is this patch based on latest EDKII? I failed to apply the patch 2/3. The 1/3 and 3/3 are good. > -----Original Message----- > From: devel@edk2.groups.io On Behalf Of Yao, > Jiewen > Sent: Monday, October 10, 2022 10:47 AM > To: Li, Yi1 ; devel@edk2.groups.io > Cc: Wang, Jian J ; Lu, Xiaoyu1 > ; Jiang, Guomin ; > Kinney, Michael D ; Gao, Liming > > Subject: Re: [edk2-devel] [PATCH V3 0/3] CryptoPkg: Extend Tls library > > Thanks for the update. > For all patches, reviewed-by: Jiewen Yao > > I think we need MdePkg owner to give R-B or A-B for TLS definition in > MdePkg. Mike or Liming? > > Thank you > Yao Jiewen > > > > -----Original Message----- > > From: Li, Yi1 > > Sent: Monday, October 10, 2022 10:40 AM > > To: devel@edk2.groups.io > > Cc: Li, Yi1 ; Yao, Jiewen ; > > Wang, Jian J ; Lu, Xiaoyu1 > > ; Jiang, Guomin ; > > Kinney, Michael D ; Gao, Liming > > > > Subject: [PATCH V3 0/3] CryptoPkg: Extend Tls library > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3892 > > > > Review PR: https://github.com/tianocore/edk2/pull/3400 > > This patch sequence is used to extend Tls library, which are wrapped > > over OpenSSL. The implementation provides library functions for EFI > > DXE dirver and Protocol. > > > > All APIs passed unit test and fuzzing test, detail as: > > 1. Unit test: > > New Tls APIs tested on Intel platform as part of WIFI WPA3 feature. > > 2. Fuzzing test: > > Various Fuzz Testing are employed across the all introduced APIs, > > and the test is used AFL (2.52b) and Libfuzzer (clang+llvm-11.0.0) > > as the fuzzer, based on HBFA. > > Fuzzing Pass Rate is 100%; > > The Code Coverage of new APIs is 91%. > > All test case show in: > > https://github.com/liyi77/edk2- > > staging/tree/HBFA/HBFA/UefiHostFuzzTestCasePkg/TestCase/CryptoPkg > > > > V2 change: > > Move the newly added APIs to the end of struct PCD. > > V3 change: > > Corrected tls specification reference and tls cipher suite names. > > > > Tested-by: Yi Li > > Cc: Jiewen Yao > > Cc: Jian J Wang > > Cc: Xiaoyu Lu > > Cc: Guomin Jiang > > Cc: Michael D Kinney > > Cc: Liming Gao > > > > Signed-off-by: Yi Li > > > > Yi Li (3): > > MdePkg: Add Tls configuration related define > > CryptoPkg: Extend Tls function library > > CryptoPkg: Add new Tls APIs to DXE and protocol > > > > CryptoPkg/Driver/Crypto.c | 155 +++++++- > > CryptoPkg/Include/Library/TlsLib.h | 126 +++++- > > .../Pcd/PcdCryptoServiceFamilyEnable.h | 5 + > > .../BaseCryptLibOnProtocolPpi/CryptLib.c | 146 ++++++- > > CryptoPkg/Library/TlsLib/InternalTlsLib.h | 4 + > > CryptoPkg/Library/TlsLib/TlsConfig.c | 366 +++++++++++++++++- > > CryptoPkg/Library/TlsLib/TlsProcess.c | 32 ++ > > CryptoPkg/Library/TlsLibNull/TlsConfigNull.c | 123 +++++- > > CryptoPkg/Library/TlsLibNull/TlsProcessNull.c | 23 ++ > > CryptoPkg/Private/Protocol/Crypto.h | 136 ++++++- > > MdePkg/Include/IndustryStandard/Tls1.h | 112 ++++-- > > 11 files changed, 1177 insertions(+), 51 deletions(-) > > > > -- > > 2.31.1.windows.1 > > > > >