patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform
@ 2021-01-08  8:40 Ankur Dwivedi
  2021-01-12  4:55 ` [dpdk-stable] [dpdk-dev] " Jerin Jacob
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Ankur Dwivedi @ 2021-01-08  8:40 UTC (permalink / raw)
  To: dev; +Cc: stable, abhinandan.gujjar, akhil.goyal, anoobj, Ankur Dwivedi

The cipher operation in the transform should be set to encrypt.

Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")

Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
---
 app/test/test_event_crypto_adapter.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/test/test_event_crypto_adapter.c b/app/test/test_event_crypto_adapter.c
index a0169aa6cf..335211cd8c 100644
--- a/app/test/test_event_crypto_adapter.c
+++ b/app/test/test_event_crypto_adapter.c
@@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
 	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
 	cipher_xform.next = NULL;
 	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
+	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
 
 	op = rte_crypto_op_alloc(params.op_mpool,
 			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
@@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
 	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
 	cipher_xform.next = NULL;
 	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
+	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
 
 	op = rte_crypto_op_alloc(params.op_mpool,
 			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
-- 
2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH] test/event_crypto_adapter: set cipher operation in transform
  2021-01-08  8:40 [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform Ankur Dwivedi
@ 2021-01-12  4:55 ` Jerin Jacob
  2021-01-18  9:11 ` [dpdk-stable] " Gujjar, Abhinandan S
  2021-01-18 16:19 ` [dpdk-stable] [PATCH v2] " Ankur Dwivedi
  2 siblings, 0 replies; 9+ messages in thread
From: Jerin Jacob @ 2021-01-12  4:55 UTC (permalink / raw)
  To: Ankur Dwivedi
  Cc: dpdk-dev, dpdk stable, Gujjar, Abhinandan S, Akhil Goyal, Anoob Joseph

On Fri, Jan 8, 2021 at 2:11 PM Ankur Dwivedi <adwivedi@marvell.com> wrote:
>
> The cipher operation in the transform should be set to encrypt.
>
> Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")
>
> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>

@Gujjar, Abhinandan S

Please review this.


