From: Aakash Sasidharan <asasidharan@marvell.com>
To: Akhil Goyal <gakhil@marvell.com>, Fan Zhang <fanzhang.oss@gmail.com>
Cc: <jerinj@marvell.com>, <anoobj@marvell.com>,
<vvelumuri@marvell.com>, <asasidharan@marvell.com>,
<dev@dpdk.org>
Subject: [PATCH 6/7] test/security: add out of place sgl tests for TLS
Date: Mon, 17 Jun 2024 11:28:40 +0530 [thread overview]
Message-ID: <20240617055841.2359729-7-asasidharan@marvell.com> (raw)
In-Reply-To: <20240617055841.2359729-1-asasidharan@marvell.com>
Add multi segmented test for TLS 1.3 and multi segmented out of place
tests for DTLS 1.2 and TLS 1.3.
Signed-off-by: Aakash Sasidharan <asasidharan@marvell.com>
---
app/test/test_cryptodev.c | 69 ++++++++++++++++++++++-----------------
1 file changed, 39 insertions(+), 30 deletions(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 89b54729eb..0afc13a7b6 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -12224,11 +12224,11 @@ test_tls_1_2_record_proto_display_list(void)
}
static int
-test_tls_1_2_record_proto_sgl(void)
+test_tls_record_proto_sgl(enum rte_security_tls_version tls_version)
{
struct tls_record_test_flags flags = {
.nb_segs_in_mbuf = 5,
- .tls_version = RTE_SECURITY_VERSION_TLS_1_2
+ .tls_version = tls_version
};
struct crypto_testsuite_params *ts_params = &testsuite_params;
struct rte_cryptodev_info dev_info;
@@ -12242,6 +12242,12 @@ test_tls_1_2_record_proto_sgl(void)
return test_tls_record_proto_all(&flags);
}
+static int
+test_tls_1_2_record_proto_sgl(void)
+{
+ return test_tls_record_proto_sgl(RTE_SECURITY_VERSION_TLS_1_2);
+}
+
static int
test_tls_record_proto_sgl_data_walkthrough(enum rte_security_tls_version tls_version)
{
@@ -12577,20 +12583,7 @@ test_dtls_1_2_record_proto_antireplay4096(void)
static int
test_dtls_1_2_record_proto_sgl(void)
{
- struct tls_record_test_flags flags = {
- .nb_segs_in_mbuf = 5,
- .tls_version = RTE_SECURITY_VERSION_DTLS_1_2
- };
- struct crypto_testsuite_params *ts_params = &testsuite_params;
- struct rte_cryptodev_info dev_info;
-
- rte_cryptodev_info_get(ts_params->valid_devs[0], &dev_info);
- if (!(dev_info.feature_flags & RTE_CRYPTODEV_FF_IN_PLACE_SGL)) {
- printf("Device doesn't support in-place scatter-gather. Test Skipped.\n");
- return TEST_SKIPPED;
- }
-
- return test_tls_record_proto_all(&flags);
+ return test_tls_record_proto_sgl(RTE_SECURITY_VERSION_DTLS_1_2);
}
static int
@@ -12599,6 +12592,12 @@ test_dtls_1_2_record_proto_sgl_data_walkthrough(void)
return test_tls_record_proto_sgl_data_walkthrough(RTE_SECURITY_VERSION_DTLS_1_2);
}
+static int
+test_dtls_1_2_record_proto_sgl_oop(void)
+{
+ return test_tls_record_proto_sgl_oop(RTE_SECURITY_VERSION_DTLS_1_2);
+}
+
static int
test_dtls_1_2_record_proto_corrupt_pkt(void)
{
@@ -12819,23 +12818,21 @@ test_tls_1_3_record_proto_data_walkthrough(void)
}
static int
-test_tls_1_3_record_proto_sgl_data_walkthrough(void)
+test_tls_1_3_record_proto_sgl(void)
{
- struct tls_record_test_flags flags = {
- .nb_segs_in_mbuf = 5,
- .tls_version = RTE_SECURITY_VERSION_TLS_1_3,
- .data_walkthrough = true
- };
- struct crypto_testsuite_params *ts_params = &testsuite_params;
- struct rte_cryptodev_info dev_info;
+ return test_tls_record_proto_sgl(RTE_SECURITY_VERSION_TLS_1_3);
+}
- rte_cryptodev_info_get(ts_params->valid_devs[0], &dev_info);
- if (!(dev_info.feature_flags & RTE_CRYPTODEV_FF_IN_PLACE_SGL)) {
- printf("Device doesn't support in-place scatter-gather. Test Skipped.\n");
- return TEST_SKIPPED;
- }
+static int
+test_tls_1_3_record_proto_sgl_data_walkthrough(void)
+{
+ return test_tls_record_proto_sgl_data_walkthrough(RTE_SECURITY_VERSION_TLS_1_3);
+}
- return test_tls_record_proto_all(&flags);
+static int
+test_tls_1_3_record_proto_sgl_oop(void)
+{
+ return test_tls_record_proto_sgl_oop(RTE_SECURITY_VERSION_TLS_1_3);
}
#endif
@@ -18153,6 +18150,10 @@ static struct unit_test_suite dtls12_record_proto_testsuite = {
"Multi-segmented mode data walkthrough",
ut_setup_security, ut_teardown,
test_dtls_1_2_record_proto_sgl_data_walkthrough),
+ TEST_CASE_NAMED_ST(
+ "Multi-segmented mode out of place",
+ ut_setup_security, ut_teardown,
+ test_dtls_1_2_record_proto_sgl_oop),
TEST_CASE_NAMED_ST(
"Packet corruption",
ut_setup_security, ut_teardown,
@@ -18294,10 +18295,18 @@ static struct unit_test_suite tls13_record_proto_testsuite = {
"Data walkthrough combined test alg list",
ut_setup_security, ut_teardown,
test_tls_1_3_record_proto_data_walkthrough),
+ TEST_CASE_NAMED_ST(
+ "Multi-segmented mode",
+ ut_setup_security, ut_teardown,
+ test_tls_1_3_record_proto_sgl),
TEST_CASE_NAMED_ST(
"Multi-segmented mode data walkthrough",
ut_setup_security, ut_teardown,
test_tls_1_3_record_proto_sgl_data_walkthrough),
+ TEST_CASE_NAMED_ST(
+ "Multi-segmented mode out of place",
+ ut_setup_security, ut_teardown,
+ test_tls_1_3_record_proto_sgl_oop),
TEST_CASES_END() /**< NULL terminate unit test array */
}
};
--
2.25.1
next prev parent reply other threads:[~2024-06-17 6:00 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-17 5:58 [PATCH 0/7] Improvements and new test cases Aakash Sasidharan
2024-06-17 5:58 ` [PATCH 1/7] test/crypto: unit tests for padding for TLS-1.3 Aakash Sasidharan
2024-06-17 5:58 ` [PATCH 2/7] test/crypto: add combined mode cases for TLS 1.3 Aakash Sasidharan
2024-06-17 5:58 ` [PATCH 3/7] test/security: add TLS 1.3 data walkthrough tests Aakash Sasidharan
2024-06-17 5:58 ` [PATCH 4/7] test/crypto: verify padding corruption in TLS-1.2 Aakash Sasidharan
2024-06-20 6:52 ` Akhil Goyal
2024-06-17 5:58 ` [PATCH 5/7] test/crypto: verify padding corruption in DTLS-1.2 Aakash Sasidharan
2024-06-17 5:58 ` Aakash Sasidharan [this message]
2024-06-17 5:58 ` [PATCH 7/7] test/security: use single session in data walkthrough test Aakash Sasidharan
2024-06-17 7:21 ` [PATCH 0/7] Improvements and new test cases Anoob Joseph
2024-06-20 14:50 ` [PATCH v2 " Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 1/7] test/crypto: unit tests for padding for TLS-1.3 Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 2/7] test/crypto: add combined mode cases for TLS 1.3 Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 3/7] test/security: add TLS 1.3 data walkthrough tests Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 4/7] test/crypto: verify padding corruption in TLS-1.2 Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 5/7] test/crypto: verify padding corruption in DTLS-1.2 Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 6/7] test/security: add out of place sgl tests for TLS Aakash Sasidharan
2024-06-20 14:50 ` [PATCH v2 7/7] test/security: use single session in data walkthrough test Aakash Sasidharan
2024-06-26 7:30 ` [PATCH v2 0/7] Improvements and new test cases 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=20240617055841.2359729-7-asasidharan@marvell.com \
--to=asasidharan@marvell.com \
--cc=anoobj@marvell.com \
--cc=dev@dpdk.org \
--cc=fanzhang.oss@gmail.com \
--cc=gakhil@marvell.com \
--cc=jerinj@marvell.com \
--cc=vvelumuri@marvell.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).