From: Tejasree Kondoj <ktejas...@marvell.com>
Add multi segmented buffer test cases in TLS suites.
Signed-off-by: Tejasree Kondoj <ktejas...@marvell.com>
Signed-off-by: Vidya Sagar Velumuri <vvelum...@marvell.com>
---
app/test/test_cryptodev.c | 29 ++++++++++++++++++-
app/test/test_cryptodev_security_tls_record.h | 1 +
doc/guides/rel_notes/release_24_03.rst | 4 +++
3 files changed, 33 insertions(+), 1 deletion(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 86e2ccd483..b208cca4e2 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -11699,6 +11699,7 @@ test_tls_record_proto_process(const struct
tls_record_test_data td[],
struct tls_record_test_data res_d[], int nb_td,
bool silent,
const struct tls_record_test_flags *flags)
{
+ int nb_segs = flags->nb_segs_in_mbuf ? flags->nb_segs_in_mbuf : 1;
struct crypto_testsuite_params *ts_params = &testsuite_params;
struct crypto_unittest_params *ut_params = &unittest_params;
struct rte_security_tls_record_xform tls_record_xform;
@@ -11801,7 +11802,7 @@ test_tls_record_proto_process(const struct
tls_record_test_data td[],
for (i = 0; i < nb_td; i++) {
/* Setup source mbuf payload */
ut_params->ibuf = create_segmented_mbuf(ts_params->mbuf_pool,
td[i].input_text.len,
- 1, 0);
+ nb_segs, 0);
pktmbuf_write(ut_params->ibuf, 0, td[i].input_text.len,
td[i].input_text.data);
/* Generate crypto op data structure */
@@ -11958,6 +11959,24 @@ test_tls_record_proto_display_list(void)
return test_tls_record_proto_all(&flags);
}
+static int
+test_tls_record_proto_sgl(void)
+{
+ struct tls_record_test_flags flags = {
+ .nb_segs_in_mbuf = 5
+ };
+ 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);
+}
+
#endif
static int
@@ -16956,6 +16975,10 @@ static struct unit_test_suite
tls12_record_proto_testsuite = {
"Combined test alg list",
ut_setup_security, ut_teardown,
test_tls_record_proto_display_list),
+ TEST_CASE_NAMED_ST(
+ "Multi-segmented mode",
+ ut_setup_security, ut_teardown,
+ test_tls_record_proto_sgl),
TEST_CASES_END() /**< NULL terminate unit test array */
}
};
@@ -17053,6 +17076,10 @@ static struct unit_test_suite
dtls12_record_proto_testsuite = {
"Combined test alg list",
ut_setup_security, ut_teardown,
test_tls_record_proto_display_list),
+ TEST_CASE_NAMED_ST(
+ "Multi-segmented mode",
+ ut_setup_security, ut_teardown,
+ test_tls_record_proto_sgl),
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 82a1776265..0bd83f88f0 100644
--- a/app/test/test_cryptodev_security_tls_record.h
+++ b/app/test/test_cryptodev_security_tls_record.h
@@ -55,6 +55,7 @@ struct tls_record_test_data {
struct tls_record_test_flags {
bool display_alg;
+ int nb_segs_in_mbuf;
};
extern struct tls_record_test_data tls_test_data_aes_128_gcm_v1;
diff --git a/doc/guides/rel_notes/release_24_03.rst
b/doc/guides/rel_notes/release_24_03.rst
index e9c9717706..0d226e9644 100644
--- a/doc/guides/rel_notes/release_24_03.rst
+++ b/doc/guides/rel_notes/release_24_03.rst
@@ -55,6 +55,10 @@ New Features
Also, make sure to start the actual text at the margin.
=======================================================
+* **Added TLS record test suite.**
+
+ Added a new test suite in cryptodev autotest framework to test TLS record
+ security offload by supported cryptodevs.
Removed Items
-------------
--
2.25.1