> ---
>  app/test/test_event_crypto_adapter.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/app/test/test_event_crypto_adapter.c b/app/test/test_event_crypto_adapter.c
> index a0169aa6cf..335211cd8c 100644
> --- a/app/test/test_event_crypto_adapter.c
> +++ b/app/test/test_event_crypto_adapter.c
> @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
>         cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>         cipher_xform.next = NULL;
>         cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> +       cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
>
>         op = rte_crypto_op_alloc(params.op_mpool,
>                         RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
>         cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>         cipher_xform.next = NULL;
>         cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> +       cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
>
>         op = rte_crypto_op_alloc(params.op_mpool,
>                         RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> --
> 2.28.0
>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform
  2021-01-08  8:40 [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform Ankur Dwivedi
  2021-01-12  4:55 ` [dpdk-stable] [dpdk-dev] " Jerin Jacob
@ 2021-01-18  9:11 ` Gujjar, Abhinandan S
  2021-01-18 11:33   ` Ankur Dwivedi
  2021-01-18 16:19 ` [dpdk-stable] [PATCH v2] " Ankur Dwivedi
  2 siblings, 1 reply; 9+ messages in thread
From: Gujjar, Abhinandan S @ 2021-01-18  9:11 UTC (permalink / raw)
  To: Ankur Dwivedi, dev; +Cc: stable, akhil.goyal, anoobj

Hi Ankur,

> -----Original Message-----
> From: Ankur Dwivedi <adwivedi@marvell.com>
> Sent: Friday, January 8, 2021 2:11 PM
> To: dev@dpdk.org
> Cc: stable@dpdk.org; Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>;
> akhil.goyal@nxp.com; anoobj@marvell.com; Ankur Dwivedi
> <adwivedi@marvell.com>
> Subject: [PATCH] test/event_crypto_adapter: set cipher operation in transform
> 
> The cipher operation in the transform should be set to encrypt.
> 
> Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")
> 
> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
> ---
>  app/test/test_event_crypto_adapter.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/app/test/test_event_crypto_adapter.c
> b/app/test/test_event_crypto_adapter.c
> index a0169aa6cf..335211cd8c 100644
> --- a/app/test/test_event_crypto_adapter.c
> +++ b/app/test/test_event_crypto_adapter.c
> @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>  	cipher_xform.next = NULL;
>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
NULL PMD does not refer " RTE_CRYPTO_CIPHER_OP_ENCRYPT ". Is your test code failing without this change?
> 
>  	op = rte_crypto_op_alloc(params.op_mpool,
>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>  	cipher_xform.next = NULL;
>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> 
>  	op = rte_crypto_op_alloc(params.op_mpool,
>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> --
> 2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform
  2021-01-18  9:11 ` [dpdk-stable] " Gujjar, Abhinandan S
@ 2021-01-18 11:33   ` Ankur Dwivedi
  2021-01-18 12:10     ` Gujjar, Abhinandan S
  0 siblings, 1 reply; 9+ messages in thread
From: Ankur Dwivedi @ 2021-01-18 11:33 UTC (permalink / raw)
  To: Gujjar, Abhinandan S, dev; +Cc: stable, akhil.goyal, Anoob Joseph

Hi Abhinandan,

>-----Original Message-----
>From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>
>Sent: Monday, January 18, 2021 2:42 PM
>To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org
>Cc: stable@dpdk.org; akhil.goyal@nxp.com; Anoob Joseph
><anoobj@marvell.com>
>Subject: [EXT] RE: [PATCH] test/event_crypto_adapter: set cipher operation in
>transform
>
>External Email
>
>----------------------------------------------------------------------
>Hi Ankur,
>
>> -----Original Message-----
>> From: Ankur Dwivedi <adwivedi@marvell.com>
>> Sent: Friday, January 8, 2021 2:11 PM
>> To: dev@dpdk.org
>> Cc: stable@dpdk.org; Gujjar, Abhinandan S
>> <abhinandan.gujjar@intel.com>; akhil.goyal@nxp.com;
>> anoobj@marvell.com; Ankur Dwivedi <adwivedi@marvell.com>
>> Subject: [PATCH] test/event_crypto_adapter: set cipher operation in
>> transform
>>
>> The cipher operation in the transform should be set to encrypt.
>>
>> Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")
>>
>> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
>> ---
>>  app/test/test_event_crypto_adapter.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/app/test/test_event_crypto_adapter.c
>> b/app/test/test_event_crypto_adapter.c
>> index a0169aa6cf..335211cd8c 100644
>> --- a/app/test/test_event_crypto_adapter.c
>> +++ b/app/test/test_event_crypto_adapter.c
>> @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
>>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>>  	cipher_xform.next = NULL;
>>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
>> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
>NULL PMD does not refer " RTE_CRYPTO_CIPHER_OP_ENCRYPT ". Is your test
>code failing without this change?
[Ankur] Yes the test is failing, because in octeontx2 crypto pmd, during crypto session configure there is a check for the cipher xform op. If the op does not match either encrypt or decrypt, the session configure fails.
>>
>>  	op = rte_crypto_op_alloc(params.op_mpool,
>>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
>> @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
>>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>>  	cipher_xform.next = NULL;
>>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
>> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
>>
>>  	op = rte_crypto_op_alloc(params.op_mpool,
>>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
>> --
>> 2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform
  2021-01-18 11:33   ` Ankur Dwivedi
@ 2021-01-18 12:10     ` Gujjar, Abhinandan S
  2021-01-18 14:54       ` Ankur Dwivedi
  0 siblings, 1 reply; 9+ messages in thread
From: Gujjar, Abhinandan S @ 2021-01-18 12:10 UTC (permalink / raw)
  To: Ankur Dwivedi, dev; +Cc: stable, akhil.goyal, Anoob Joseph

Hi Ankur,

> -----Original Message-----
> From: Ankur Dwivedi <adwivedi@marvell.com>
> Sent: Monday, January 18, 2021 5:04 PM
> To: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; dev@dpdk.org
> Cc: stable@dpdk.org; akhil.goyal@nxp.com; Anoob Joseph
> <anoobj@marvell.com>
> Subject: RE: [PATCH] test/event_crypto_adapter: set cipher operation in
> transform
> 
> Hi Abhinandan,
> 
> >-----Original Message-----
> >From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>
> >Sent: Monday, January 18, 2021 2:42 PM
> >To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org
> >Cc: stable@dpdk.org; akhil.goyal@nxp.com; Anoob Joseph
> ><anoobj@marvell.com>
> >Subject: [EXT] RE: [PATCH] test/event_crypto_adapter: set cipher
> >operation in transform
> >
> >External Email
> >
> >----------------------------------------------------------------------
> >Hi Ankur,
> >
> >> -----Original Message-----
> >> From: Ankur Dwivedi <adwivedi@marvell.com>
> >> Sent: Friday, January 8, 2021 2:11 PM
> >> To: dev@dpdk.org
> >> Cc: stable@dpdk.org; Gujjar, Abhinandan S
> >> <abhinandan.gujjar@intel.com>; akhil.goyal@nxp.com;
> >> anoobj@marvell.com; Ankur Dwivedi <adwivedi@marvell.com>
> >> Subject: [PATCH] test/event_crypto_adapter: set cipher operation in
> >> transform
> >>
> >> The cipher operation in the transform should be set to encrypt.
> >>
> >> Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")
> >>
> >> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
> >> ---
> >>  app/test/test_event_crypto_adapter.c | 2 ++
> >>  1 file changed, 2 insertions(+)
> >>
> >> diff --git a/app/test/test_event_crypto_adapter.c
> >> b/app/test/test_event_crypto_adapter.c
> >> index a0169aa6cf..335211cd8c 100644
> >> --- a/app/test/test_event_crypto_adapter.c
> >> +++ b/app/test/test_event_crypto_adapter.c
> >> @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
> >>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
> >>  	cipher_xform.next = NULL;
> >>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> >> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> >NULL PMD does not refer " RTE_CRYPTO_CIPHER_OP_ENCRYPT ". Is your test
> >code failing without this change?
> [Ankur] Yes the test is failing, because in octeontx2 crypto pmd, during crypto
> session configure there is a check for the cipher xform op. If the op does not
> match either encrypt or decrypt, the session configure fails.
[Abhinandan] Ok. I have not come across such issue on my system.
Could you please add more details into the commit message of the patch and send?
> >>
> >>  	op = rte_crypto_op_alloc(params.op_mpool,
> >>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> >> @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
> >>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
> >>  	cipher_xform.next = NULL;
> >>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> >> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> >>
> >>  	op = rte_crypto_op_alloc(params.op_mpool,
> >>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> >> --
> >> 2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform
  2021-01-18 12:10     ` Gujjar, Abhinandan S
@ 2021-01-18 14:54       ` Ankur Dwivedi
  0 siblings, 0 replies; 9+ messages in thread
From: Ankur Dwivedi @ 2021-01-18 14:54 UTC (permalink / raw)
  To: Gujjar, Abhinandan S, dev; +Cc: stable, akhil.goyal, Anoob Joseph

Hi Abhinandan,

>-----Original Message-----
>From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>
>Sent: Monday, January 18, 2021 5:41 PM
>To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org
>Cc: stable@dpdk.org; akhil.goyal@nxp.com; Anoob Joseph
><anoobj@marvell.com>
>Subject: [EXT] RE: [PATCH] test/event_crypto_adapter: set cipher operation in
>transform
>
>External Email
>
>----------------------------------------------------------------------
>Hi Ankur,
>
>> -----Original Message-----
>> From: Ankur Dwivedi <adwivedi@marvell.com>
>> Sent: Monday, January 18, 2021 5:04 PM
>> To: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; dev@dpdk.org
>> Cc: stable@dpdk.org; akhil.goyal@nxp.com; Anoob Joseph
>> <anoobj@marvell.com>
>> Subject: RE: [PATCH] test/event_crypto_adapter: set cipher operation
>> in transform
>>
>> Hi Abhinandan,
>>
>> >-----Original Message-----
>> >From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>
>> >Sent: Monday, January 18, 2021 2:42 PM
>> >To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org
>> >Cc: stable@dpdk.org; akhil.goyal@nxp.com; Anoob Joseph
>> ><anoobj@marvell.com>
>> >Subject: [EXT] RE: [PATCH] test/event_crypto_adapter: set cipher
>> >operation in transform
>> >
>> >External Email
>> >
>> >---------------------------------------------------------------------
>> >-
>> >Hi Ankur,
>> >
>> >> -----Original Message-----
>> >> From: Ankur Dwivedi <adwivedi@marvell.com>
>> >> Sent: Friday, January 8, 2021 2:11 PM
>> >> To: dev@dpdk.org
>> >> Cc: stable@dpdk.org; Gujjar, Abhinandan S
>> >> <abhinandan.gujjar@intel.com>; akhil.goyal@nxp.com;
>> >> anoobj@marvell.com; Ankur Dwivedi <adwivedi@marvell.com>
>> >> Subject: [PATCH] test/event_crypto_adapter: set cipher operation in
>> >> transform
>> >>
>> >> The cipher operation in the transform should be set to encrypt.
>> >>
>> >> Fixes: 74449375237f ("test/event_crypto_adapter: fix
>> >> configuration")
>> >>
>> >> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
>> >> ---
>> >>  app/test/test_event_crypto_adapter.c | 2 ++
>> >>  1 file changed, 2 insertions(+)
>> >>
>> >> diff --git a/app/test/test_event_crypto_adapter.c
>> >> b/app/test/test_event_crypto_adapter.c
>> >> index a0169aa6cf..335211cd8c 100644
>> >> --- a/app/test/test_event_crypto_adapter.c
>> >> +++ b/app/test/test_event_crypto_adapter.c
>> >> @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
>> >>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>> >>  	cipher_xform.next = NULL;
>> >>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
>> >> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
>> >NULL PMD does not refer " RTE_CRYPTO_CIPHER_OP_ENCRYPT ". Is your
>> >test code failing without this change?
>> [Ankur] Yes the test is failing, because in octeontx2 crypto pmd,
>> during crypto session configure there is a check for the cipher xform
>> op. If the op does not match either encrypt or decrypt, the session configure
>fails.
>[Abhinandan] Ok. I have not come across such issue on my system.
>Could you please add more details into the commit message of the patch and
>send?
[Ankur] Sure. I will update the commit message in v2.
>> >>
>> >>  	op = rte_crypto_op_alloc(params.op_mpool,
>> >>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
>> >> @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
>> >>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>> >>  	cipher_xform.next = NULL;
>> >>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
>> >> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
>> >>
>> >>  	op = rte_crypto_op_alloc(params.op_mpool,
>> >>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
>> >> --
>> >> 2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* [dpdk-stable] [PATCH v2] test/event_crypto_adapter: set cipher operation in transform
  2021-01-08  8:40 [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform Ankur Dwivedi
  2021-01-12  4:55 ` [dpdk-stable] [dpdk-dev] " Jerin Jacob
  2021-01-18  9:11 ` [dpdk-stable] " Gujjar, Abhinandan S
@ 2021-01-18 16:19 ` Ankur Dwivedi
  2021-01-19  7:13   ` Gujjar, Abhinandan S
  2 siblings, 1 reply; 9+ messages in thread
From: Ankur Dwivedi @ 2021-01-18 16:19 UTC (permalink / raw)
  To: dev; +Cc: stable, abhinandan.gujjar, akhil.goyal, anoobj, jerinj, Ankur Dwivedi

The symmetric session configure callback function in OCTEON TX2 crypto
PMD returns error if the cipher operation is not set to either encrypt
or decrypt. This patch sets the cipher operation for the null cipher
to encrypt.

Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")

Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
---
v2:
* Updated the commit message with details.

 app/test/test_event_crypto_adapter.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/test/test_event_crypto_adapter.c b/app/test/test_event_crypto_adapter.c
index a0169aa6cf..335211cd8c 100644
--- a/app/test/test_event_crypto_adapter.c
+++ b/app/test/test_event_crypto_adapter.c
@@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
 	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
 	cipher_xform.next = NULL;
 	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
+	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
 
 	op = rte_crypto_op_alloc(params.op_mpool,
 			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
@@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
 	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
 	cipher_xform.next = NULL;
 	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
+	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
 
 	op = rte_crypto_op_alloc(params.op_mpool,
 			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
-- 
2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [PATCH v2] test/event_crypto_adapter: set cipher operation in transform
  2021-01-18 16:19 ` [dpdk-stable] [PATCH v2] " Ankur Dwivedi
@ 2021-01-19  7:13   ` Gujjar, Abhinandan S
  2021-01-26 19:32     ` [dpdk-stable] [dpdk-dev] " Jerin Jacob
  0 siblings, 1 reply; 9+ messages in thread
From: Gujjar, Abhinandan S @ 2021-01-19  7:13 UTC (permalink / raw)
  To: Ankur Dwivedi, dev; +Cc: stable, akhil.goyal, anoobj, jerinj

Acked-by: abhinandan.gujjar@intel.com

Thanks & Regards
Abhinandan

> -----Original Message-----
> From: Ankur Dwivedi <adwivedi@marvell.com>
> Sent: Monday, January 18, 2021 9:50 PM
> To: dev@dpdk.org
> Cc: stable@dpdk.org; Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>;
> akhil.goyal@nxp.com; anoobj@marvell.com; jerinj@marvell.com; Ankur
> Dwivedi <adwivedi@marvell.com>
> Subject: [PATCH v2] test/event_crypto_adapter: set cipher operation in
> transform
> 
> The symmetric session configure callback function in OCTEON TX2 crypto PMD
> returns error if the cipher operation is not set to either encrypt or decrypt. This
> patch sets the cipher operation for the null cipher to encrypt.
> 
> Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")
> 
> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
> ---
> v2:
> * Updated the commit message with details.
> 
>  app/test/test_event_crypto_adapter.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/app/test/test_event_crypto_adapter.c
> b/app/test/test_event_crypto_adapter.c
> index a0169aa6cf..335211cd8c 100644
> --- a/app/test/test_event_crypto_adapter.c
> +++ b/app/test/test_event_crypto_adapter.c
> @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>  	cipher_xform.next = NULL;
>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> 
>  	op = rte_crypto_op_alloc(params.op_mpool,
>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
>  	cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
>  	cipher_xform.next = NULL;
>  	cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> +	cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> 
>  	op = rte_crypto_op_alloc(params.op_mpool,
>  			RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> --
> 2.28.0


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH v2] test/event_crypto_adapter: set cipher operation in transform
  2021-01-19  7:13   ` Gujjar, Abhinandan S
@ 2021-01-26 19:32     ` Jerin Jacob
  0 siblings, 0 replies; 9+ messages in thread
From: Jerin Jacob @ 2021-01-26 19:32 UTC (permalink / raw)
  To: Gujjar, Abhinandan S
  Cc: Ankur Dwivedi, dev, stable, akhil.goyal, anoobj, jerinj

On Tue, Jan 19, 2021 at 12:43 PM Gujjar, Abhinandan S
<abhinandan.gujjar@intel.com> wrote:
>
> Acked-by: abhinandan.gujjar@intel.com


Applied to dpdk-next-eventdev/for-main. Thanks.



>
> Thanks & Regards
> Abhinandan
>
> > -----Original Message-----
> > From: Ankur Dwivedi <adwivedi@marvell.com>
> > Sent: Monday, January 18, 2021 9:50 PM
> > To: dev@dpdk.org
> > Cc: stable@dpdk.org; Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>;
> > akhil.goyal@nxp.com; anoobj@marvell.com; jerinj@marvell.com; Ankur
> > Dwivedi <adwivedi@marvell.com>
> > Subject: [PATCH v2] test/event_crypto_adapter: set cipher operation in
> > transform
> >
> > The symmetric session configure callback function in OCTEON TX2 crypto PMD
> > returns error if the cipher operation is not set to either encrypt or decrypt. This
> > patch sets the cipher operation for the null cipher to encrypt.
> >
> > Fixes: 74449375237f ("test/event_crypto_adapter: fix configuration")
> >
> > Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
> > ---
> > v2:
> > * Updated the commit message with details.
> >
> >  app/test/test_event_crypto_adapter.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/app/test/test_event_crypto_adapter.c
> > b/app/test/test_event_crypto_adapter.c
> > index a0169aa6cf..335211cd8c 100644
> > --- a/app/test/test_event_crypto_adapter.c
> > +++ b/app/test/test_event_crypto_adapter.c
> > @@ -183,6 +183,7 @@ test_op_forward_mode(uint8_t session_less)
> >       cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
> >       cipher_xform.next = NULL;
> >       cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> > +     cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> >
> >       op = rte_crypto_op_alloc(params.op_mpool,
> >                       RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> > @@ -382,6 +383,7 @@ test_op_new_mode(uint8_t session_less)
> >       cipher_xform.type = RTE_CRYPTO_SYM_XFORM_CIPHER;
> >       cipher_xform.next = NULL;
> >       cipher_xform.cipher.algo = RTE_CRYPTO_CIPHER_NULL;
> > +     cipher_xform.cipher.op = RTE_CRYPTO_CIPHER_OP_ENCRYPT;
> >
> >       op = rte_crypto_op_alloc(params.op_mpool,
> >                       RTE_CRYPTO_OP_TYPE_SYMMETRIC);
> > --
> > 2.28.0
>

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2021-01-26 19:32 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-08  8:40 [dpdk-stable] [PATCH] test/event_crypto_adapter: set cipher operation in transform Ankur Dwivedi
2021-01-12  4:55 ` [dpdk-stable] [dpdk-dev] " Jerin Jacob
2021-01-18  9:11 ` [dpdk-stable] " Gujjar, Abhinandan S
2021-01-18 11:33   ` Ankur Dwivedi
2021-01-18 12:10     ` Gujjar, Abhinandan S
2021-01-18 14:54       ` Ankur Dwivedi
2021-01-18 16:19 ` [dpdk-stable] [PATCH v2] " Ankur Dwivedi
2021-01-19  7:13   ` Gujjar, Abhinandan S
2021-01-26 19:32     ` [dpdk-stable] [dpdk-dev] " Jerin Jacob

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).