For series:
Acked-by: abhinandan.guj...@intel.com

I noticed that these patches have issue with "Intel-compilation".
Hope, it will be taken care before merging. 

> -----Original Message-----
> From: Hemant Agrawal <hemant.agra...@nxp.com>
> Sent: Thursday, November 7, 2019 2:31 PM
> To: dev@dpdk.org
> Cc: jerinjac...@gmail.com; Gujjar, Abhinandan S
> <abhinandan.guj...@intel.com>
> Subject: [PATCH v2 1/3] test/event_crypto: fix missing IV value for AES algo
> 
> The IV was not set, which was causing HW based SEC on DPAA1 to fail.
> 
> Fixes: ce02103ad072 ("test/event_crypto: change the SEC cipher algo")
> 
> Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com>
> ---
> Sending only the fixes. dropped the functional changes
> 
>  app/test/test_event_crypto_adapter.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/app/test/test_event_crypto_adapter.c
> b/app/test/test_event_crypto_adapter.c
> index 652e458f0..bf83a6357 100644
> --- a/app/test/test_event_crypto_adapter.c
> +++ b/app/test/test_event_crypto_adapter.c
> @@ -171,6 +171,7 @@ test_op_forward_mode(uint8_t session_less)
>       struct rte_event ev;
>       uint32_t cap;
>       int ret;
> +     uint8_t cipher_key[17];
> 
>       memset(&m_data, 0, sizeof(m_data));
> 
> @@ -186,6 +187,11 @@ test_op_forward_mode(uint8_t session_less)
>       cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_AES_CBC;
>       cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> 
> +     cipher_xform.cipher.key.data = cipher_key;
> +     cipher_xform.cipher.key.length = 16;
> +     cipher_xform.cipher.iv.offset = IV_OFFSET;
> +     cipher_xform.cipher.iv.length = 16;
> +
>       op = rte_crypto_op_alloc(params.op_mpool,
>                       RTE_CRYPTO_OP_TYPE_SYMMETRIC);
>       TEST_ASSERT_NOT_NULL(op,
> @@ -364,6 +370,7 @@ test_op_new_mode(uint8_t session_less)
>       struct rte_mbuf *m;
>       uint32_t cap;
>       int ret;
> +     uint8_t cipher_key[17];
> 
>       memset(&m_data, 0, sizeof(m_data));
> 
> @@ -379,6 +386,11 @@ test_op_new_mode(uint8_t session_less)
>       cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_AES_CBC;
>       cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> 
> +     cipher_xform.cipher.key.data = cipher_key;
> +     cipher_xform.cipher.key.length = 16;
> +     cipher_xform.cipher.iv.offset = IV_OFFSET;
> +     cipher_xform.cipher.iv.length = 16;
> +
>       op = rte_crypto_op_alloc(params.op_mpool,
>                       RTE_CRYPTO_OP_TYPE_SYMMETRIC);
>       TEST_ASSERT_NOT_NULL(op, "Failed to allocate crypto_op!\n");
> --
> 2.17.1

Reply via email to