* [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions
@ 2017-10-03 9:29 Akhil Goyal
2017-10-05 8:19 ` De Lara Guarch, Pablo
2017-10-05 12:50 ` [dpdk-dev] [PATCH v2] " Akhil Goyal
0 siblings, 2 replies; 6+ messages in thread
From: Akhil Goyal @ 2017-10-03 9:29 UTC (permalink / raw)
To: dev
Cc: declan.doherty, pablo.de.lara.guarch, hemant.agrawal, stable,
Akhil Goyal
Fixes: 7a364faef185 ("cryptodev: remove crypto device type enumeration")
Cc: stable@dpdk.org
Signed-off-by: Akhil Goyal <akhil.goyal@nxp.com>
---
test/test/test_cryptodev.c | 2 +-
test/test/test_cryptodev_perf.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c
index a4116c6..132f99c 100644
--- a/test/test/test_cryptodev.c
+++ b/test/test/test_cryptodev.c
@@ -1816,7 +1816,7 @@ test_authonly_dpaa2_sec_all(void)
ts_params->session_mpool,
ts_params->valid_devs[0],
rte_cryptodev_driver_id_get(
- RTE_STR(RTE_CRYPTODEV_DPAA2_SEC_PMD)),
+ RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD)),
BLKCIPHER_AUTHONLY_TYPE);
TEST_ASSERT_EQUAL(status, 0, "Test failed");
diff --git a/test/test/test_cryptodev_perf.c b/test/test/test_cryptodev_perf.c
index 3b57e6d..5df2e6e 100644
--- a/test/test/test_cryptodev_perf.c
+++ b/test/test/test_cryptodev_perf.c
@@ -211,7 +211,7 @@ static const char *pmd_name(uint8_t driver_id)
{
uint8_t null_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_NULL_PMD));
- uint8_t dpaa2_pmd = rte_cryptodev_driver_id_get(
+ uint8_t dpaa2_sec_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD));
uint8_t snow3g_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD));
@@ -232,7 +232,7 @@ static const char *pmd_name(uint8_t driver_id)
return RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD);
else if (driver_id == snow3g_pmd)
return RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD);
- else if (driver_id == dpaa2_pmd)
+ else if (driver_id == dpaa2_sec_pmd)
return RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD);
else
return "";
--
2.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions
2017-10-03 9:29 [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions Akhil Goyal
@ 2017-10-05 8:19 ` De Lara Guarch, Pablo
2017-10-05 12:25 ` Akhil Goyal
2017-10-05 12:50 ` [dpdk-dev] [PATCH v2] " Akhil Goyal
1 sibling, 1 reply; 6+ messages in thread
From: De Lara Guarch, Pablo @ 2017-10-05 8:19 UTC (permalink / raw)
To: Akhil Goyal, dev; +Cc: Doherty, Declan, hemant.agrawal, stable
Hi Akhil,
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Akhil Goyal
> Sent: Tuesday, October 3, 2017 10:29 AM
> To: dev@dpdk.org
> Cc: Doherty, Declan <declan.doherty@intel.com>; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>; hemant.agrawal@nxp.com;
> stable@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>
> Subject: [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and
> definitions
>
> Fixes: 7a364faef185 ("cryptodev: remove crypto device type enumeration")
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Akhil Goyal <akhil.goyal@nxp.com>
> ---
> test/test/test_cryptodev.c | 2 +-
> test/test/test_cryptodev_perf.c | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c index
> a4116c6..132f99c 100644
> --- a/test/test/test_cryptodev.c
> +++ b/test/test/test_cryptodev.c
> @@ -1816,7 +1816,7 @@ test_authonly_dpaa2_sec_all(void)
> ts_params->session_mpool,
> ts_params->valid_devs[0],
> rte_cryptodev_driver_id_get(
> - RTE_STR(RTE_CRYPTODEV_DPAA2_SEC_PMD)),
> + RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD)),
> BLKCIPHER_AUTHONLY_TYPE);
>
> TEST_ASSERT_EQUAL(status, 0, "Test failed"); diff --git
> a/test/test/test_cryptodev_perf.c b/test/test/test_cryptodev_perf.c index
> 3b57e6d..5df2e6e 100644
> --- a/test/test/test_cryptodev_perf.c
> +++ b/test/test/test_cryptodev_perf.c
> @@ -211,7 +211,7 @@ static const char *pmd_name(uint8_t driver_id) {
> uint8_t null_pmd = rte_cryptodev_driver_id_get(
> RTE_STR(CRYPTODEV_NAME_NULL_PMD));
> - uint8_t dpaa2_pmd = rte_cryptodev_driver_id_get(
> + uint8_t dpaa2_sec_pmd = rte_cryptodev_driver_id_get(
I think, in order to keep consistency, you should make this change also in test_cryptodev.c:
test/test/test_cryptodev_blockcipher.c
580: int dpaa2_pmd = rte_cryptodev_driver_id_get(
646: else if (driver_id == dpaa2_pmd)
Also, I submitted a patch to remove test_cryptodev_perf.c.
I will apply this patch first, so it is easier to integrate in the stable version,
but could you ack my patch if you are OK with it?
The sooner we remove it, the better, as we are investing a lot of time
in the crypto-perf app, and there is no need to maintain a duplicate, in my opinion.
Thanks,
Pablo
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions
2017-10-05 8:19 ` De Lara Guarch, Pablo
@ 2017-10-05 12:25 ` Akhil Goyal
2017-10-05 12:59 ` De Lara Guarch, Pablo
0 siblings, 1 reply; 6+ messages in thread
From: Akhil Goyal @ 2017-10-05 12:25 UTC (permalink / raw)
To: De Lara Guarch, Pablo, dev; +Cc: Doherty, Declan, hemant.agrawal, stable
Hi Pablo,
On 10/5/2017 1:49 PM, De Lara Guarch, Pablo wrote:
> Hi Akhil,
>
>> -----Original Message-----
>> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Akhil Goyal
>> Sent: Tuesday, October 3, 2017 10:29 AM
>> To: dev@dpdk.org
>> Cc: Doherty, Declan <declan.doherty@intel.com>; De Lara Guarch, Pablo
>> <pablo.de.lara.guarch@intel.com>; hemant.agrawal@nxp.com;
>> stable@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>
>> Subject: [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and
>> definitions
>>
>> Fixes: 7a364faef185 ("cryptodev: remove crypto device type enumeration")
>>
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Akhil Goyal <akhil.goyal@nxp.com>
>> ---
>> test/test/test_cryptodev.c | 2 +-
>> test/test/test_cryptodev_perf.c | 4 ++--
>> 2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c index
>> a4116c6..132f99c 100644
>> --- a/test/test/test_cryptodev.c
>> +++ b/test/test/test_cryptodev.c
>> @@ -1816,7 +1816,7 @@ test_authonly_dpaa2_sec_all(void)
>> ts_params->session_mpool,
>> ts_params->valid_devs[0],
>> rte_cryptodev_driver_id_get(
>> - RTE_STR(RTE_CRYPTODEV_DPAA2_SEC_PMD)),
>> + RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD)),
>> BLKCIPHER_AUTHONLY_TYPE);
>>
>> TEST_ASSERT_EQUAL(status, 0, "Test failed"); diff --git
>> a/test/test/test_cryptodev_perf.c b/test/test/test_cryptodev_perf.c index
>> 3b57e6d..5df2e6e 100644
>> --- a/test/test/test_cryptodev_perf.c
>> +++ b/test/test/test_cryptodev_perf.c
>> @@ -211,7 +211,7 @@ static const char *pmd_name(uint8_t driver_id) {
>> uint8_t null_pmd = rte_cryptodev_driver_id_get(
>> RTE_STR(CRYPTODEV_NAME_NULL_PMD));
>> - uint8_t dpaa2_pmd = rte_cryptodev_driver_id_get(
>> + uint8_t dpaa2_sec_pmd = rte_cryptodev_driver_id_get(
>
> I think, in order to keep consistency, you should make this change also in test_cryptodev.c:
>
> test/test/test_cryptodev_blockcipher.c
> 580: int dpaa2_pmd = rte_cryptodev_driver_id_get(
> 646: else if (driver_id == dpaa2_pmd)
>
> Also, I submitted a patch to remove test_cryptodev_perf.c.
> I will apply this patch first, so it is easier to integrate in the stable version,
> but could you ack my patch if you are OK with it?
>
> The sooner we remove it, the better, as we are investing a lot of time
> in the crypto-perf app, and there is no need to maintain a duplicate, in my opinion.
>
thanks for pointing this out. I would send a v2.
I have a question regarding test_cryptodev_perf.c for dpaa_sec patches.
The patchset for dpaa_sec includes changes in test_cryptodev_perf.c.
Do you plan to add those patches on the tree as well before removing
this file or I need to send another version?
In test/test/test_cryptodev_blockcipher.c, I missed out this change for
dpaa_sec also. So, I was sending another version for dpaa_sec patches to
include this change. Should I remove the changes in the
test_cryptodev_perf.c from the dpaa_sec patchset also?
Thanks,
Akhil
^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-dev] [PATCH v2] test/crypto: fix dpaa2_sec macros and definitions
2017-10-03 9:29 [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions Akhil Goyal
2017-10-05 8:19 ` De Lara Guarch, Pablo
@ 2017-10-05 12:50 ` Akhil Goyal
2017-10-05 13:30 ` De Lara Guarch, Pablo
1 sibling, 1 reply; 6+ messages in thread
From: Akhil Goyal @ 2017-10-05 12:50 UTC (permalink / raw)
To: dev
Cc: declan.doherty, pablo.de.lara.guarch, hemant.agrawal, stable,
Akhil Goyal
Fixes: 7a364faef185 ("cryptodev: remove crypto device type enumeration")
Cc: stable@dpdk.org
Signed-off-by: Akhil Goyal <akhil.goyal@nxp.com>
---
changes in v2:
incorporated comments from Pablo
test/test/test_cryptodev.c | 2 +-
test/test/test_cryptodev_blockcipher.c | 4 ++--
test/test/test_cryptodev_perf.c | 4 ++--
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c
index a4116c6..132f99c 100644
--- a/test/test/test_cryptodev.c
+++ b/test/test/test_cryptodev.c
@@ -1816,7 +1816,7 @@ test_authonly_dpaa2_sec_all(void)
ts_params->session_mpool,
ts_params->valid_devs[0],
rte_cryptodev_driver_id_get(
- RTE_STR(RTE_CRYPTODEV_DPAA2_SEC_PMD)),
+ RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD)),
BLKCIPHER_AUTHONLY_TYPE);
TEST_ASSERT_EQUAL(status, 0, "Test failed");
diff --git a/test/test/test_cryptodev_blockcipher.c b/test/test/test_cryptodev_blockcipher.c
index 6089af4..baa8512 100644
--- a/test/test/test_cryptodev_blockcipher.c
+++ b/test/test/test_cryptodev_blockcipher.c
@@ -602,7 +602,7 @@ test_blockcipher_all_tests(struct rte_mempool *mbuf_pool,
int openssl_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_OPENSSL_PMD));
- int dpaa2_pmd = rte_cryptodev_driver_id_get(
+ int dpaa2_sec_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD));
int scheduler_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_SCHEDULER_PMD));
@@ -668,7 +668,7 @@ test_blockcipher_all_tests(struct rte_mempool *mbuf_pool,
target_pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_ARMV8;
else if (driver_id == scheduler_pmd)
target_pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_SCHEDULER;
- else if (driver_id == dpaa2_pmd)
+ else if (driver_id == dpaa2_sec_pmd)
target_pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_DPAA2_SEC;
else
TEST_ASSERT(0, "Unrecognized cryptodev type");
diff --git a/test/test/test_cryptodev_perf.c b/test/test/test_cryptodev_perf.c
index 3b57e6d..5df2e6e 100644
--- a/test/test/test_cryptodev_perf.c
+++ b/test/test/test_cryptodev_perf.c
@@ -211,7 +211,7 @@ static const char *pmd_name(uint8_t driver_id)
{
uint8_t null_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_NULL_PMD));
- uint8_t dpaa2_pmd = rte_cryptodev_driver_id_get(
+ uint8_t dpaa2_sec_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD));
uint8_t snow3g_pmd = rte_cryptodev_driver_id_get(
RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD));
@@ -232,7 +232,7 @@ static const char *pmd_name(uint8_t driver_id)
return RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD);
else if (driver_id == snow3g_pmd)
return RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD);
- else if (driver_id == dpaa2_pmd)
+ else if (driver_id == dpaa2_sec_pmd)
return RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD);
else
return "";
--
2.9.3
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions
2017-10-05 12:25 ` Akhil Goyal
@ 2017-10-05 12:59 ` De Lara Guarch, Pablo
0 siblings, 0 replies; 6+ messages in thread
From: De Lara Guarch, Pablo @ 2017-10-05 12:59 UTC (permalink / raw)
To: Akhil Goyal, dev; +Cc: Doherty, Declan, hemant.agrawal, stable
> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal@nxp.com]
> Sent: Thursday, October 5, 2017 1:26 PM
> To: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>;
> dev@dpdk.org
> Cc: Doherty, Declan <declan.doherty@intel.com>;
> hemant.agrawal@nxp.com; stable@dpdk.org
> Subject: Re: [PATCH] test/crypto: fix dpaa2_sec macros and definitions
>
> Hi Pablo,
> On 10/5/2017 1:49 PM, De Lara Guarch, Pablo wrote:
> > Hi Akhil,
> >
> >> -----Original Message-----
> >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Akhil Goyal
> >> Sent: Tuesday, October 3, 2017 10:29 AM
> >> To: dev@dpdk.org
> >> Cc: Doherty, Declan <declan.doherty@intel.com>; De Lara Guarch, Pablo
> >> <pablo.de.lara.guarch@intel.com>; hemant.agrawal@nxp.com;
> >> stable@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>
> >> Subject: [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and
> >> definitions
> >>
> >> Fixes: 7a364faef185 ("cryptodev: remove crypto device type
> >> enumeration")
> >>
> >> Cc: stable@dpdk.org
> >>
> >> Signed-off-by: Akhil Goyal <akhil.goyal@nxp.com>
> >> ---
> >> test/test/test_cryptodev.c | 2 +-
> >> test/test/test_cryptodev_perf.c | 4 ++--
> >> 2 files changed, 3 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/test/test/test_cryptodev.c b/test/test/test_cryptodev.c
> >> index a4116c6..132f99c 100644
> >> --- a/test/test/test_cryptodev.c
> >> +++ b/test/test/test_cryptodev.c
> >> @@ -1816,7 +1816,7 @@ test_authonly_dpaa2_sec_all(void)
> >> ts_params->session_mpool,
> >> ts_params->valid_devs[0],
> >> rte_cryptodev_driver_id_get(
> >> - RTE_STR(RTE_CRYPTODEV_DPAA2_SEC_PMD)),
> >> + RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD)),
> >> BLKCIPHER_AUTHONLY_TYPE);
> >>
> >> TEST_ASSERT_EQUAL(status, 0, "Test failed"); diff --git
> >> a/test/test/test_cryptodev_perf.c b/test/test/test_cryptodev_perf.c
> >> index 3b57e6d..5df2e6e 100644
> >> --- a/test/test/test_cryptodev_perf.c
> >> +++ b/test/test/test_cryptodev_perf.c
> >> @@ -211,7 +211,7 @@ static const char *pmd_name(uint8_t driver_id)
> {
> >> uint8_t null_pmd = rte_cryptodev_driver_id_get(
> >> RTE_STR(CRYPTODEV_NAME_NULL_PMD));
> >> - uint8_t dpaa2_pmd = rte_cryptodev_driver_id_get(
> >> + uint8_t dpaa2_sec_pmd = rte_cryptodev_driver_id_get(
> >
> > I think, in order to keep consistency, you should make this change also in
> test_cryptodev.c:
> >
> > test/test/test_cryptodev_blockcipher.c
> > 580: int dpaa2_pmd = rte_cryptodev_driver_id_get(
> > 646: else if (driver_id == dpaa2_pmd)
> >
> > Also, I submitted a patch to remove test_cryptodev_perf.c.
> > I will apply this patch first, so it is easier to integrate in the
> > stable version, but could you ack my patch if you are OK with it?
> >
> > The sooner we remove it, the better, as we are investing a lot of time
> > in the crypto-perf app, and there is no need to maintain a duplicate, in my
> opinion.
> >
> thanks for pointing this out. I would send a v2.
>
> I have a question regarding test_cryptodev_perf.c for dpaa_sec patches.
> The patchset for dpaa_sec includes changes in test_cryptodev_perf.c.
> Do you plan to add those patches on the tree as well before removing this
> file or I need to send another version?
> In test/test/test_cryptodev_blockcipher.c, I missed out this change for
> dpaa_sec also. So, I was sending another version for dpaa_sec patches to
> include this change. Should I remove the changes in the
> test_cryptodev_perf.c from the dpaa_sec patchset also?
You can leave them as they are. If you are OK with it, I think we can remove
test_cryptodev_perf.c and by the time I apply your patches, I will discard
the changes in that file.
Thanks,
Pablo
>
> Thanks,
> Akhil
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-dev] [PATCH v2] test/crypto: fix dpaa2_sec macros and definitions
2017-10-05 12:50 ` [dpdk-dev] [PATCH v2] " Akhil Goyal
@ 2017-10-05 13:30 ` De Lara Guarch, Pablo
0 siblings, 0 replies; 6+ messages in thread
From: De Lara Guarch, Pablo @ 2017-10-05 13:30 UTC (permalink / raw)
To: Akhil Goyal, dev; +Cc: Doherty, Declan, hemant.agrawal, stable
> -----Original Message-----
> From: Akhil Goyal [mailto:akhil.goyal@nxp.com]
> Sent: Thursday, October 5, 2017 1:50 PM
> To: dev@dpdk.org
> Cc: Doherty, Declan <declan.doherty@intel.com>; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>; hemant.agrawal@nxp.com;
> stable@dpdk.org; Akhil Goyal <akhil.goyal@nxp.com>
> Subject: [PATCH v2] test/crypto: fix dpaa2_sec macros and definitions
>
> Fixes: 7a364faef185 ("cryptodev: remove crypto device type enumeration")
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Akhil Goyal <akhil.goyal@nxp.com>
Applied to dpdk-next-crypto.
Thanks,
Pablo
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-10-05 13:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-03 9:29 [dpdk-dev] [PATCH] test/crypto: fix dpaa2_sec macros and definitions Akhil Goyal
2017-10-05 8:19 ` De Lara Guarch, Pablo
2017-10-05 12:25 ` Akhil Goyal
2017-10-05 12:59 ` De Lara Guarch, Pablo
2017-10-05 12:50 ` [dpdk-dev] [PATCH v2] " Akhil Goyal
2017-10-05 13:30 ` De Lara Guarch, Pablo
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).