Thanks. Merged https://github.com/tianocore/edk2/pull/3458
> -----Original Message----- > From: Li, Yi1 <yi1...@intel.com> > Sent: Monday, October 10, 2022 1:38 PM > To: Yao, Jiewen <jiewen....@intel.com>; devel@edk2.groups.io > Cc: Wang, Jian J <jian.j.w...@intel.com>; Lu, Xiaoyu1 > <xiaoyu1...@intel.com>; Jiang, Guomin <guomin.ji...@intel.com>; Kinney, > Michael D <michael.d.kin...@intel.com>; Gao, Liming > <gaolim...@byosoft.com.cn> > Subject: RE: [edk2-devel] [PATCH V3 0/3] CryptoPkg: Extend Tls library > > 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 <jiewen....@intel.com> > Sent: Monday, October 10, 2022 12:46 PM > To: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com>; Li, Yi1 > <yi1...@intel.com> > Cc: Wang, Jian J <jian.j.w...@intel.com>; Lu, Xiaoyu1 > <xiaoyu1...@intel.com>; Jiang, Guomin <guomin.ji...@intel.com>; Kinney, > Michael D <michael.d.kin...@intel.com>; Gao, Liming > <gaolim...@byosoft.com.cn> > 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 <devel@edk2.groups.io> On Behalf Of Yao, > > Jiewen > > Sent: Monday, October 10, 2022 10:47 AM > > To: Li, Yi1 <yi1...@intel.com>; devel@edk2.groups.io > > Cc: Wang, Jian J <jian.j.w...@intel.com>; Lu, Xiaoyu1 > > <xiaoyu1...@intel.com>; Jiang, Guomin <guomin.ji...@intel.com>; > > Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming > > <gaolim...@byosoft.com.cn> > > Subject: Re: [edk2-devel] [PATCH V3 0/3] CryptoPkg: Extend Tls library > > > > Thanks for the update. > > For all patches, reviewed-by: Jiewen Yao <jiewen....@intel.com> > > > > 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 <yi1...@intel.com> > > > Sent: Monday, October 10, 2022 10:40 AM > > > To: devel@edk2.groups.io > > > Cc: Li, Yi1 <yi1...@intel.com>; Yao, Jiewen <jiewen....@intel.com>; > > > Wang, Jian J <jian.j.w...@intel.com>; Lu, Xiaoyu1 > > > <xiaoyu1...@intel.com>; Jiang, Guomin <guomin.ji...@intel.com>; > > > Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming > > > <gaolim...@byosoft.com.cn> > > > 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 <yi1...@intel.com> > > > Cc: Jiewen Yao <jiewen....@intel.com> > > > Cc: Jian J Wang <jian.j.w...@intel.com> > > > Cc: Xiaoyu Lu <xiaoyu1...@intel.com> > > > Cc: Guomin Jiang <guomin.ji...@intel.com> > > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > > > > > Signed-off-by: Yi Li <yi1...@intel.com> > > > > > > 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 > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#94875): https://edk2.groups.io/g/devel/message/94875 Mute This Topic: https://groups.io/mt/94229172/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
