> -----Original Message----- > From: De Lara Guarch, Pablo <[email protected]> > Sent: Thursday, April 16, 2020 9:29 PM > To: Ruifeng Wang <[email protected]>; Doherty, Declan > <[email protected]>; [email protected]; [email protected]; > Zhang, Roy Fan <[email protected]>; Trahe, Fiona > <[email protected]>; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected] > Cc: [email protected]; [email protected]; nd <[email protected]> > Subject: RE: [PATCH 0/3] Crypto test refactoring (second phase) > > Hi Ruifeng, > > > -----Original Message----- > > From: Ruifeng Wang <[email protected]> > > Sent: Thursday, April 16, 2020 9:29 AM > > To: De Lara Guarch, Pablo <[email protected]>; Doherty, > > Declan <[email protected]>; [email protected]; > > [email protected]; Zhang, Roy Fan <[email protected]>; Trahe, > > Fiona <[email protected]>; [email protected]; > > [email protected]; [email protected]; [email protected]; > > [email protected]; [email protected] > > Cc: [email protected]; [email protected]; nd <[email protected]> > > Subject: RE: [PATCH 0/3] Crypto test refactoring (second phase) > > > > > > > -----Original Message----- > > > From: Pablo de Lara <[email protected]> > > > Sent: Wednesday, April 15, 2020 2:04 AM > > > To: [email protected]; [email protected]; Ruifeng Wang > > > <[email protected]>; [email protected]; > [email protected]; > > > [email protected]; [email protected]; [email protected]; > > > [email protected]; [email protected]; [email protected]; > > > [email protected] > > > Cc: [email protected]; [email protected]; Pablo de Lara > > > <[email protected]> > > > Subject: [PATCH 0/3] Crypto test refactoring (second phase) > > > > > > This patchset is the phase two of the crypto test refactoring effort. > > > It mainly focuses on removing the PMD bitmask used to select which > > > PMDs are used to run the tests against. > > > Instead, now all PMDs will run all test cases and the capability > > > checking will determine which ones are supported and therefore, are > > > not > > skipped. > > > > > > Since there are tests cases that require sessionless support, a new > > > feature flag has been added, which reflects this support per PMD. > > > > > > I am CC'ing all PMD maintainers, asking for their collaboration to > > > test this patchset, as this might impact the testing of each PMD. > > > > > > > > > This patchset depends on: > > > (1) http://patches.dpdk.org/patch/68433/ (crypto/openssl: fix > > > out-of-place > > > encryption) > > > (2) http://patches.dpdk.org/patch/68434/ (cryptodev: add missing > > > feature > > > name) > > > (3) http://patches.dpdk.org/cover/68435/ (Crypto test refactoring > > > (first > > > phase)) > > > > > > Pablo de Lara (3): > > > cryptodev: add sessionless support feature flag > > > test/crypto: check if device supports sessionless > > > test/crypto: do not check for PMD in tests > > > > > There is one test case failure with crypto_armv8 after applying these > patches. > > I will try to locate it. > > > > 33) TestCase AES-128-CBC HMAC-SHA1 Decryption Digest Verify > > Sessionless PASS > > 34) TestCase NULL-CIPHER-NULL-AUTH encrypt & gen x8byte PASS > > 35) TestCase NULL-AUTH-NULL-CIPHER verify & decrypt x8byte PASS > > 36) TestCase NULL-CIPHER-NULL-AUTH encrypt & gen x8byte - OOP PASS > > 37) TestCase NULL-AUTH-NULL-CIPHER verify & decrypt x8byte - OOP > PASS > > 38) TestCase NULL-CIPHER-NULL-AUTH encrypt & gen x4byte PASS > > 39) TestCase NULL-AUTH-NULL-CIPHER verify & decrypt x4byte PASS > > 40) TestCase NULL-CIPHER-NULL-AUTH encrypt & gen x4byte - OOP PASS > > 41) TestCase NULL-AUTH-NULL-CIPHER verify & decrypt x4byte - OOP > PASS > > 42) TestCase NULL-CIPHER-NULL-AUTH encrypt & gen x1byte PASS > > 43) TestCase NULL-AUTH-NULL-CIPHER verify & decrypt x1byte PASS > > 44) TestCase NULL-CIPHER-NULL-AUTH encrypt & gen x1byte - OOP PASS > > 45) TestCase NULL-AUTH-NULL-CIPHER verify & decrypt x1byte - OOP > > PASS + > > These test cases shouldn't be run, as your PMD doesn't support NULL > cipher/auth. > Have you merged the first phase of this refactoring work? Yes, I applied listed dependencies. 68433/68434/68436/68437/68439/68440/68441
> I just pushed new versions for both phases, as there were a few problems in > the first ones: > http://patches.dpdk.org/cover/68677/ > http://patches.dpdk.org/cover/68681/ > > Could you tried the new versions? (remember to also apply > http://patches.dpdk.org/patch/68433/ > and http://patches.dpdk.org/patch/68434/) The new versions works fine. Since OOP tests are skipped. Thanks. /Ruifeng > > Thanks, > Pablo > > >
