From: Deepak Kumar Jain <deepak.k.jain@intel.com>
To: dev@dpdk.org
Cc: pablo.de.lara.guarch@intel.com,
Deepak Kumar Jain <deepak.k.jain@intel.com>
Subject: [dpdk-dev] [PATCH v3 1/3] app/test: cleanup of test code for kasumi
Date: Thu, 15 Sep 2016 11:03:34 +0100 [thread overview]
Message-ID: <1473933815-185834-2-git-send-email-deepak.k.jain@intel.com> (raw)
In-Reply-To: <1473933815-185834-1-git-send-email-deepak.k.jain@intel.com>
Cleanup for easier kasumi enabling.
Changed name of funcitons for clear understanding.
Signed-off-by: Deepak Kumar Jain <deepak.k.jain@intel.com>
---
app/test/test_cryptodev.c | 117 ++++++++++++++++++++++------------------------
1 file changed, 55 insertions(+), 62 deletions(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 67ca912..89d627f 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -1448,74 +1448,67 @@ create_snow3g_kasumi_cipher_hash_operation(const uint8_t *auth_tag,
/* set crypto operation source mbuf */
sym_op->m_src = ut_params->ibuf;
+ /* digest */
+ sym_op->auth.digest.data = (uint8_t *)rte_pktmbuf_append(
+ ut_params->ibuf, auth_tag_len);
- /* iv */
- if (cipher_algo == RTE_CRYPTO_CIPHER_KASUMI_F8)
- iv_pad_len = RTE_ALIGN_CEIL(iv_len, 8);
+ TEST_ASSERT_NOT_NULL(sym_op->auth.digest.data,
+ "no room to append auth tag");
+ ut_params->digest = sym_op->auth.digest.data;
+ sym_op->auth.digest.phys_addr = rte_pktmbuf_mtophys_offset(
+ ut_params->ibuf, data_pad_len);
+ sym_op->auth.digest.length = auth_tag_len;
+ if (op == RTE_CRYPTO_AUTH_OP_GENERATE)
+ memset(sym_op->auth.digest.data, 0, auth_tag_len);
else
- iv_pad_len = RTE_ALIGN_CEIL(iv_len, 16);
-
- sym_op->cipher.iv.data = (uint8_t *)rte_pktmbuf_prepend(
- ut_params->ibuf, iv_pad_len);
- TEST_ASSERT_NOT_NULL(sym_op->cipher.iv.data, "no room to prepend iv");
-
- memset(sym_op->cipher.iv.data, 0, iv_pad_len);
- sym_op->cipher.iv.phys_addr = rte_pktmbuf_mtophys(ut_params->ibuf);
- sym_op->cipher.iv.length = iv_pad_len;
-
- rte_memcpy(sym_op->cipher.iv.data, iv, iv_len);
+ rte_memcpy(sym_op->auth.digest.data, auth_tag, auth_tag_len);
- sym_op->cipher.data.length = cipher_len;
- sym_op->cipher.data.offset = cipher_offset;
+ TEST_HEXDUMP(stdout, "digest:",
+ sym_op->auth.digest.data,
+ sym_op->auth.digest.length);
/* aad */
- /*
- * Always allocate the aad up to the block size.
- * The cryptodev API calls out -
- * - the array must be big enough to hold the AAD, plus any
- * space to round this up to the nearest multiple of the
- * block size (8 bytes for KASUMI and 16 bytes for SNOW3G).
- */
+ /*
+ * Always allocate the aad up to the block size.
+ * The cryptodev API calls out -
+ * - the array must be big enough to hold the AAD, plus any
+ * space to round this up to the nearest multiple of the
+ * block size (8 bytes for KASUMI and 16 bytes for SNOW3G).
+ */
if (auth_algo == RTE_CRYPTO_AUTH_KASUMI_F9)
aad_buffer_len = ALIGN_POW2_ROUNDUP(aad_len, 8);
else
aad_buffer_len = ALIGN_POW2_ROUNDUP(aad_len, 16);
-
sym_op->auth.aad.data =
- (uint8_t *)rte_pktmbuf_mtod(ut_params->ibuf, uint8_t *);
+ (uint8_t *)rte_pktmbuf_prepend(
+ ut_params->ibuf, aad_buffer_len);
TEST_ASSERT_NOT_NULL(sym_op->auth.aad.data,
"no room to prepend aad");
sym_op->auth.aad.phys_addr = rte_pktmbuf_mtophys(
ut_params->ibuf);
sym_op->auth.aad.length = aad_len;
-
- memset(sym_op->auth.aad.data, 0, aad_buffer_len);
+ memset(sym_op->auth.aad.data, 0, aad_buffer_len);
rte_memcpy(sym_op->auth.aad.data, aad, aad_len);
+ TEST_HEXDUMP(stdout, "aad:",
+ sym_op->auth.aad.data, aad_len);
- TEST_HEXDUMP(stdout, "aad:",
- sym_op->auth.aad.data, aad_len);
-
- /* digest */
- sym_op->auth.digest.data = (uint8_t *)rte_pktmbuf_append(
- ut_params->ibuf, auth_tag_len);
-
- TEST_ASSERT_NOT_NULL(sym_op->auth.digest.data,
- "no room to append auth tag");
- ut_params->digest = sym_op->auth.digest.data;
- sym_op->auth.digest.phys_addr = rte_pktmbuf_mtophys_offset(
- ut_params->ibuf, data_pad_len + aad_len);
- sym_op->auth.digest.length = auth_tag_len;
- if (op == RTE_CRYPTO_AUTH_OP_GENERATE)
- memset(sym_op->auth.digest.data, 0, auth_tag_len);
+ /* iv */
+ if (cipher_algo == RTE_CRYPTO_CIPHER_KASUMI_F8)
+ iv_pad_len = RTE_ALIGN_CEIL(iv_len, 8);
else
- rte_memcpy(sym_op->auth.digest.data, auth_tag, auth_tag_len);
-
- TEST_HEXDUMP(stdout, "digest:",
- sym_op->auth.digest.data,
- sym_op->auth.digest.length);
+ iv_pad_len = RTE_ALIGN_CEIL(iv_len, 16);
+ sym_op->cipher.iv.data = (uint8_t *)rte_pktmbuf_prepend(
+ ut_params->ibuf, iv_pad_len);
- sym_op->auth.data.length = auth_len;
- sym_op->auth.data.offset = auth_offset;
+ TEST_ASSERT_NOT_NULL(sym_op->cipher.iv.data, "no room to prepend iv");
+ memset(sym_op->cipher.iv.data, 0, iv_pad_len);
+ sym_op->cipher.iv.phys_addr = rte_pktmbuf_mtophys(ut_params->ibuf);
+ sym_op->cipher.iv.length = iv_pad_len;
+ rte_memcpy(sym_op->cipher.iv.data, iv, iv_len);
+ sym_op->cipher.data.length = cipher_len;
+ sym_op->cipher.data.offset = cipher_offset + auth_offset;
+ sym_op->auth.data.length = auth_len;
+ sym_op->auth.data.offset = auth_offset + cipher_offset;
return 0;
}
@@ -2655,7 +2648,7 @@ static int test_snow3g_decryption_oop(const struct snow3g_test_data *tdata)
}
static int
-test_snow3g_authenticated_encryption(const struct snow3g_test_data *tdata)
+test_snow3g_cipher_auth(const struct snow3g_test_data *tdata)
{
struct crypto_testsuite_params *ts_params = &testsuite_params;
struct crypto_unittest_params *ut_params = &unittest_params;
@@ -2714,12 +2707,12 @@ test_snow3g_authenticated_encryption(const struct snow3g_test_data *tdata)
ut_params->obuf = ut_params->op->sym->m_src;
if (ut_params->obuf)
ciphertext = rte_pktmbuf_mtod(ut_params->obuf, uint8_t *)
- + tdata->iv.len;
+ + tdata->iv.len + tdata->aad.len;
else
ciphertext = plaintext;
TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
-
+ TEST_HEXDUMP(stdout, "OUTPUT BUFFER:", ut_params->obuf, 512);
/* Validate obuf */
TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
ciphertext,
@@ -2728,7 +2721,7 @@ test_snow3g_authenticated_encryption(const struct snow3g_test_data *tdata)
"Snow3G Ciphertext data not as expected");
ut_params->digest = rte_pktmbuf_mtod(ut_params->obuf, uint8_t *)
- + plaintext_pad_len + tdata->aad.len;
+ + plaintext_pad_len + tdata->aad.len + tdata->iv.len;
/* Validate obuf */
TEST_ASSERT_BUFFERS_ARE_EQUAL(
@@ -2739,7 +2732,7 @@ test_snow3g_authenticated_encryption(const struct snow3g_test_data *tdata)
return 0;
}
static int
-test_snow3g_encrypted_authentication(const struct snow3g_test_data *tdata)
+test_snow3g_auth_cipher(const struct snow3g_test_data *tdata)
{
struct crypto_testsuite_params *ts_params = &testsuite_params;
struct crypto_unittest_params *ut_params = &unittest_params;
@@ -2973,15 +2966,15 @@ test_snow3g_decryption_test_case_5(void)
return test_snow3g_decryption(&snow3g_test_case_5);
}
static int
-test_snow3g_authenticated_encryption_test_case_1(void)
+test_snow3g_cipher_auth_test_case_1(void)
{
- return test_snow3g_authenticated_encryption(&snow3g_test_case_3);
+ return test_snow3g_cipher_auth(&snow3g_test_case_3);
}
static int
-test_snow3g_encrypted_authentication_test_case_1(void)
+test_snow3g_auth_cipher_test_case_1(void)
{
- return test_snow3g_encrypted_authentication(&snow3g_test_case_6);
+ return test_snow3g_auth_cipher(&snow3g_test_case_6);
}
/* ***** AES-GCM Tests ***** */
@@ -4122,9 +4115,9 @@ static struct unit_test_suite cryptodev_qat_testsuite = {
TEST_CASE_ST(ut_setup, ut_teardown,
test_snow3g_hash_verify_test_case_3),
TEST_CASE_ST(ut_setup, ut_teardown,
- test_snow3g_authenticated_encryption_test_case_1),
+ test_snow3g_cipher_auth_test_case_1),
TEST_CASE_ST(ut_setup, ut_teardown,
- test_snow3g_encrypted_authentication_test_case_1),
+ test_snow3g_auth_cipher_test_case_1),
/** HMAC_MD5 Authentication */
TEST_CASE_ST(ut_setup, ut_teardown,
@@ -4323,9 +4316,9 @@ static struct unit_test_suite cryptodev_sw_snow3g_testsuite = {
TEST_CASE_ST(ut_setup, ut_teardown,
test_snow3g_hash_verify_test_case_6),
TEST_CASE_ST(ut_setup, ut_teardown,
- test_snow3g_authenticated_encryption_test_case_1),
+ test_snow3g_cipher_auth_test_case_1),
TEST_CASE_ST(ut_setup, ut_teardown,
- test_snow3g_encrypted_authentication_test_case_1),
+ test_snow3g_auth_cipher_test_case_1),
TEST_CASES_END() /**< NULL terminate unit test array */
}
--
2.5.5
next prev parent reply other threads:[~2016-09-15 10:03 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-23 21:35 [dpdk-dev] [PATCH 0/4] add kasumi in Intel(R) QuickAssist driver Deepak Kumar Jain
2016-08-23 21:35 ` [dpdk-dev] [PATCH 1/4] crypto/qat: enable Kasumi F9 support in QAT driver Deepak Kumar Jain
2016-08-23 21:35 ` [dpdk-dev] [PATCH 2/4] app/test: add Kasumi f9 tests in QAT test suite Deepak Kumar Jain
2016-08-23 21:35 ` [dpdk-dev] [PATCH 3/4] crypto/qat: enable support of Kasumi F8 in QAT cryptodev Deepak Kumar Jain
2016-08-23 21:35 ` [dpdk-dev] [PATCH 4/4] app/test: add kasumi f8 test into QAT testsuite Deepak Kumar Jain
2016-08-25 13:23 ` [dpdk-dev] [PATCH v2 0/4] add kasumi in Intel(R) QuickAssist driver Deepak Kumar Jain
2016-08-25 13:23 ` [dpdk-dev] [PATCH v2 1/4] crypto/qat: enable Kasumi F9 support in QAT driver Deepak Kumar Jain
2016-08-25 13:23 ` [dpdk-dev] [PATCH v2 2/4] app/test: add Kasumi f9 tests in QAT test suite Deepak Kumar Jain
2016-08-25 13:23 ` [dpdk-dev] [PATCH v2 3/4] crypto/qat: enable support of Kasumi F8 in QAT cryptodev Deepak Kumar Jain
2016-08-25 13:23 ` [dpdk-dev] [PATCH v2 4/4] app/test: add kasumi f8 test into QAT testsuite Deepak Kumar Jain
2016-09-15 10:03 ` [dpdk-dev] [PATCH v3 0/3] add kasumi in Intel(R) QuickAssist driver Deepak Kumar Jain
2016-09-15 10:03 ` Deepak Kumar Jain [this message]
2016-09-15 14:44 ` [dpdk-dev] [PATCH v3 1/3] app/test: cleanup of test code for kasumi Trahe, Fiona
2016-09-17 0:59 ` De Lara Guarch, Pablo
2016-09-19 9:17 ` Jain, Deepak K
2016-09-15 10:03 ` [dpdk-dev] [PATCH v3 2/3] crypto/qat: add Kasumi support in Intel(R) QAT driver Deepak Kumar Jain
2016-09-15 14:45 ` Trahe, Fiona
2016-09-16 23:13 ` De Lara Guarch, Pablo
2016-09-15 10:05 ` [dpdk-dev] [PATCH v3 3/3] app/test: add Kasumi tests in QAT test suite Deepak Kumar Jain
2016-09-15 14:46 ` Trahe, Fiona
2016-09-16 23:04 ` De Lara Guarch, Pablo
2016-09-19 11:00 ` [dpdk-dev] [PATCH v4 0/4] add kasumi in Intel(R) QuickAssist driver Deepak Kumar Jain
2016-09-19 11:00 ` [dpdk-dev] [PATCH v4 1/4] app/test: crypto test code cleanup Deepak Kumar Jain
2016-09-19 11:00 ` [dpdk-dev] [PATCH v4 2/4] app/test: rename functions name Deepak Kumar Jain
2016-09-19 11:00 ` [dpdk-dev] [PATCH v4 3/4] crypto/qat: add Kasumi support in Intel(R) QAT driver Deepak Kumar Jain
2016-09-19 11:00 ` [dpdk-dev] [PATCH v4 4/4] app/test: add Kasumi tests in QAT test suite Deepak Kumar Jain
2016-09-19 23:05 ` [dpdk-dev] [PATCH v4 0/4] add kasumi in Intel(R) QuickAssist driver De Lara Guarch, Pablo
2016-09-07 18:06 ` [dpdk-dev] [PATCH " De Lara Guarch, Pablo
2016-09-07 19:33 ` Jain, Deepak K
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=1473933815-185834-2-git-send-email-deepak.k.jain@intel.com \
--to=deepak.k.jain@intel.com \
--cc=dev@dpdk.org \
--cc=pablo.de.lara.guarch@intel.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).