DPDK patches and discussions
 help / color / mirror / Atom feed
From: Olivier Matz <olivier.matz@6wind.com>
To: dev@dpdk.org, Declan Doherty <declan.doherty@intel.com>
Cc: Pavan Nikhilesh Bhagavatula <pbhagavatula@caviumnetworks.com>
Subject: Re: [dpdk-dev] [PATCH 1/2] test: rely on dynamic log level to display hexdumps
Date: Tue, 16 Jan 2018 16:54:46 +0100	[thread overview]
Message-ID: <20180116155446.r4gsk65garskwhjg@platinum> (raw)
In-Reply-To: <20171208132122.11101-1-olivier.matz@6wind.com>

Hi Declan,

Since it mostly impacts cryptodev test, do you have any comment on this
patchset?

Thanks,
Olivier



On Fri, Dec 08, 2017 at 02:21:21PM +0100, Olivier Matz wrote:
> Instead of relying on a compile-time option, use the global log-level
> to decide if the hexdumps should be displayed in the tests.
> 
> Valitation:
> 
>   # build/app/test --no-huge
>   RTE>>crc_autotest
>   Test OK
> 
>   # build/app/test --no-huge --log-level=8
>   RTE>>crc_autotest
>   [many hexdumps...]
>   Test OK
> 
> Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
> ---
>  test/test/test.h                       |  12 +--
>  test/test/test_crc.c                   |   2 +-
>  test/test/test_cryptodev.c             | 144 +++++++++++++++++----------------
>  test/test/test_cryptodev_blockcipher.c |   8 +-
>  4 files changed, 85 insertions(+), 81 deletions(-)
> 
> diff --git a/test/test/test.h b/test/test/test.h
> index 08ffe949c..fcb983363 100644
> --- a/test/test/test.h
> +++ b/test/test/test.h
> @@ -37,6 +37,7 @@
>  #include <stddef.h>
>  #include <sys/queue.h>
>  
> +#include <rte_hexdump.h>
>  #include <rte_common.h>
>  #include <rte_log.h>
>  
> @@ -204,11 +205,12 @@ struct unit_test_case {
>  
>  #define TEST_CASES_END() { NULL, NULL, NULL, NULL, 0 }
>  
> -#if RTE_LOG_LEVEL >= RTE_LOG_DEBUG
> -#define TEST_HEXDUMP(file, title, buf, len) rte_hexdump(file, title, buf, len)
> -#else
> -#define TEST_HEXDUMP(file, title, buf, len) do {} while (0)
> -#endif
> +static inline void
> +debug_hexdump(FILE *file, const char *title, const void *buf, size_t len)
> +{
> +	if (rte_log_get_global_level() == RTE_LOG_DEBUG)
> +		rte_hexdump(file, title, buf, len);
> +}
>  
>  struct unit_test_suite {
>  	const char *suite_name;
> diff --git a/test/test/test_crc.c b/test/test/test_crc.c
> index 9f2a17d49..d45cd4cfb 100644
> --- a/test/test/test_crc.c
> +++ b/test/test/test_crc.c
> @@ -86,7 +86,7 @@ crc_calc(const uint8_t *vec,
>  	uint32_t ret = rte_net_crc_calc(vec, vec_len, type);
>  
>  	/* dump data on console */
> -	TEST_HEXDUMP(stdout, NULL, vec, vec_len);
> +	debug_hexdump(stdout, NULL, vec, vec_len);
>  
>  	return  ret;
>  }
> diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c
> index 1bed65dad..6662bf373 100644
> --- a/test/test/test_cryptodev.c
> +++ b/test/test/test_cryptodev.c
> @@ -2052,7 +2052,7 @@ create_wireless_algo_hash_session(uint8_t dev_id,
>  
>  	memcpy(hash_key, key, key_len);
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	/* Setup Authentication Parameters */
>  	ut_params->auth_xform.type = RTE_CRYPTO_SYM_XFORM_AUTH;
> @@ -2099,7 +2099,7 @@ create_wireless_algo_cipher_session(uint8_t dev_id,
>  	ut_params->cipher_xform.cipher.iv.offset = IV_OFFSET;
>  	ut_params->cipher_xform.cipher.iv.length = iv_len;
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	/* Create Crypto session */
>  	ut_params->sess = rte_cryptodev_sym_session_create(
> @@ -2215,7 +2215,7 @@ create_wireless_algo_cipher_auth_session(uint8_t dev_id,
>  	ut_params->cipher_xform.cipher.iv.offset = IV_OFFSET;
>  	ut_params->cipher_xform.cipher.iv.length = cipher_iv_len;
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	/* Create Crypto session*/
>  	ut_params->sess = rte_cryptodev_sym_session_create(
> @@ -2274,7 +2274,7 @@ create_wireless_cipher_auth_session(uint8_t dev_id,
>  	ut_params->cipher_xform.cipher.iv.length = cipher_iv_len;
>  
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	/* Create Crypto session*/
>  	ut_params->sess = rte_cryptodev_sym_session_create(
> @@ -2336,7 +2336,7 @@ create_wireless_algo_auth_cipher_session(uint8_t dev_id,
>  	ut_params->cipher_xform.cipher.iv.offset = IV_OFFSET;
>  	ut_params->cipher_xform.cipher.iv.length = cipher_iv_len;
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	/* Create Crypto session*/
>  	ut_params->sess = rte_cryptodev_sym_session_create(
> @@ -2393,7 +2393,7 @@ create_wireless_algo_hash_operation(const uint8_t *auth_tag,
>  	else
>  		rte_memcpy(sym_op->auth.digest.data, auth_tag, auth_tag_len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  		sym_op->auth.digest.data,
>  		auth_tag_len);
>  
> @@ -2449,7 +2449,7 @@ create_wireless_cipher_hash_operation(const struct wireless_test_data *tdata,
>  	else
>  		rte_memcpy(sym_op->auth.digest.data, auth_tag, auth_tag_len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  		sym_op->auth.digest.data,
>  		auth_tag_len);
>  
> @@ -2516,7 +2516,7 @@ create_wireless_algo_cipher_hash_operation(const uint8_t *auth_tag,
>  	else
>  		rte_memcpy(sym_op->auth.digest.data, auth_tag, auth_tag_len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  		sym_op->auth.digest.data,
>  		auth_tag_len);
>  
> @@ -2572,7 +2572,7 @@ create_wireless_algo_auth_cipher_operation(unsigned int auth_tag_len,
>  
>  	memset(sym_op->auth.digest.data, 0, auth_tag_len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  			sym_op->auth.digest.data,
>  			auth_tag_len);
>  
> @@ -3005,7 +3005,7 @@ test_kasumi_encryption(const struct kasumi_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create KASUMI operation */
>  	retval = create_wireless_algo_cipher_operation(tdata->cipher_iv.data,
> @@ -3025,7 +3025,7 @@ test_kasumi_encryption(const struct kasumi_test_data *tdata)
>  	else
>  		ciphertext = plaintext + (tdata->validCipherOffsetInBits.len >> 3);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	const uint8_t *reference_ciphertext = tdata->ciphertext.data +
>  				(tdata->validCipherOffsetInBits.len >> 3);
> @@ -3105,7 +3105,7 @@ test_kasumi_encryption_sgl(const struct kasumi_test_data *tdata)
>  				plaintext_len, buffer);
>  
>  	/* Validate obuf */
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	const uint8_t *reference_ciphertext = tdata->ciphertext.data +
>  				(tdata->validCipherOffsetInBits.len >> 3);
> @@ -3154,7 +3154,7 @@ test_kasumi_encryption_oop(const struct kasumi_test_data *tdata)
>  	rte_pktmbuf_append(ut_params->obuf, plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create KASUMI operation */
>  	retval = create_wireless_algo_cipher_operation_oop(tdata->cipher_iv.data,
> @@ -3174,7 +3174,7 @@ test_kasumi_encryption_oop(const struct kasumi_test_data *tdata)
>  	else
>  		ciphertext = plaintext + (tdata->validCipherOffsetInBits.len >> 3);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	const uint8_t *reference_ciphertext = tdata->ciphertext.data +
>  				(tdata->validCipherOffsetInBits.len >> 3);
> @@ -3301,7 +3301,7 @@ test_kasumi_decryption_oop(const struct kasumi_test_data *tdata)
>  	rte_pktmbuf_append(ut_params->obuf, ciphertext_pad_len);
>  	memcpy(ciphertext, tdata->ciphertext.data, ciphertext_len);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, ciphertext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, ciphertext_len);
>  
>  	/* Create KASUMI operation */
>  	retval = create_wireless_algo_cipher_operation_oop(tdata->cipher_iv.data,
> @@ -3321,7 +3321,7 @@ test_kasumi_decryption_oop(const struct kasumi_test_data *tdata)
>  	else
>  		plaintext = ciphertext + (tdata->validCipherOffsetInBits.len >> 3);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, ciphertext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, ciphertext_len);
>  
>  	const uint8_t *reference_plaintext = tdata->plaintext.data +
>  				(tdata->validCipherOffsetInBits.len >> 3);
> @@ -3368,7 +3368,7 @@ test_kasumi_decryption(const struct kasumi_test_data *tdata)
>  				ciphertext_pad_len);
>  	memcpy(ciphertext, tdata->ciphertext.data, ciphertext_len);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, ciphertext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, ciphertext_len);
>  
>  	/* Create KASUMI operation */
>  	retval = create_wireless_algo_cipher_operation(tdata->cipher_iv.data,
> @@ -3388,7 +3388,7 @@ test_kasumi_decryption(const struct kasumi_test_data *tdata)
>  	else
>  		plaintext = ciphertext + (tdata->validCipherOffsetInBits.len >> 3);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, ciphertext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, ciphertext_len);
>  
>  	const uint8_t *reference_plaintext = tdata->plaintext.data +
>  				(tdata->validCipherOffsetInBits.len >> 3);
> @@ -3435,7 +3435,7 @@ test_snow3g_encryption(const struct snow3g_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create SNOW 3G operation */
>  	retval = create_wireless_algo_cipher_operation(tdata->cipher_iv.data,
> @@ -3455,7 +3455,7 @@ test_snow3g_encryption(const struct snow3g_test_data *tdata)
>  	else
>  		ciphertext = plaintext;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
> @@ -3508,7 +3508,7 @@ test_snow3g_encryption_oop(const struct snow3g_test_data *tdata)
>  	rte_pktmbuf_append(ut_params->obuf, plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create SNOW 3G operation */
>  	retval = create_wireless_algo_cipher_operation_oop(tdata->cipher_iv.data,
> @@ -3528,7 +3528,7 @@ test_snow3g_encryption_oop(const struct snow3g_test_data *tdata)
>  	else
>  		ciphertext = plaintext;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
> @@ -3606,7 +3606,7 @@ test_snow3g_encryption_oop_sgl(const struct snow3g_test_data *tdata)
>  		ciphertext = rte_pktmbuf_read(ut_params->ibuf, 0,
>  				plaintext_len, buffer);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
> @@ -3764,7 +3764,7 @@ static int test_snow3g_decryption(const struct snow3g_test_data *tdata)
>  				ciphertext_pad_len);
>  	memcpy(ciphertext, tdata->ciphertext.data, ciphertext_len);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, ciphertext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, ciphertext_len);
>  
>  	/* Create SNOW 3G operation */
>  	retval = create_wireless_algo_cipher_operation(tdata->cipher_iv.data,
> @@ -3783,7 +3783,7 @@ static int test_snow3g_decryption(const struct snow3g_test_data *tdata)
>  	else
>  		plaintext = ciphertext;
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, ciphertext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, ciphertext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(plaintext,
> @@ -3837,7 +3837,7 @@ static int test_snow3g_decryption_oop(const struct snow3g_test_data *tdata)
>  	rte_pktmbuf_append(ut_params->obuf, ciphertext_pad_len);
>  	memcpy(ciphertext, tdata->ciphertext.data, ciphertext_len);
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, ciphertext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, ciphertext_len);
>  
>  	/* Create SNOW 3G operation */
>  	retval = create_wireless_algo_cipher_operation_oop(tdata->cipher_iv.data,
> @@ -3856,7 +3856,7 @@ static int test_snow3g_decryption_oop(const struct snow3g_test_data *tdata)
>  	else
>  		plaintext = ciphertext;
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, ciphertext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, ciphertext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(plaintext,
> @@ -3916,7 +3916,7 @@ test_zuc_cipher_auth(const struct wireless_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create ZUC operation */
>  	retval = create_zuc_cipher_hash_generate_operation(tdata);
> @@ -3932,7 +3932,7 @@ test_zuc_cipher_auth(const struct wireless_test_data *tdata)
>  	else
>  		ciphertext = plaintext;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
>  			ciphertext,
> @@ -3989,7 +3989,7 @@ test_snow3g_cipher_auth(const struct snow3g_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create SNOW 3G operation */
>  	retval = create_wireless_algo_cipher_hash_operation(tdata->digest.data,
> @@ -4014,7 +4014,7 @@ test_snow3g_cipher_auth(const struct snow3g_test_data *tdata)
>  	else
>  		ciphertext = plaintext;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
>  			ciphertext,
> @@ -4071,7 +4071,7 @@ test_snow3g_auth_cipher(const struct snow3g_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create SNOW 3G operation */
>  	retval = create_wireless_algo_auth_cipher_operation(
> @@ -4098,7 +4098,7 @@ test_snow3g_auth_cipher(const struct snow3g_test_data *tdata)
>  
>  	ut_params->digest = rte_pktmbuf_mtod(ut_params->obuf, uint8_t *)
>  			+ plaintext_pad_len;
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
> @@ -4154,7 +4154,7 @@ test_kasumi_auth_cipher(const struct kasumi_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create KASUMI operation */
>  	retval = create_wireless_algo_auth_cipher_operation(tdata->digest.len,
> @@ -4240,7 +4240,7 @@ test_kasumi_cipher_auth(const struct kasumi_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create KASUMI operation */
>  	retval = create_wireless_algo_cipher_hash_operation(tdata->digest.data,
> @@ -4332,7 +4332,7 @@ test_zuc_encryption(const struct wireless_test_data *tdata)
>  				plaintext_pad_len);
>  	memcpy(plaintext, tdata->plaintext.data, plaintext_len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, plaintext_len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, plaintext_len);
>  
>  	/* Create ZUC operation */
>  	retval = create_wireless_algo_cipher_operation(tdata->cipher_iv.data,
> @@ -4352,7 +4352,7 @@ test_zuc_encryption(const struct wireless_test_data *tdata)
>  	else
>  		ciphertext = plaintext;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
> @@ -4439,7 +4439,7 @@ test_zuc_encryption_sgl(const struct wireless_test_data *tdata)
>  			0, plaintext_len, ciphertext_buffer);
>  
>  	/* Validate obuf */
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, plaintext_len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, plaintext_len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL_BIT(
> @@ -5083,7 +5083,7 @@ create_aead_session(uint8_t dev_id, enum rte_crypto_aead_algorithm algo,
>  	ut_params->aead_xform.aead.digest_length = auth_len;
>  	ut_params->aead_xform.aead.aad_length = aad_len;
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	/* Create Crypto session*/
>  	ut_params->sess = rte_cryptodev_sym_session_create(
> @@ -5122,7 +5122,7 @@ create_aead_xform(struct rte_crypto_op *op,
>  	sym_op->xform->aead.digest_length = auth_len;
>  	sym_op->xform->aead.aad_length = aad_len;
>  
> -	TEST_HEXDUMP(stdout, "key:", key, key_len);
> +	debug_hexdump(stdout, "key:", key, key_len);
>  
>  	return 0;
>  }
> @@ -5157,7 +5157,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  				rte_pktmbuf_iova(ut_params->ibuf);
>  		/* Copy AAD 18 bytes after the AAD pointer, according to the API */
>  		memcpy(sym_op->aead.aad.data + 18, tdata->aad.data, tdata->aad.len);
> -		TEST_HEXDUMP(stdout, "aad:", sym_op->aead.aad.data,
> +		debug_hexdump(stdout, "aad:", sym_op->aead.aad.data,
>  			tdata->aad.len);
>  
>  		/* Append IV at the end of the crypto operation*/
> @@ -5166,7 +5166,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  
>  		/* Copy IV 1 byte after the IV pointer, according to the API */
>  		rte_memcpy(iv_ptr + 1, tdata->iv.data, tdata->iv.len);
> -		TEST_HEXDUMP(stdout, "iv:", iv_ptr,
> +		debug_hexdump(stdout, "iv:", iv_ptr,
>  			tdata->iv.len);
>  	} else {
>  		aad_pad_len = RTE_ALIGN_CEIL(tdata->aad.len, 16);
> @@ -5178,7 +5178,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  		sym_op->aead.aad.phys_addr =
>  				rte_pktmbuf_iova(ut_params->ibuf);
>  		memcpy(sym_op->aead.aad.data, tdata->aad.data, tdata->aad.len);
> -		TEST_HEXDUMP(stdout, "aad:", sym_op->aead.aad.data,
> +		debug_hexdump(stdout, "aad:", sym_op->aead.aad.data,
>  			tdata->aad.len);
>  
>  		/* Append IV at the end of the crypto operation*/
> @@ -5186,7 +5186,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  				uint8_t *, IV_OFFSET);
>  
>  		rte_memcpy(iv_ptr, tdata->iv.data, tdata->iv.len);
> -		TEST_HEXDUMP(stdout, "iv:", iv_ptr,
> +		debug_hexdump(stdout, "iv:", iv_ptr,
>  			tdata->iv.len);
>  	}
>  
> @@ -5198,7 +5198,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  		TEST_ASSERT_NOT_NULL(plaintext, "no room to append plaintext");
>  
>  		memcpy(plaintext, tdata->plaintext.data, tdata->plaintext.len);
> -		TEST_HEXDUMP(stdout, "plaintext:", plaintext,
> +		debug_hexdump(stdout, "plaintext:", plaintext,
>  				tdata->plaintext.len);
>  
>  		if (ut_params->obuf) {
> @@ -5220,7 +5220,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  
>  		memcpy(ciphertext, tdata->ciphertext.data,
>  				tdata->ciphertext.len);
> -		TEST_HEXDUMP(stdout, "ciphertext:", ciphertext,
> +		debug_hexdump(stdout, "ciphertext:", ciphertext,
>  				tdata->ciphertext.len);
>  
>  		if (ut_params->obuf) {
> @@ -5260,7 +5260,7 @@ create_aead_operation(enum rte_crypto_aead_operation op,
>  
>  		rte_memcpy(sym_op->aead.digest.data, tdata->auth_tag.data,
>  			tdata->auth_tag.len);
> -		TEST_HEXDUMP(stdout, "digest:",
> +		debug_hexdump(stdout, "digest:",
>  			sym_op->aead.digest.data,
>  			tdata->auth_tag.len);
>  	}
> @@ -5334,8 +5334,8 @@ test_authenticated_encryption(const struct aead_test_data *tdata)
>  		auth_tag = ciphertext + plaintext_pad_len;
>  	}
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, tdata->ciphertext.len);
> -	TEST_HEXDUMP(stdout, "auth tag:", auth_tag, tdata->auth_tag.len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, tdata->ciphertext.len);
> +	debug_hexdump(stdout, "auth tag:", auth_tag, tdata->auth_tag.len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
> @@ -5548,7 +5548,7 @@ test_authenticated_decryption(const struct aead_test_data *tdata)
>  				uint8_t *,
>  				ut_params->op->sym->cipher.data.offset);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, tdata->ciphertext.len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, tdata->ciphertext.len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
> @@ -5753,8 +5753,8 @@ test_authenticated_encryption_oop(const struct aead_test_data *tdata)
>  			ut_params->op->sym->cipher.data.offset);
>  	auth_tag = ciphertext + plaintext_pad_len;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, tdata->ciphertext.len);
> -	TEST_HEXDUMP(stdout, "auth tag:", auth_tag, tdata->auth_tag.len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, tdata->ciphertext.len);
> +	debug_hexdump(stdout, "auth tag:", auth_tag, tdata->auth_tag.len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
> @@ -5827,7 +5827,7 @@ test_authenticated_decryption_oop(const struct aead_test_data *tdata)
>  	plaintext = rte_pktmbuf_mtod_offset(ut_params->obuf, uint8_t *,
>  			ut_params->op->sym->cipher.data.offset);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, tdata->ciphertext.len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, tdata->ciphertext.len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
> @@ -5903,8 +5903,8 @@ test_authenticated_encryption_sessionless(
>  			ut_params->op->sym->cipher.data.offset);
>  	auth_tag = ciphertext + plaintext_pad_len;
>  
> -	TEST_HEXDUMP(stdout, "ciphertext:", ciphertext, tdata->ciphertext.len);
> -	TEST_HEXDUMP(stdout, "auth tag:", auth_tag, tdata->auth_tag.len);
> +	debug_hexdump(stdout, "ciphertext:", ciphertext, tdata->ciphertext.len);
> +	debug_hexdump(stdout, "auth tag:", auth_tag, tdata->auth_tag.len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
> @@ -5981,7 +5981,7 @@ test_authenticated_decryption_sessionless(
>  	plaintext = rte_pktmbuf_mtod_offset(ut_params->ibuf, uint8_t *,
>  			ut_params->op->sym->cipher.data.offset);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, tdata->ciphertext.len);
> +	debug_hexdump(stdout, "plaintext:", plaintext, tdata->ciphertext.len);
>  
>  	/* Validate obuf */
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
> @@ -6969,7 +6969,7 @@ create_gmac_operation(enum rte_crypto_auth_operation op,
>  	if (op == RTE_CRYPTO_AUTH_OP_VERIFY) {
>  		rte_memcpy(sym_op->auth.digest.data, tdata->gmac_tag.data,
>  				tdata->gmac_tag.len);
> -		TEST_HEXDUMP(stdout, "digest:",
> +		debug_hexdump(stdout, "digest:",
>  				sym_op->auth.digest.data,
>  				tdata->gmac_tag.len);
>  	}
> @@ -6979,7 +6979,7 @@ create_gmac_operation(enum rte_crypto_auth_operation op,
>  
>  	rte_memcpy(iv_ptr, tdata->iv.data, tdata->iv.len);
>  
> -	TEST_HEXDUMP(stdout, "iv:", iv_ptr, tdata->iv.len);
> +	debug_hexdump(stdout, "iv:", iv_ptr, tdata->iv.len);
>  
>  	sym_op->cipher.data.length = 0;
>  	sym_op->cipher.data.offset = 0;
> @@ -7069,7 +7069,7 @@ test_AES_GMAC_authentication(const struct gmac_test_data *tdata)
>  	TEST_ASSERT_NOT_NULL(plaintext, "no room to append plaintext");
>  
>  	memcpy(plaintext, tdata->plaintext.data, tdata->plaintext.len);
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext,
> +	debug_hexdump(stdout, "plaintext:", plaintext,
>  			tdata->plaintext.len);
>  
>  	retval = create_gmac_operation(RTE_CRYPTO_AUTH_OP_GENERATE,
> @@ -7095,7 +7095,7 @@ test_AES_GMAC_authentication(const struct gmac_test_data *tdata)
>  		auth_tag = plaintext + plaintext_pad_len;
>  	}
>  
> -	TEST_HEXDUMP(stdout, "auth tag:", auth_tag, tdata->gmac_tag.len);
> +	debug_hexdump(stdout, "auth tag:", auth_tag, tdata->gmac_tag.len);
>  
>  	TEST_ASSERT_BUFFERS_ARE_EQUAL(
>  			auth_tag,
> @@ -7173,7 +7173,7 @@ test_AES_GMAC_authentication_verify(const struct gmac_test_data *tdata)
>  	TEST_ASSERT_NOT_NULL(plaintext, "no room to append plaintext");
>  
>  	memcpy(plaintext, tdata->plaintext.data, tdata->plaintext.len);
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext,
> +	debug_hexdump(stdout, "plaintext:", plaintext,
>  			tdata->plaintext.len);
>  
>  	retval = create_gmac_operation(RTE_CRYPTO_AUTH_OP_VERIFY,
> @@ -7495,7 +7495,7 @@ create_auth_operation(struct crypto_testsuite_params *ts_params,
>  				reference->digest.data,
>  				reference->digest.len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  			sym_op->auth.digest.data,
>  			reference->digest.len);
>  
> @@ -7542,7 +7542,7 @@ create_auth_GMAC_operation(struct crypto_testsuite_params *ts_params,
>  				reference->digest.data,
>  				reference->digest.len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  			sym_op->auth.digest.data,
>  			reference->digest.len);
>  
> @@ -7595,7 +7595,7 @@ create_cipher_auth_operation(struct crypto_testsuite_params *ts_params,
>  				reference->digest.data,
>  				reference->digest.len);
>  
> -	TEST_HEXDUMP(stdout, "digest:",
> +	debug_hexdump(stdout, "digest:",
>  			sym_op->auth.digest.data,
>  			reference->digest.len);
>  
> @@ -7668,7 +7668,8 @@ test_authentication_verify_fail_when_data_corruption(
>  	TEST_ASSERT_NOT_NULL(plaintext, "no room to append plaintext");
>  	memcpy(plaintext, reference->plaintext.data, reference->plaintext.len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, reference->plaintext.len);
> +	debug_hexdump(stdout, "plaintext:", plaintext,
> +		reference->plaintext.len);
>  
>  	/* Create operation */
>  	retval = create_auth_verify_operation(ts_params, ut_params, reference);
> @@ -7726,7 +7727,8 @@ test_authentication_verify_GMAC_fail_when_corruption(
>  	TEST_ASSERT_NOT_NULL(plaintext, "no room to append plaintext");
>  	memcpy(plaintext, reference->plaintext.data, reference->plaintext.len);
>  
> -	TEST_HEXDUMP(stdout, "plaintext:", plaintext, reference->plaintext.len);
> +	debug_hexdump(stdout, "plaintext:", plaintext,
> +		reference->plaintext.len);
>  
>  	/* Create operation */
>  	retval = create_auth_verify_GMAC_operation(ts_params,
> @@ -7845,7 +7847,7 @@ create_aead_operation_SGL(enum rte_crypto_aead_operation op,
>  	if (op == RTE_CRYPTO_AEAD_OP_DECRYPT) {
>  		rte_memcpy(sym_op->aead.digest.data, tdata->auth_tag.data,
>  				auth_tag_len);
> -		TEST_HEXDUMP(stdout, "digest:",
> +		debug_hexdump(stdout, "digest:",
>  				sym_op->aead.digest.data,
>  				auth_tag_len);
>  	}
> @@ -7871,8 +7873,8 @@ create_aead_operation_SGL(enum rte_crypto_aead_operation op,
>  		/* Copy AAD 18 bytes after the AAD pointer, according to the API */
>  		rte_memcpy(sym_op->aead.aad.data, tdata->aad.data, aad_len);
>  
> -		TEST_HEXDUMP(stdout, "iv:", iv_ptr, iv_len);
> -		TEST_HEXDUMP(stdout, "aad:",
> +		debug_hexdump(stdout, "iv:", iv_ptr, iv_len);
> +		debug_hexdump(stdout, "aad:",
>  				sym_op->aead.aad.data, aad_len);
>  	} else {
>  		uint8_t *iv_ptr = rte_crypto_op_ctod_offset(ut_params->op,
> @@ -7890,8 +7892,8 @@ create_aead_operation_SGL(enum rte_crypto_aead_operation op,
>  		memset(sym_op->aead.aad.data, 0, aad_len);
>  		rte_memcpy(sym_op->aead.aad.data, tdata->aad.data, aad_len);
>  
> -		TEST_HEXDUMP(stdout, "iv:", iv_ptr, iv_len);
> -		TEST_HEXDUMP(stdout, "aad:",
> +		debug_hexdump(stdout, "iv:", iv_ptr, iv_len);
> +		debug_hexdump(stdout, "aad:",
>  				sym_op->aead.aad.data, aad_len);
>  	}
>  
> diff --git a/test/test/test_cryptodev_blockcipher.c b/test/test/test_cryptodev_blockcipher.c
> index 20f3296d2..faed76ef0 100644
> --- a/test/test/test_cryptodev_blockcipher.c
> +++ b/test/test/test_cryptodev_blockcipher.c
> @@ -362,12 +362,12 @@ test_blockcipher_one_case(const struct blockcipher_test_case *t,
>  		rte_crypto_op_attach_sym_session(op, sess);
>  	}
>  
> -	TEST_HEXDUMP(stdout, "m_src(before):",
> +	debug_hexdump(stdout, "m_src(before):",
>  			sym_op->m_src->buf_addr, sym_op->m_src->buf_len);
>  	rte_memcpy(tmp_src_buf, sym_op->m_src->buf_addr,
>  						sym_op->m_src->buf_len);
>  	if (t->feature_mask & BLOCKCIPHER_TEST_FEATURE_OOP) {
> -		TEST_HEXDUMP(stdout, "m_dst(before):",
> +		debug_hexdump(stdout, "m_dst(before):",
>  			sym_op->m_dst->buf_addr, sym_op->m_dst->buf_len);
>  		rte_memcpy(tmp_dst_buf, sym_op->m_dst->buf_addr,
>  						sym_op->m_dst->buf_len);
> @@ -395,10 +395,10 @@ test_blockcipher_one_case(const struct blockcipher_test_case *t,
>  		goto error_exit;
>  	}
>  
> -	TEST_HEXDUMP(stdout, "m_src(after):",
> +	debug_hexdump(stdout, "m_src(after):",
>  			sym_op->m_src->buf_addr, sym_op->m_src->buf_len);
>  	if (t->feature_mask & BLOCKCIPHER_TEST_FEATURE_OOP)
> -		TEST_HEXDUMP(stdout, "m_dst(after):",
> +		debug_hexdump(stdout, "m_dst(after):",
>  			sym_op->m_dst->buf_addr, sym_op->m_dst->buf_len);
>  
>  	/* Verify results */
> -- 
> 2.11.0
> 

  parent reply	other threads:[~2018-01-16 15:54 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-08 13:21 Olivier Matz
2017-12-08 13:21 ` [dpdk-dev] [PATCH 2/2] test: add a testcase for dynamic logs Olivier Matz
2018-01-16 15:54 ` Olivier Matz [this message]
2018-01-17 11:09 ` [dpdk-dev] [PATCH 1/2] test: rely on dynamic log level to display hexdumps Thomas Monjalon

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=20180116155446.r4gsk65garskwhjg@platinum \
    --to=olivier.matz@6wind.com \
    --cc=declan.doherty@intel.com \
    --cc=dev@dpdk.org \
    --cc=pbhagavatula@caviumnetworks.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).