Thread (278 messages) 278 messages, 7 authors, 2021-07-21

Re: [dpdk-dev] [EXT] [PATCH v5 14/15] test/crypto: add mlx5 crypto driver

From: Akhil Goyal <hidden>
Date: 2021-07-06 08:47:16

quoted hunk ↗ jump to hunk
In order to test the new mlx5 crypto PMD, the driver is added to the
crypto test application.

Added mlx5 vectors that include 4 testing vectors with length 512 and
4096.

Added mlx5 encryption function and decryption function that will both
use the mlx5 vectors that will also set the dataunit_len and use a
wrapped key.

The added tests will test both data integrity and correct stat values.

Signed-off-by: Shiri Kuzin <redacted>
Acked-by: Matan Azrad <redacted>
---
 app/test/meson.build                        |    1 +
 app/test/test_cryptodev.c                   |  334 ++-
 app/test/test_cryptodev.h                   |    3 +-
 app/test/test_cryptodev_mlx5_test_vectors.h | 2502 +++++++++++++++++++
 4 files changed, 2823 insertions(+), 17 deletions(-)
 create mode 100644 app/test/test_cryptodev_mlx5_test_vectors.h
diff --git a/app/test/meson.build b/app/test/meson.build
index 0a5f425578..c36655888e 100644
--- a/app/test/meson.build
+++ b/app/test/meson.build
@@ -313,6 +313,7 @@ driver_test_names = [
         'cryptodev_aesni_gcm_autotest',
         'cryptodev_dpaa_sec_autotest',
         'cryptodev_dpaa2_sec_autotest',
+        'cryptodev_mlx5_autotest',
         'cryptodev_null_autotest',
         'cryptodev_octeontx2_autotest',
         'cryptodev_openssl_autotest',
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 39db52b17a..8dbe324b81 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -35,6 +35,7 @@
 #include "test_cryptodev_hash_test_vectors.h"
 #include "test_cryptodev_kasumi_test_vectors.h"
 #include "test_cryptodev_kasumi_hash_test_vectors.h"
+#include "test_cryptodev_mlx5_test_vectors.h"
 #include "test_cryptodev_snow3g_test_vectors.h"
 #include "test_cryptodev_snow3g_hash_test_vectors.h"
 #include "test_cryptodev_zuc_test_vectors.h"
@@ -1267,6 +1268,39 @@ negative_hmac_sha1_testsuite_setup(void)
 	return 0;
 }

+static int
+mlx5_testsuite_setup(void)
Please see how other drivers are enabled in this test app.
We no longer allow PMD specific testsuite.
Please see the cn9k/cn10k PMD which recently got merged.
static int
test_cryptodev_cn9k(void)
{
        return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_CN9K_PMD));
}

The cases which are not supported by mlx5 crypto PMD will get automatically skipped
If all capabilities and feature flags are set properly.

The cases should be generic enough to be used by other PMDs as well.
@Thomas Monjalon: This patchset is adding PMD specific test cases again
which we removed in past 3-4 releases.

Regards,
Akhil

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help