DPDK patches and discussions
 help / color / mirror / Atom feed
From: Pablo de Lara <pablo.de.lara.guarch@intel.com>
To: declan.doherty@intel.com, ravi1.kumar@amd.com,
	ruifeng.wang@arm.com, anoobj@marvell.com,
	roy.fan.zhang@intel.com, fiona.trahe@intel.com, tdu@semihalf.com,
	rnagadheeraj@marvell.com, adwivedi@marvell.com, g.singh@nxp.com,
	hemant.agrawal@nxp.com, jianjay.zhou@huawei.com
Cc: dev@dpdk.org, akhil.goyal@nxp.com,
	Pablo de Lara <pablo.de.lara.guarch@intel.com>
Subject: [dpdk-dev] [PATCH v2 0/4] Crypto test refactoring (second phase)
Date: Thu, 16 Apr 2020 10:25:28 +0100	[thread overview]
Message-ID: <20200416092532.1922099-1-pablo.de.lara.guarch@intel.com> (raw)
In-Reply-To: <1586887459-222433-1-git-send-email-pablo.de.lara.guarch@intel.com>

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/68677/ (Crypto test refactoring (first phase))

Changes in v2:
- Added new commit, adding a check for out-of-place operations

Pablo de Lara (4):
  cryptodev: add sessionless support feature flag
  test/crypto: check if device supports sessionless
  test/crypto: check if device supports out-of-place ops
  test/crypto: do not check for PMD in tests

 app/test/test_cryptodev.c                    |   1 -
 app/test/test_cryptodev_aes_test_vectors.h   | 519 -------------------
 app/test/test_cryptodev_blockcipher.c        | 155 +-----
 app/test/test_cryptodev_blockcipher.h        |  18 -
 app/test/test_cryptodev_des_test_vectors.h   | 160 ------
 app/test/test_cryptodev_hash_test_vectors.h  | 219 --------
 doc/guides/cryptodevs/features/aesni_gcm.ini |   1 +
 doc/guides/cryptodevs/features/aesni_mb.ini  |   1 +
 doc/guides/cryptodevs/features/armv8.ini     |   1 +
 doc/guides/cryptodevs/features/caam_jr.ini   |   2 +-
 doc/guides/cryptodevs/features/ccp.ini       |   3 +-
 doc/guides/cryptodevs/features/default.ini   |   1 +
 doc/guides/cryptodevs/features/kasumi.ini    |   3 +-
 doc/guides/cryptodevs/features/octeontx.ini  |   1 +
 doc/guides/cryptodevs/features/octeontx2.ini |   1 +
 doc/guides/cryptodevs/features/openssl.ini   |   1 +
 doc/guides/cryptodevs/features/snow3g.ini    |   3 +-
 doc/guides/cryptodevs/features/zuc.ini       |   1 +
 drivers/crypto/aesni_gcm/aesni_gcm_pmd.c     |   3 +-
 drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c   |   3 +-
 drivers/crypto/armv8/rte_armv8_pmd.c         |   3 +-
 drivers/crypto/ccp/rte_ccp_pmd.c             |   3 +-
 drivers/crypto/kasumi/rte_kasumi_pmd.c       |   3 +-
 drivers/crypto/null/null_crypto_pmd.c        |   3 +-
 drivers/crypto/octeontx/otx_cryptodev_ops.c  |   3 +-
 drivers/crypto/octeontx2/otx2_cryptodev.c    |   3 +-
 drivers/crypto/openssl/rte_openssl_pmd.c     |   3 +-
 drivers/crypto/snow3g/rte_snow3g_pmd.c       |   3 +-
 drivers/crypto/zuc/rte_zuc_pmd.c             |   3 +-
 lib/librte_cryptodev/rte_cryptodev.c         |   2 +
 lib/librte_cryptodev/rte_cryptodev.h         |   2 +
 31 files changed, 67 insertions(+), 1061 deletions(-)

-- 
2.25.2


  parent reply	other threads:[~2020-04-16 13:26 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-14 18:04 [dpdk-dev] [PATCH 0/3] " Pablo de Lara
2020-04-14 18:04 ` [dpdk-dev] [PATCH 1/3] cryptodev: add sessionless support feature flag Pablo de Lara
2020-04-15 10:38   ` Dybkowski, AdamX
2020-04-14 18:04 ` [dpdk-dev] [PATCH 2/3] test/crypto: check if device supports sessionless Pablo de Lara
2020-04-15 10:38   ` Dybkowski, AdamX
2020-04-14 18:04 ` [dpdk-dev] [PATCH 3/3] test/crypto: do not check for PMD in tests Pablo de Lara
2020-04-15 10:38   ` Dybkowski, AdamX
2020-04-16  8:28 ` [dpdk-dev] [PATCH 0/3] Crypto test refactoring (second phase) Ruifeng Wang
2020-04-16 13:28   ` De Lara Guarch, Pablo
2020-04-16 15:23     ` Ruifeng Wang
2020-04-16  9:25 ` Pablo de Lara [this message]
2020-04-16  9:25   ` [dpdk-dev] [PATCH v2 1/4] cryptodev: add sessionless support feature flag Pablo de Lara
2020-04-16 15:27     ` Ruifeng Wang
2020-04-16  9:25   ` [dpdk-dev] [PATCH v2 2/4] test/crypto: check if device supports sessionless Pablo de Lara
2020-04-16 15:27     ` Ruifeng Wang
2020-04-16  9:25   ` [dpdk-dev] [PATCH v2 3/4] test/crypto: check if device supports out-of-place ops Pablo de Lara
2020-04-16 13:41     ` Dybkowski, AdamX
2020-04-16 15:28     ` Ruifeng Wang
2020-04-16  9:25   ` [dpdk-dev] [PATCH v2 4/4] test/crypto: do not check for PMD in tests Pablo de Lara
2020-04-16 15:28     ` Ruifeng Wang
2020-04-16 16:52   ` [dpdk-dev] [PATCH v2 0/4] Crypto test refactoring (second phase) Trahe, Fiona
2020-04-19 21:13     ` Akhil Goyal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200416092532.1922099-1-pablo.de.lara.guarch@intel.com \
    --to=pablo.de.lara.guarch@intel.com \
    --cc=adwivedi@marvell.com \
    --cc=akhil.goyal@nxp.com \
    --cc=anoobj@marvell.com \
    --cc=declan.doherty@intel.com \
    --cc=dev@dpdk.org \
    --cc=fiona.trahe@intel.com \
    --cc=g.singh@nxp.com \
    --cc=hemant.agrawal@nxp.com \
    --cc=jianjay.zhou@huawei.com \
    --cc=ravi1.kumar@amd.com \
    --cc=rnagadheeraj@marvell.com \
    --cc=roy.fan.zhang@intel.com \
    --cc=ruifeng.wang@arm.com \
    --cc=tdu@semihalf.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).