diff mbox series

[v2,04/21] test/security: add TLS SG data walkthrough test

Message ID	20240312071805.1354530-5-asasidharan@marvell.com (mailing list archive)
State	Superseded, archived
Delegated to:	akhil goyal
Headers	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 v2 04/21] test/security: add TLS SG data walkthrough test Date: Tue, 12 Mar 2024 12:47:48 +0530 Message-ID: <20240312071805.1354530-5-asasidharan@marvell.com> In-Reply-To: <20240312071805.1354530-1-asasidharan@marvell.com> References: <20240305072213.283205-1-asasidharan@marvell.com> <20240312071805.1354530-1-asasidharan@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain Precedence: list Errors-To: dev-bounces@dpdk.org
Series	Improvements and new test cases \| [v2,00/21] Improvements and new test cases [v2,01/21] test/security: enable AES-GCM in combined mode TLS [v2,02/21] test/security: add TLS 1.2 data walkthrough test [v2,03/21] test/security: add DTLS 1.2 data walkthrough test [v2,04/21] test/security: add TLS SG data walkthrough test [v2,05/21] test/security: unit test for TLS packet corruption [v2,06/21] test/security: unit test for custom content verification [v2,07/21] test/cryptodev: allow zero packet length buffers [v2,08/21] test/security: unit test to verify zero TLS records [v2,09/21] test/security: add unit tests for DTLS-1.2 [v2,10/21] test/security: add TLS/DTLS 1.2 AES-256-SHA384 vectors [v2,11/21] test/security: add DTLS 1.2 anti-replay tests [v2,12/21] test/security: add more DTLS anti-replay window sz [v2,13/21] test/crypto: update verification of header [v2,14/21] test/crypto: add TLS 1.3 vectors [v2,15/21] test/crypto: update framework to verify tls-1.3 [v2,16/21] test/crypto: test to verify hdr corruption in TLS [v2,17/21] test/crypto: test to verify custom content type in TLS [v2,18/21] test/crypto: test to verify zero len record in TLS [v2,19/21] test/crypto: unit tests to verify padding in TLS [v2,20/21] test/crypto: unit tests for padding in DTLS-1.2 [v2,21/21] test/security: add out of place sgl test case for TLS 1.2

Checks

Context	Check	Description
ci/checkpatch	success	coding style OK

Commit Message

Aakash Sasidharan March 12, 2024, 7:17 a.m. UTC

  Add multi segment packet data walkthrough test for TLS 1.2
and DTLS 1.2.

Signed-off-by: Aakash Sasidharan <asasidharan@marvell.com>
---
 app/test/test_cryptodev.c                     | 42 +++++++++++++++++++
 app/test/test_cryptodev_security_tls_record.h |  2 +-
 2 files changed, 43 insertions(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index e0695e9eb3..3591c91130 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -11981,6 +11981,8 @@  test_tls_record_proto_all(const struct tls_record_test_flags *flags)
 
 	for (i = 0; i < RTE_DIM(sec_alg_list); i++) {
 		payload_len = TLS_RECORD_PLAINTEXT_MIN_LEN;
+		if (flags->nb_segs_in_mbuf)
+			payload_len = RTE_MAX(payload_len, flags->nb_segs_in_mbuf);
 again:
 		test_tls_record_td_prepare(sec_alg_list[i].param1, sec_alg_list[i].param2, flags,
 					   td_outb, nb_pkts, payload_len);
@@ -12061,6 +12063,32 @@  test_tls_1_2_record_proto_sgl(void)
 	return test_tls_record_proto_all(&flags);
 }
 
+static int
+test_tls_record_proto_sgl_data_walkthrough(enum rte_security_tls_version tls_version)
+{
+	struct tls_record_test_flags flags = {
+		.nb_segs_in_mbuf = 5,
+		.tls_version = tls_version,
+		.data_walkthrough = true
+	};
+	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);
+}
+
+static int
+test_tls_1_2_record_proto_sgl_data_walkthrough(void)
+{
+	return test_tls_record_proto_sgl_data_walkthrough(RTE_SECURITY_VERSION_TLS_1_2);
+}
+
 static int
 test_dtls_1_2_record_proto_data_walkthrough(void)
 {
@@ -12106,6 +12134,12 @@  test_dtls_1_2_record_proto_sgl(void)
 	return test_tls_record_proto_all(&flags);
 }
 
+static int
+test_dtls_1_2_record_proto_sgl_data_walkthrough(void)
+{
+	return test_tls_record_proto_sgl_data_walkthrough(RTE_SECURITY_VERSION_DTLS_1_2);
+}
+
 #endif
 
 static int
@@ -17171,6 +17205,10 @@  static struct unit_test_suite tls12_record_proto_testsuite  = {
 			"Multi-segmented mode",
 			ut_setup_security, ut_teardown,
 			test_tls_1_2_record_proto_sgl),
+		TEST_CASE_NAMED_ST(
+			"Multi-segmented mode data walkthrough",
+			ut_setup_security, ut_teardown,
+			test_tls_1_2_record_proto_sgl_data_walkthrough),
 		TEST_CASES_END() /**< NULL terminate unit test array */
 	}
 };
@@ -17276,6 +17314,10 @@  static struct unit_test_suite dtls12_record_proto_testsuite  = {
 			"Multi-segmented mode",
 			ut_setup_security, ut_teardown,
 			test_dtls_1_2_record_proto_sgl),
+		TEST_CASE_NAMED_ST(
+			"Multi-segmented mode data walkthrough",
+			ut_setup_security, ut_teardown,
+			test_dtls_1_2_record_proto_sgl_data_walkthrough),
 		TEST_CASES_END() /**< NULL terminate unit test array */
 	}
 };
diff --git a/app/test/test_cryptodev_security_tls_record.h b/app/test/test_cryptodev_security_tls_record.h
index 14f73a5073..34758fb0e1 100644
--- a/app/test/test_cryptodev_security_tls_record.h
+++ b/app/test/test_cryptodev_security_tls_record.h
@@ -85,7 +85,7 @@  struct tls_record_test_data {
 
 struct tls_record_test_flags {
 	bool display_alg;
-	int nb_segs_in_mbuf;
+	uint8_t nb_segs_in_mbuf;
 	bool data_walkthrough;
 	enum rte_security_tls_version tls_version;
 };