* [dpdk-stable] [PATCH] test/crypto: fix autotest function parameters
@ 2021-06-28 16:15 Rebecca Troy
2021-06-29 14:32 ` Power, Ciara
2021-06-30 14:01 ` [dpdk-stable] [PATCH v2] " Rebecca Troy
0 siblings, 2 replies; 5+ messages in thread
From: Rebecca Troy @ 2021-06-28 16:15 UTC (permalink / raw)
To: dev
Cc: ciara.power, Rebecca Troy, declan.doherty, jianjay.zhou,
pablo.de.lara.guarch, roy.fan.zhang, akhil.goyal, hemant.agrawal,
stable
Fixed parameters on autotest functions by removing comments.
Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")
Fixes: 4ed1e79e7819 ("test/crypto: add tests for virtio-crypto")
Fixes: 3aafc423cf4d ("snow3g: add driver for SNOW 3G library")
Fixes: 27a1c4714d54 ("app/test: add KASUMI crypto")
Fixes: 4c99481f49c4 ("app/test: add ZUC")
Fixes: c8e69fce7046 ("crypto/scheduler: add unit test")
Fixes: ae002048bbea ("test/crypto: add DPAA2 crypto functional test")
Fixes: b674d6d0381a ("test/crypto: add dpaa crypto test cases")
Fixes: a8dbd44d6b4c ("test/crypto: add CAAM JR validation cases")
Fixes: 4868f6591c6f ("test/crypto: add cases for raw datapath API")
Cc: declan.doherty@intel.com
Cc: jianjay.zhou@huawei.com
Cc: pablo.de.lara.guarch@intel.com
Cc: roy.fan.zhang@intel.com
Cc: akhil.goyal@nxp.com
Cc: hemant.agrawal@nxp.com
Cc: stable@dpdk.org
Signed-off-by: Rebecca Troy <rebecca.troy@intel.com>
---
app/test/test_cryptodev.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 39db52b17a..c18730f138 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -14521,19 +14521,19 @@ run_cryptodev_testsuite(const char *pmd_name)
}
static int
-test_cryptodev_qat(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_qat(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD));
}
static int
-test_cryptodev_virtio(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_virtio(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_VIRTIO_PMD));
}
static int
-test_cryptodev_aesni_mb(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_aesni_mb(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_AESNI_MB_PMD));
}
@@ -14579,19 +14579,19 @@ test_cryptodev_null(void)
}
static int
-test_cryptodev_sw_snow3g(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_sw_snow3g(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD));
}
static int
-test_cryptodev_sw_kasumi(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_sw_kasumi(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_KASUMI_PMD));
}
static int
-test_cryptodev_sw_zuc(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_sw_zuc(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_ZUC_PMD));
}
@@ -14611,7 +14611,7 @@ test_cryptodev_mrvl(void)
#ifdef RTE_CRYPTO_SCHEDULER
static int
-test_cryptodev_scheduler(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_scheduler(void)
{
uint8_t ret, sched_i, j, i = 0, blk_start_idx = 0;
const enum blockcipher_test_type blk_suites[] = {
@@ -14719,13 +14719,13 @@ REGISTER_TEST_COMMAND(cryptodev_scheduler_autotest, test_cryptodev_scheduler);
#endif
static int
-test_cryptodev_dpaa2_sec(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_dpaa2_sec(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD));
}
static int
-test_cryptodev_dpaa_sec(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_dpaa_sec(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_DPAA_SEC_PMD));
}
@@ -14749,7 +14749,7 @@ test_cryptodev_octeontx2(void)
}
static int
-test_cryptodev_caam_jr(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_caam_jr(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_CAAM_JR_PMD));
}
@@ -14767,7 +14767,7 @@ test_cryptodev_bcmfs(void)
}
static int
-test_cryptodev_qat_raw_api(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_qat_raw_api(void)
{
int ret;
--
2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-stable] [PATCH] test/crypto: fix autotest function parameters
2021-06-28 16:15 [dpdk-stable] [PATCH] test/crypto: fix autotest function parameters Rebecca Troy
@ 2021-06-29 14:32 ` Power, Ciara
2021-06-30 14:01 ` [dpdk-stable] [PATCH v2] " Rebecca Troy
1 sibling, 0 replies; 5+ messages in thread
From: Power, Ciara @ 2021-06-29 14:32 UTC (permalink / raw)
To: Troy, Rebecca, dev
Cc: Doherty, Declan, jianjay.zhou, De Lara Guarch, Pablo, Zhang,
Roy Fan, Akhil Goyal, hemant.agrawal, stable
Hi Rebecca,
Looks good, thanks.
One minor change added inline below.
With this change,
Acked-by: Ciara Power <ciara.power@intel.com>
>-----Original Message-----
>From: Troy, Rebecca <rebecca.troy@intel.com>
>Sent: Monday 28 June 2021 17:15
>To: dev@dpdk.org
>Cc: Power, Ciara <ciara.power@intel.com>; Troy, Rebecca
><rebecca.troy@intel.com>; Doherty, Declan <declan.doherty@intel.com>;
>jianjay.zhou@huawei.com; De Lara Guarch, Pablo
><pablo.de.lara.guarch@intel.com>; Zhang, Roy Fan
><roy.fan.zhang@intel.com>; akhil.goyal@nxp.com;
>hemant.agrawal@nxp.com; stable@dpdk.org
>Subject: [PATCH] test/crypto: fix autotest function parameters
>
>Fixed parameters on autotest functions by removing comments.
>
>Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")
>Fixes: 4ed1e79e7819 ("test/crypto: add tests for virtio-crypto")
>Fixes: 3aafc423cf4d ("snow3g: add driver for SNOW 3G library")
>Fixes: 27a1c4714d54 ("app/test: add KASUMI crypto")
>Fixes: 4c99481f49c4 ("app/test: add ZUC")
>Fixes: c8e69fce7046 ("crypto/scheduler: add unit test")
>Fixes: ae002048bbea ("test/crypto: add DPAA2 crypto functional test")
>Fixes: b674d6d0381a ("test/crypto: add dpaa crypto test cases")
>Fixes: a8dbd44d6b4c ("test/crypto: add CAAM JR validation cases")
>Fixes: 4868f6591c6f ("test/crypto: add cases for raw datapath API")
>
>Cc: declan.doherty@intel.com
>Cc: jianjay.zhou@huawei.com
>Cc: pablo.de.lara.guarch@intel.com
>Cc: roy.fan.zhang@intel.com
>Cc: akhil.goyal@nxp.com
This should be updated to Akhil's new email address (cc'd): <gakhil@marvell.com>
>Cc: hemant.agrawal@nxp.com
>Cc: stable@dpdk.org
>
>Signed-off-by: Rebecca Troy <rebecca.troy@intel.com>
>---
> app/test/test_cryptodev.c | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
>diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c index
>39db52b17a..c18730f138 100644
>--- a/app/test/test_cryptodev.c
>+++ b/app/test/test_cryptodev.c
>@@ -14521,19 +14521,19 @@ run_cryptodev_testsuite(const char
>*pmd_name) }
>
> static int
>-test_cryptodev_qat(void /*argv __rte_unused, int argc __rte_unused*/)
>+test_cryptodev_qat(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD));
> }
>
> static int
>-test_cryptodev_virtio(void /*argv __rte_unused, int argc __rte_unused*/)
>+test_cryptodev_virtio(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_VIRTIO_PMD));
> }
>
> static int
>-test_cryptodev_aesni_mb(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_aesni_mb(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_AESNI_MB_PMD));
> }
>@@ -14579,19 +14579,19 @@ test_cryptodev_null(void) }
>
> static int
>-test_cryptodev_sw_snow3g(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_sw_snow3g(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD));
> }
>
> static int
>-test_cryptodev_sw_kasumi(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_sw_kasumi(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_KASUMI_PMD));
> }
>
> static int
>-test_cryptodev_sw_zuc(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_sw_zuc(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_ZUC_PMD));
> }
>@@ -14611,7 +14611,7 @@ test_cryptodev_mrvl(void) #ifdef
>RTE_CRYPTO_SCHEDULER
>
> static int
>-test_cryptodev_scheduler(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_scheduler(void)
> {
> uint8_t ret, sched_i, j, i = 0, blk_start_idx = 0;
> const enum blockcipher_test_type blk_suites[] = { @@ -14719,13
>+14719,13 @@ REGISTER_TEST_COMMAND(cryptodev_scheduler_autotest,
>test_cryptodev_scheduler); #endif
>
> static int
>-test_cryptodev_dpaa2_sec(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_dpaa2_sec(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD));
> }
>
> static int
>-test_cryptodev_dpaa_sec(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_dpaa_sec(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_DPAA_SEC_PMD));
> }
>@@ -14749,7 +14749,7 @@ test_cryptodev_octeontx2(void) }
>
> static int
>-test_cryptodev_caam_jr(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_caam_jr(void)
> {
> return
>run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_CAAM_JR_PMD));
> }
>@@ -14767,7 +14767,7 @@ test_cryptodev_bcmfs(void) }
>
> static int
>-test_cryptodev_qat_raw_api(void /*argv __rte_unused, int argc
>__rte_unused*/)
>+test_cryptodev_qat_raw_api(void)
> {
> int ret;
>
>--
>2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [dpdk-stable] [PATCH v2] test/crypto: fix autotest function parameters
2021-06-28 16:15 [dpdk-stable] [PATCH] test/crypto: fix autotest function parameters Rebecca Troy
2021-06-29 14:32 ` Power, Ciara
@ 2021-06-30 14:01 ` Rebecca Troy
2021-06-30 15:50 ` Hemant Agrawal
1 sibling, 1 reply; 5+ messages in thread
From: Rebecca Troy @ 2021-06-30 14:01 UTC (permalink / raw)
To: dev
Cc: ciara.power, Rebecca Troy, declan.doherty, jianjay.zhou,
pablo.de.lara.guarch, roy.fan.zhang, gakhil, hemant.agrawal,
stable
Fixed parameters on autotest functions by removing comments.
Fixes: 202d375c60bc ("app/test: add cryptodev unit and performance tests")
Fixes: 4ed1e79e7819 ("test/crypto: add tests for virtio-crypto")
Fixes: 3aafc423cf4d ("snow3g: add driver for SNOW 3G library")
Fixes: 27a1c4714d54 ("app/test: add KASUMI crypto")
Fixes: 4c99481f49c4 ("app/test: add ZUC")
Fixes: c8e69fce7046 ("crypto/scheduler: add unit test")
Fixes: ae002048bbea ("test/crypto: add DPAA2 crypto functional test")
Fixes: b674d6d0381a ("test/crypto: add dpaa crypto test cases")
Fixes: a8dbd44d6b4c ("test/crypto: add CAAM JR validation cases")
Fixes: 4868f6591c6f ("test/crypto: add cases for raw datapath API")
Cc: declan.doherty@intel.com
Cc: jianjay.zhou@huawei.com
Cc: pablo.de.lara.guarch@intel.com
Cc: roy.fan.zhang@intel.com
Cc: gakhil@marvell.com
Cc: hemant.agrawal@nxp.com
Cc: stable@dpdk.org
Signed-off-by: Rebecca Troy <rebecca.troy@intel.com>
Acked-by: Ciara Power <ciara.power@intel.com>
---
app/test/test_cryptodev.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
index 39db52b17a..c18730f138 100644
--- a/app/test/test_cryptodev.c
+++ b/app/test/test_cryptodev.c
@@ -14521,19 +14521,19 @@ run_cryptodev_testsuite(const char *pmd_name)
}
static int
-test_cryptodev_qat(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_qat(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_QAT_SYM_PMD));
}
static int
-test_cryptodev_virtio(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_virtio(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_VIRTIO_PMD));
}
static int
-test_cryptodev_aesni_mb(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_aesni_mb(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_AESNI_MB_PMD));
}
@@ -14579,19 +14579,19 @@ test_cryptodev_null(void)
}
static int
-test_cryptodev_sw_snow3g(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_sw_snow3g(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_SNOW3G_PMD));
}
static int
-test_cryptodev_sw_kasumi(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_sw_kasumi(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_KASUMI_PMD));
}
static int
-test_cryptodev_sw_zuc(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_sw_zuc(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_ZUC_PMD));
}
@@ -14611,7 +14611,7 @@ test_cryptodev_mrvl(void)
#ifdef RTE_CRYPTO_SCHEDULER
static int
-test_cryptodev_scheduler(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_scheduler(void)
{
uint8_t ret, sched_i, j, i = 0, blk_start_idx = 0;
const enum blockcipher_test_type blk_suites[] = {
@@ -14719,13 +14719,13 @@ REGISTER_TEST_COMMAND(cryptodev_scheduler_autotest, test_cryptodev_scheduler);
#endif
static int
-test_cryptodev_dpaa2_sec(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_dpaa2_sec(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_DPAA2_SEC_PMD));
}
static int
-test_cryptodev_dpaa_sec(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_dpaa_sec(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_DPAA_SEC_PMD));
}
@@ -14749,7 +14749,7 @@ test_cryptodev_octeontx2(void)
}
static int
-test_cryptodev_caam_jr(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_caam_jr(void)
{
return run_cryptodev_testsuite(RTE_STR(CRYPTODEV_NAME_CAAM_JR_PMD));
}
@@ -14767,7 +14767,7 @@ test_cryptodev_bcmfs(void)
}
static int
-test_cryptodev_qat_raw_api(void /*argv __rte_unused, int argc __rte_unused*/)
+test_cryptodev_qat_raw_api(void)
{
int ret;
--
2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-stable] [PATCH v2] test/crypto: fix autotest function parameters
2021-06-30 14:01 ` [dpdk-stable] [PATCH v2] " Rebecca Troy
@ 2021-06-30 15:50 ` Hemant Agrawal
2021-07-07 14:16 ` Akhil Goyal
0 siblings, 1 reply; 5+ messages in thread
From: Hemant Agrawal @ 2021-06-30 15:50 UTC (permalink / raw)
To: Rebecca Troy, dev
Cc: ciara.power, declan.doherty, jianjay.zhou, pablo.de.lara.guarch,
roy.fan.zhang, gakhil, stable
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [dpdk-stable] [PATCH v2] test/crypto: fix autotest function parameters
2021-06-30 15:50 ` Hemant Agrawal
@ 2021-07-07 14:16 ` Akhil Goyal
0 siblings, 0 replies; 5+ messages in thread
From: Akhil Goyal @ 2021-07-07 14:16 UTC (permalink / raw)
To: Hemant Agrawal, Rebecca Troy, dev
Cc: ciara.power, declan.doherty, jianjay.zhou, pablo.de.lara.guarch,
roy.fan.zhang, stable
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Acked-by: Akhil Goyal <gakhil@marvell.com>
Applied to dpdk-next-crypto
Thanks.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-07-07 14:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-28 16:15 [dpdk-stable] [PATCH] test/crypto: fix autotest function parameters Rebecca Troy
2021-06-29 14:32 ` Power, Ciara
2021-06-30 14:01 ` [dpdk-stable] [PATCH v2] " Rebecca Troy
2021-06-30 15:50 ` Hemant Agrawal
2021-07-07 14:16 ` Akhil Goyal
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).