<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="margin: 0in; font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);" class="elementToProof">
Acked-by: Kai Ji <kai.ji@intel.com></div>
<div id="appendonsend"></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<hr style="display: inline-block; width: 98%;">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<b>From:</b> Finn, Emma <emma.finn@intel.com><br>
<b>Sent:</b> Tuesday, July 29, 2025 10:50<br>
<b>To:</b> Ji, Kai <kai.ji@intel.com><br>
<b>Cc:</b> dev@dpdk.org <dev@dpdk.org>; gakhil@marvell.com <gakhil@marvell.com>; Finn, Emma <emma.finn@intel.com><br>
<b>Subject:</b> [v2] app/crypto-perf: Only run AEAD decrypt with valid test vector file.
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-size: 11pt;">The test dummy vector data is incorrect for AEAD decryption, so we should<br>
only run when passed a valid test vector file.<br>
<br>
Signed-off-by: Emma Finn <emma.finn@intel.com><br>
---<br>
app/test-crypto-perf/cperf_options_parsing.c | 19 +++++++++++++++++--<br>
1 file changed, 17 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/app/test-crypto-perf/cperf_options_parsing.c b/app/test-crypto-perf/cperf_options_parsing.c<br>
index 0e0dc4fd06..12623e4cfe 100644<br>
--- a/app/test-crypto-perf/cperf_options_parsing.c<br>
+++ b/app/test-crypto-perf/cperf_options_parsing.c<br>
@@ -1482,11 +1482,26 @@ cperf_options_check(struct cperf_options *options)<br>
}<br>
}<br>
<br>
- if (options->test == CPERF_TEST_TYPE_THROUGHPUT &&<br>
+ if ((options->test == CPERF_TEST_TYPE_THROUGHPUT ||<br>
+ options->test == CPERF_TEST_TYPE_LATENCY) &&<br>
(options->aead_op == RTE_CRYPTO_AEAD_OP_DECRYPT ||<br>
options->auth_op == RTE_CRYPTO_AUTH_OP_VERIFY) &&<br>
!options->out_of_place) {<br>
- RTE_LOG(ERR, USER1, "Only out-of-place is allowed in throughput decryption.\n");<br>
+ RTE_LOG(ERR, USER1, "Only out-of-place is allowed in throughput and"<br>
+ " latency decryption.\n");<br>
+ return -EINVAL;<br>
+ }<br>
+<br>
+ if ((options->test == CPERF_TEST_TYPE_THROUGHPUT ||<br>
+ options->test == CPERF_TEST_TYPE_LATENCY) &&<br>
+ (options->aead_op == RTE_CRYPTO_AEAD_OP_DECRYPT ||<br>
+ options->auth_op == RTE_CRYPTO_AUTH_OP_VERIFY) &&<br>
+ options->test_name == NULL &&<br>
+ options->test_file == NULL) {<br>
+ RTE_LOG(ERR, USER1, "Define path to the file with test"<br>
+ " vectors.\n");<br>
+ RTE_LOG(ERR, USER1, "Define test name to get the correct digest"<br>
+ " from the test vectors.\n");<br>
return -EINVAL;<br>
}<br>
<br>
--<br>
2.34.1<br>
<br>
</div>
</body>
</html>