* [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create
@ 2018-10-02 19:05 Junxiao Shi
2018-10-09 10:50 ` Akhil Goyal
0 siblings, 1 reply; 6+ messages in thread
From: Junxiao Shi @ 2018-10-02 19:05 UTC (permalink / raw)
To: dev
Signed-off-by: Junxiao Shi <git@mail1.yoursunny.com>
---
lib/librte_cryptodev/rte_cryptodev.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c
index 63ae23f..3d6f474 100644
--- a/lib/librte_cryptodev/rte_cryptodev.c
+++ b/lib/librte_cryptodev/rte_cryptodev.c
@@ -1477,6 +1477,8 @@ rte_crypto_op_pool_create(const char *name, enum rte_crypto_op_type type,
elt_size += sizeof(struct rte_crypto_sym_op);
} else if (type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC) {
elt_size += sizeof(struct rte_crypto_asym_op);
+ } else if (type == RTE_CRYPTO_OP_TYPE_UNDEFINED) {
+ elt_size += RTE_MAX(sizeof(struct rte_crypto_sym_op), sizeof(struct rte_crypto_asym_op));
} else {
CDEV_LOG_ERR("Invalid op_type\n");
return NULL;
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create
2018-10-02 19:05 [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create Junxiao Shi
@ 2018-10-09 10:50 ` Akhil Goyal
2018-10-09 10:58 ` yoursunny
0 siblings, 1 reply; 6+ messages in thread
From: Akhil Goyal @ 2018-10-09 10:50 UTC (permalink / raw)
To: Junxiao Shi, dev
Hi Junxiao,
On 10/3/2018 12:35 AM, Junxiao Shi wrote:
> Signed-off-by: Junxiao Shi <git@mail1.yoursunny.com>
> ---
> lib/librte_cryptodev/rte_cryptodev.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c
> index 63ae23f..3d6f474 100644
> --- a/lib/librte_cryptodev/rte_cryptodev.c
> +++ b/lib/librte_cryptodev/rte_cryptodev.c
> @@ -1477,6 +1477,8 @@ rte_crypto_op_pool_create(const char *name, enum rte_crypto_op_type type,
> elt_size += sizeof(struct rte_crypto_sym_op);
> } else if (type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC) {
> elt_size += sizeof(struct rte_crypto_asym_op);
> + } else if (type == RTE_CRYPTO_OP_TYPE_UNDEFINED) {
> + elt_size += RTE_MAX(sizeof(struct rte_crypto_sym_op), sizeof(struct rte_crypto_asym_op));
> } else {
> CDEV_LOG_ERR("Invalid op_type\n");
> return NULL;
Could you please explain the need for this change. If there is some type
which is missing, we can add that.
defining the size of undefined op type does not make sense.
-Akhil
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create
2018-10-09 10:50 ` Akhil Goyal
@ 2018-10-09 10:58 ` yoursunny
2018-10-09 11:11 ` Akhil Goyal
0 siblings, 1 reply; 6+ messages in thread
From: yoursunny @ 2018-10-09 10:58 UTC (permalink / raw)
To: Akhil Goyal; +Cc: Junxiao Shi, dev
Hi Akhil
The documentation of rte_crypto_op_pool_create indicates that _UNDEFINED
creates a pool that supports both symmetric and assymetric crypto. This
change makes the code consistent with documentation.
Yours, Junxiao
On Tue, Oct 9, 2018 at 06:51 Akhil Goyal <akhil.goyal@nxp.com> wrote:
> Hi Junxiao,
>
> On 10/3/2018 12:35 AM, Junxiao Shi wrote:
> > Signed-off-by: Junxiao Shi <git@mail1.yoursunny.com>
> > ---
> > lib/librte_cryptodev/rte_cryptodev.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/lib/librte_cryptodev/rte_cryptodev.c
> b/lib/librte_cryptodev/rte_cryptodev.c
> > index 63ae23f..3d6f474 100644
> > --- a/lib/librte_cryptodev/rte_cryptodev.c
> > +++ b/lib/librte_cryptodev/rte_cryptodev.c
> > @@ -1477,6 +1477,8 @@ rte_crypto_op_pool_create(const char *name, enum
> rte_crypto_op_type type,
> > elt_size += sizeof(struct rte_crypto_sym_op);
> > } else if (type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC) {
> > elt_size += sizeof(struct rte_crypto_asym_op);
> > + } else if (type == RTE_CRYPTO_OP_TYPE_UNDEFINED) {
> > + elt_size += RTE_MAX(sizeof(struct rte_crypto_sym_op),
> sizeof(struct rte_crypto_asym_op));
> > } else {
> > CDEV_LOG_ERR("Invalid op_type\n");
> > return NULL;
> Could you please explain the need for this change. If there is some type
> which is missing, we can add that.
> defining the size of undefined op type does not make sense.
>
> -Akhil
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create
2018-10-09 10:58 ` yoursunny
@ 2018-10-09 11:11 ` Akhil Goyal
2018-10-09 12:51 ` Akhil Goyal
0 siblings, 1 reply; 6+ messages in thread
From: Akhil Goyal @ 2018-10-09 11:11 UTC (permalink / raw)
To: yoursunny; +Cc: Junxiao Shi, dev
On 10/9/2018 4:28 PM, yoursunny wrote:
> Hi Akhil
>
> The documentation of rte_crypto_op_pool_create indicates that _UNDEFINED
> creates a pool that supports both symmetric and assymetric crypto. This
> change makes the code consistent with documentation.
>
> Yours, Junxiao
Please add this information in the patch description and this should be
marked as a fix.
Please also add
Fixes: c0f87eb5252b ("cryptodev: change burst API to be crypto op oriented")
and cc stable@dpdk.org
> On Tue, Oct 9, 2018 at 06:51 Akhil Goyal <akhil.goyal@nxp.com> wrote:
>
>> Hi Junxiao,
>>
>> On 10/3/2018 12:35 AM, Junxiao Shi wrote:
>>> Signed-off-by: Junxiao Shi <git@mail1.yoursunny.com>
>>> ---
>>> lib/librte_cryptodev/rte_cryptodev.c | 2 ++
>>> 1 file changed, 2 insertions(+)
>>>
>>> diff --git a/lib/librte_cryptodev/rte_cryptodev.c
>> b/lib/librte_cryptodev/rte_cryptodev.c
>>> index 63ae23f..3d6f474 100644
>>> --- a/lib/librte_cryptodev/rte_cryptodev.c
>>> +++ b/lib/librte_cryptodev/rte_cryptodev.c
>>> @@ -1477,6 +1477,8 @@ rte_crypto_op_pool_create(const char *name, enum
>> rte_crypto_op_type type,
>>> elt_size += sizeof(struct rte_crypto_sym_op);
>>> } else if (type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC) {
>>> elt_size += sizeof(struct rte_crypto_asym_op);
>>> + } else if (type == RTE_CRYPTO_OP_TYPE_UNDEFINED) {
>>> + elt_size += RTE_MAX(sizeof(struct rte_crypto_sym_op),
>> sizeof(struct rte_crypto_asym_op));
>>> } else {
>>> CDEV_LOG_ERR("Invalid op_type\n");
>>> return NULL;
>> Could you please explain the need for this change. If there is some type
>> which is missing, we can add that.
>> defining the size of undefined op type does not make sense.
>>
>> -Akhil
>>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create
2018-10-09 11:11 ` Akhil Goyal
@ 2018-10-09 12:51 ` Akhil Goyal
0 siblings, 0 replies; 6+ messages in thread
From: Akhil Goyal @ 2018-10-09 12:51 UTC (permalink / raw)
To: yoursunny; +Cc: Junxiao Shi, dev
Also change title to
crypto: fix element size for undefined crypto op
On 10/9/2018 4:41 PM, Akhil Goyal wrote:
>
>
> On 10/9/2018 4:28 PM, yoursunny wrote:
>> Hi Akhil
>>
>> The documentation of rte_crypto_op_pool_create indicates that _UNDEFINED
>> creates a pool that supports both symmetric and assymetric crypto. This
>> change makes the code consistent with documentation.
>>
>> Yours, Junxiao
> Please add this information in the patch description and this should
> be marked as a fix.
>
> Please also add
> Fixes: c0f87eb5252b ("cryptodev: change burst API to be crypto op
> oriented")
>
> and cc stable@dpdk.org
>> On Tue, Oct 9, 2018 at 06:51 Akhil Goyal <akhil.goyal@nxp.com> wrote:
>>
>>> Hi Junxiao,
>>>
>>> On 10/3/2018 12:35 AM, Junxiao Shi wrote:
>>>> Signed-off-by: Junxiao Shi <git@mail1.yoursunny.com>
>>>> ---
>>>> lib/librte_cryptodev/rte_cryptodev.c | 2 ++
>>>> 1 file changed, 2 insertions(+)
>>>>
>>>> diff --git a/lib/librte_cryptodev/rte_cryptodev.c
>>> b/lib/librte_cryptodev/rte_cryptodev.c
>>>> index 63ae23f..3d6f474 100644
>>>> --- a/lib/librte_cryptodev/rte_cryptodev.c
>>>> +++ b/lib/librte_cryptodev/rte_cryptodev.c
>>>> @@ -1477,6 +1477,8 @@ rte_crypto_op_pool_create(const char *name, enum
>>> rte_crypto_op_type type,
>>>> elt_size += sizeof(struct rte_crypto_sym_op);
>>>> } else if (type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC) {
>>>> elt_size += sizeof(struct rte_crypto_asym_op);
>>>> + } else if (type == RTE_CRYPTO_OP_TYPE_UNDEFINED) {
>>>> + elt_size += RTE_MAX(sizeof(struct rte_crypto_sym_op),
>>> sizeof(struct rte_crypto_asym_op));
one more nit..
The line is beyond 80 characters.
Before sending the patch you should run checkpatch script.
>>>> } else {
>>>> CDEV_LOG_ERR("Invalid op_type\n");
>>>> return NULL;
>>> Could you please explain the need for this change. If there is some
>>> type
>>> which is missing, we can add that.
>>> defining the size of undefined op type does not make sense.
>>>
>>> -Akhil
>>>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create
@ 2018-10-02 19:05 Junxiao Shi
0 siblings, 0 replies; 6+ messages in thread
From: Junxiao Shi @ 2018-10-02 19:05 UTC (permalink / raw)
To: dev
Signed-off-by: Junxiao Shi <git@mail1.yoursunny.com>
---
lib/librte_cryptodev/rte_cryptodev.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/lib/librte_cryptodev/rte_cryptodev.c b/lib/librte_cryptodev/rte_cryptodev.c
index 63ae23f..3d6f474 100644
--- a/lib/librte_cryptodev/rte_cryptodev.c
+++ b/lib/librte_cryptodev/rte_cryptodev.c
@@ -1477,6 +1477,8 @@ rte_crypto_op_pool_create(const char *name, enum rte_crypto_op_type type,
elt_size += sizeof(struct rte_crypto_sym_op);
} else if (type == RTE_CRYPTO_OP_TYPE_ASYMMETRIC) {
elt_size += sizeof(struct rte_crypto_asym_op);
+ } else if (type == RTE_CRYPTO_OP_TYPE_UNDEFINED) {
+ elt_size += RTE_MAX(sizeof(struct rte_crypto_sym_op), sizeof(struct rte_crypto_asym_op));
} else {
CDEV_LOG_ERR("Invalid op_type\n");
return NULL;
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-10-09 12:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-02 19:05 [dpdk-dev] [PATCH] crypto: recognize OP_TYPE_UNDEFINED in rte_crypto_op_pool_create Junxiao Shi
2018-10-09 10:50 ` Akhil Goyal
2018-10-09 10:58 ` yoursunny
2018-10-09 11:11 ` Akhil Goyal
2018-10-09 12:51 ` Akhil Goyal
2018-10-02 19:05 Junxiao Shi
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).