DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [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-dev] [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-dev] [PATCH] test/crypto: fix autotest function parameters
  2021-06-28 16:15 [dpdk-dev] [PATCH] test/crypto: fix autotest function parameters Rebecca Troy
@ 2021-06-29 14:32 ` Power, Ciara
  2021-06-30 14:01 ` [dpdk-dev] [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-dev] [PATCH v2] test/crypto: fix autotest function parameters
  2021-06-28 16:15 [dpdk-dev] [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-dev] [PATCH v2] test/crypto: fix autotest function parameters
  2021-06-30 14:01 ` [dpdk-dev] [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-dev] [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-dev] [PATCH] test/crypto: fix autotest function parameters Rebecca Troy
2021-06-29 14:32 ` Power, Ciara
2021-06-30 14:01 ` [dpdk-dev] [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).