From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Trahe, Fiona" Subject: Re: [PATCH v6 3/4] app/test: added tests for libcrypto PMD Date: Thu, 6 Oct 2016 13:30:33 +0000 Message-ID: <348A99DA5F5B7549AA880327E580B435890EE063@IRSMSX101.ger.corp.intel.com> References: <1475504792-31557-1-git-send-email-slawomirx.mrozowicz@intel.com> <1475593882-32364-1-git-send-email-slawomirx.mrozowicz@intel.com> <1475593882-32364-4-git-send-email-slawomirx.mrozowicz@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: "Mrozowicz, SlawomirX" , "Azarewicz, PiotrX T" , "Kerlin, MarcinX" , "Mrzyglod, DanielX T" , "Trahe, Fiona" To: "Mrozowicz, SlawomirX" , "dev@dpdk.org" Return-path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 60EB62BF6 for ; Thu, 6 Oct 2016 15:30:37 +0200 (CEST) In-Reply-To: <1475593882-32364-4-git-send-email-slawomirx.mrozowicz@intel.com> Content-Language: en-US List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Slawomir, > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Slawomir Mrozowicz > Sent: Tuesday, October 4, 2016 4:11 PM > To: dev@dpdk.org > Cc: Mrozowicz, SlawomirX ; Azarewicz, > PiotrX T ; Kerlin, MarcinX > ; Mrzyglod, DanielX T > > Subject: [dpdk-dev] [PATCH v6 3/4] app/test: added tests for libcrypto PM= D >=20 > This patch contains unit tests for libcrypto PMD. User can > use app/test application to check how to use this pmd and to > verify crypto processing. >=20 > Test name is cryptodev_libcrypto_autotest. > For performance test cryptodev_libcrypto_perftest can be used. >=20 > Signed-off-by: Piotr Azarewicz > Signed-off-by: Marcin Kerlin > Signed-off-by: Daniel Mrzyglod > --- > v2: > - rename AES-named functions to blockcipher > - replace different test cases with blockcipher functions pattern > - add 3DES tests into QuickAssist PMD testsuite >=20 > v3: > - add nagative verification tests > - add big data test >=20 > v4: > - move aes test rework to another patch > - move big data test to another patch > - checking if libcrypto pmd is available >=20 > v5: > - add reduced big data test >=20 > v6: > - fix checkpatch warnings This patch breaks autotests for all PMDs, due to increasing the MBUF size t= o UNIT16_MAX. USER1: Can't create CRYPTO_MBUFPOOL It needs more than 500MBs in the MBUFPOOL to run this test. Setting this back to MBUF_SIZE fixes the issue, but breaks 2 tests in libcr= ypto_autotest TestCase create_gmac_operation() line 5060 failed: no room to append aad = =20 + TestCase [27] : test_AES_GMAC_authentication_test_case_4 failed = =20 TestCase create_gmac_operation() line 5060 failed: no room to append aad = =20 + TestCase [28] : test_AES_GMAC_authentication_verify_test_case_4 failed = =20 Can you investigate if it's possible to run these 2 tests using a different= smaller MBUFPOOL - maybe with only a few large mbufs?