one more... title should be test/crypto: add octeontx unit tests

On 10/8/2018 6:03 PM, Akhil Goyal wrote:

Hi Anoob,

On 10/5/2018 6:29 PM, Anoob Joseph wrote:
From: Ankur Dwivedi <ankur.dwiv...@caviumnetworks.com>

This patch adds validation tests for octeontx crypto device.

Signed-off-by: Ankur Dwivedi <ankur.dwiv...@caviumnetworks.com>
Signed-off-by: Anoob Joseph <anoob.jos...@caviumnetworks.com>
Signed-off-by: Murthy NSSR <nidadavolu.mur...@caviumnetworks.com>
Signed-off-by: Nithin Dabilpuram <nithin.dabilpu...@caviumnetworks.com>
Signed-off-by: Ragothaman Jayaraman <rjayara...@caviumnetworks.com>
Signed-off-by: Srisivasubramanian S <ssriniva...@caviumnetworks.com>
Signed-off-by: Tejasree Kondoj <kondoj.tejas...@caviumnetworks.com>
---
  test/test/meson.build                        |   1 +
  test/test/test_cryptodev.c                   | 316 +++++++++++++++++++++++++++
  test/test/test_cryptodev.h                   |   1 +
  test/test/test_cryptodev_aes_test_vectors.h  | 114 ++++++----
  test/test/test_cryptodev_blockcipher.c       |   9 +-
  test/test/test_cryptodev_blockcipher.h       |   1 +
  test/test/test_cryptodev_des_test_vectors.h  |  12 +-
  test/test/test_cryptodev_hash_test_vectors.h |  78 ++++---
  8 files changed, 463 insertions(+), 69 deletions(-)

diff --git a/test/test/meson.build b/test/test/meson.build
index b1dd6ec..2d59041 100644
--- a/test/test/meson.build
+++ b/test/test/meson.build
@@ -141,6 +141,7 @@ test_names = [
      'cryptodev_sw_mvsam_autotest',
      'cryptodev_dpaa2_sec_autotest',
      'cryptodev_dpaa_sec_autotest',
+    'cryptodev_octeontx_autotest',
      'cycles_autotest',
      'debug_autotest',
      'devargs_autotest',
diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c
index c63662d..b9731b9 100644
--- a/test/test/test_cryptodev.c
+++ b/test/test/test_cryptodev.c
@@ -2142,6 +2142,96 @@ test_3DES_cipheronly_mrvl_all(void)
      return TEST_SUCCESS;
  }
  +static int
+test_AES_chain_cpt_all(void)
+{
+    struct crypto_testsuite_params *ts_params = &testsuite_params;
+    int status;
+
+    status = test_blockcipher_all_tests(ts_params->mbuf_pool,
+        ts_params->op_mpool, ts_params->session_mpool,
+        ts_params->valid_devs[0],
+        rte_cryptodev_driver_id_get(
+        RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)),
+        BLKCIPHER_AES_CHAIN_TYPE);
+
+    TEST_ASSERT_EQUAL(status, 0, "Test failed");
+
+    return TEST_SUCCESS;
+}
+
+static int
+test_AES_cipheronly_cpt_all(void)
+{
+    struct crypto_testsuite_params *ts_params = &testsuite_params;
+    int status;
+
+    status = test_blockcipher_all_tests(ts_params->mbuf_pool,
+        ts_params->op_mpool, ts_params->session_mpool,
+        ts_params->valid_devs[0],
+        rte_cryptodev_driver_id_get(
+        RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)),
+        BLKCIPHER_AES_CIPHERONLY_TYPE);
+
+    TEST_ASSERT_EQUAL(status, 0, "Test failed");
+
+    return TEST_SUCCESS;
+}
+
+static int
+test_3DES_chain_cpt_all(void)
+{
+    struct crypto_testsuite_params *ts_params = &testsuite_params;
+    int status;
+
+    status = test_blockcipher_all_tests(ts_params->mbuf_pool,
+        ts_params->op_mpool, ts_params->session_mpool,
+        ts_params->valid_devs[0],
+        rte_cryptodev_driver_id_get(
+        RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)),
+        BLKCIPHER_3DES_CHAIN_TYPE);
+
+    TEST_ASSERT_EQUAL(status, 0, "Test failed");
+
+    return TEST_SUCCESS;
+}
+
+static int
+test_3DES_cipheronly_cpt_all(void)
+{
+    struct crypto_testsuite_params *ts_params = &testsuite_params;
+    int status;
+
+    status = test_blockcipher_all_tests(ts_params->mbuf_pool,
+        ts_params->op_mpool, ts_params->session_mpool,
+        ts_params->valid_devs[0],
+        rte_cryptodev_driver_id_get(
+        RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)),
+        BLKCIPHER_3DES_CIPHERONLY_TYPE);
+
+    TEST_ASSERT_EQUAL(status, 0, "Test failed");
+
+    return TEST_SUCCESS;
+}
+
+static int
+test_authonly_cpt_all(void)
+{
+    struct crypto_testsuite_params *ts_params = &testsuite_params;
+    int status;
+
+    status = test_blockcipher_all_tests(ts_params->mbuf_pool,
+        ts_params->op_mpool, ts_params->session_mpool,
+        ts_params->valid_devs[0],
+        rte_cryptodev_driver_id_get(
+        RTE_STR(CRYPTODEV_NAME_OCTEONTX_SYM_PMD)),
+        BLKCIPHER_AUTHONLY_TYPE);
+
+    TEST_ASSERT_EQUAL(status, 0, "Test failed");
+
+    return TEST_SUCCESS;
+}
+
  /* ***** SNOW 3G Tests ***** */
  static int
  create_wireless_algo_hash_session(uint8_t dev_id,
@@ -9951,6 +10041,216 @@ static struct unit_test_suite cryptodev_ccp_testsuite  = {
      }
  };
  +static struct unit_test_suite cryptodev_cpt_testsuite  = {
Shouldn't this be cryptodev_octeontx_testsuite and for other functions as well? I believe cpt is helper API set, and main PMD is octeontx, so the test shall also have octeontx reference.

Thanks,
Akhil

Reply via email to