| Message ID | 1630588395-2804-8-git-send-email-fengchengwen@huawei.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Delegated to: | Thomas Monjalon |
| Headers | show |
| Series | support dmadev | expand |
| Context | Check | Description |
|---|---|---|
| ci/iol-x86_64-compile-testing | success | Testing PASS |
| ci/iol-mellanox-Performance | success | Performance Testing PASS |
| ci/iol-x86_64-unit-testing | success | Testing PASS |
| ci/intel-Testing | success | Testing PASS |
| ci/Intel-compilation | success | Compilation OK |
| ci/iol-aarch64-compile-testing | success | Testing PASS |
| ci/iol-intel-Functional | success | Functional Testing PASS |
| ci/iol-intel-Performance | success | Performance Testing PASS |
| ci/iol-broadcom-Performance | success | Performance Testing PASS |
| ci/iol-broadcom-Functional | success | Functional Testing PASS |
| ci/iol-spell-check-testing | success | Testing PASS |
| ci/github-robot: build | success | github build: passed |
| ci/checkpatch | success | coding style OK |
Hi Chengwen, The output from the API tests is not very straightforward to interpret if you are not familiar with these tests. Could we change the log level of the dmadev library before and after the API tests using something similar to The code I have included inline below? > +static int > +testsuite_setup(uint16_t dev_id) > +{ > + test_dev_id = dev_id; > + invalid_dev_id = RTE_DMADEV_MAX_DEVS; > + > + src = rte_malloc("dmadev_test_src", TEST_MEMCPY_SIZE, 0); > + if (src == NULL) > + return -ENOMEM; > + dst = rte_malloc("dmadev_test_dst", TEST_MEMCPY_SIZE, 0); > + if (dst == NULL) > + return -ENOMEM; /* Set dmadev log level to critical to suppress unnecessary output during API tests. */ rte_log_set_level_pattern("lib.dmadev", RTE_LOG_CRIT); > + > + total = 0; > + passed = 0; > + failed = 0; > + > + return 0; > +} > + > +static void > +testsuite_teardown(void) > +{ > + rte_free(src); > + rte_free(dst); > + /* Ensure the dmadev is stopped. */ > + rte_dmadev_stop(test_dev_id); rte_log_set_level_pattern("lib.dmadev", RTE_LOG_INFO); > +} This change would bring your output down from: ### Test dmadev infrastructure using skeleton driver test_dmadev_get_dev_id Passed test_dmadev_is_valid_dev Passed test_dmadev_count Passed Invalid dev_id=64 test_dmadev_info_get Passed Invalid dev_id=64 Device 1 configure zero vchans Device 1 configure too many vchans Device 1 don't support silent test_dmadev_configure Passed Invalid dev_id=64 Device 1 number of descriptors invalid Device 1 vchan out range! Device 1 direction invalid! Device 1 direction invalid! Device 1 don't support mem2dev transfer Device 1 don't support dev2mem transfer Device 1 don't support dev2dev transfer Device 1 number of descriptors invalid Device 1 number of descriptors invalid Device 1 source port type invalid Device 1 destination port type invalid test_dmadev_vchan_setup Passed Invalid dev_id=64 Invalid dev_id=64 Device 1 must be stopped to allow configuration Device 1 must be stopped to allow configuration test_dmadev_start_stop Passed Invalid dev_id=64 Invalid dev_id=64 Invalid dev_id=64 Device 1 vchan 1 out of range Device 1 vchan 1 out of range test_dmadev_stats Passed test_dmadev_completed Passed test_dmadev_completed_status Passed Device 1 already stopped Total tests : 10 Passed : 10 Failed : 0 skeldma_remove(): Remove dma_skeleton dmadev To: ### Test dmadev infrastructure using skeleton driver test_dmadev_get_dev_id Passed test_dmadev_is_valid_dev Passed test_dmadev_count Passed test_dmadev_info_get Passed test_dmadev_configure Passed test_dmadev_vchan_setup Passed test_dmadev_start_stop Passed test_dmadev_stats Passed test_dmadev_completed Passed test_dmadev_completed_status Passed Total tests : 10 Passed : 10 Failed : 0 skeldma_remove(): Remove dma_skeleton dmadev Thanks, Conor.
This is a tradeoff point. If we changed the log level of dmadev, it is difficult to know where the test case fails. So I prefer add more meaningful information, at least print out the function name. And V19 add format function name in log default, so the rte_dmadev's log will show like: rte_dmadev_configure(): Invalid dev_id=64 rte_dmadev_configure(): Device 4 configure zero vchans rte_dmadev_configure(): Device 4 configure too many vchans rte_dmadev_configure(): Device 4 don't support silent test_dmadev_configure Passed rte_dmadev_vchan_setup(): Invalid dev_id=64 rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid rte_dmadev_vchan_setup(): Device 4 vchan out range! rte_dmadev_vchan_setup(): Device 4 direction invalid! rte_dmadev_vchan_setup(): Device 4 direction invalid! rte_dmadev_vchan_setup(): Device 4 don't support mem2dev transfer rte_dmadev_vchan_setup(): Device 4 don't support dev2mem transfer rte_dmadev_vchan_setup(): Device 4 don't support dev2dev transfer rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid rte_dmadev_vchan_setup(): Device 4 source port type invalid rte_dmadev_vchan_setup(): Device 4 destination port type invalid On 2021/9/2 22:11, Walsh, Conor wrote: > Hi Chengwen, > > The output from the API tests is not very straightforward to interpret if you are not familiar with these tests. > Could we change the log level of the dmadev library before and after the API tests using something similar to > The code I have included inline below? > >> +static int >> +testsuite_setup(uint16_t dev_id) >> +{ >> + test_dev_id = dev_id; >> + invalid_dev_id = RTE_DMADEV_MAX_DEVS; >> + >> + src = rte_malloc("dmadev_test_src", TEST_MEMCPY_SIZE, 0); >> + if (src == NULL) >> + return -ENOMEM; >> + dst = rte_malloc("dmadev_test_dst", TEST_MEMCPY_SIZE, 0); >> + if (dst == NULL) >> + return -ENOMEM; > > /* Set dmadev log level to critical to suppress unnecessary output during API tests. */ > rte_log_set_level_pattern("lib.dmadev", RTE_LOG_CRIT); >> + >> + total = 0; >> + passed = 0; >> + failed = 0; >> + >> + return 0; >> +} >> + >> +static void >> +testsuite_teardown(void) >> +{ >> + rte_free(src); >> + rte_free(dst); >> + /* Ensure the dmadev is stopped. */ >> + rte_dmadev_stop(test_dev_id); > rte_log_set_level_pattern("lib.dmadev", RTE_LOG_INFO); >> +} > > This change would bring your output down from: > > ### Test dmadev infrastructure using skeleton driver > test_dmadev_get_dev_id Passed > test_dmadev_is_valid_dev Passed > test_dmadev_count Passed > Invalid dev_id=64 > test_dmadev_info_get Passed > Invalid dev_id=64 > Device 1 configure zero vchans > Device 1 configure too many vchans > Device 1 don't support silent > test_dmadev_configure Passed > Invalid dev_id=64 > Device 1 number of descriptors invalid > Device 1 vchan out range! > Device 1 direction invalid! > Device 1 direction invalid! > Device 1 don't support mem2dev transfer > Device 1 don't support dev2mem transfer > Device 1 don't support dev2dev transfer > Device 1 number of descriptors invalid > Device 1 number of descriptors invalid > Device 1 source port type invalid > Device 1 destination port type invalid > test_dmadev_vchan_setup Passed > Invalid dev_id=64 > Invalid dev_id=64 > Device 1 must be stopped to allow configuration > Device 1 must be stopped to allow configuration > test_dmadev_start_stop Passed > Invalid dev_id=64 > Invalid dev_id=64 > Invalid dev_id=64 > Device 1 vchan 1 out of range > Device 1 vchan 1 out of range > test_dmadev_stats Passed > test_dmadev_completed Passed > test_dmadev_completed_status Passed > Device 1 already stopped > Total tests : 10 > Passed : 10 > Failed : 0 > skeldma_remove(): Remove dma_skeleton dmadev > > To: > > ### Test dmadev infrastructure using skeleton driver > test_dmadev_get_dev_id Passed > test_dmadev_is_valid_dev Passed > test_dmadev_count Passed > test_dmadev_info_get Passed > test_dmadev_configure Passed > test_dmadev_vchan_setup Passed > test_dmadev_start_stop Passed > test_dmadev_stats Passed > test_dmadev_completed Passed > test_dmadev_completed_status Passed > Total tests : 10 > Passed : 10 > Failed : 0 > skeldma_remove(): Remove dma_skeleton dmadev > > Thanks, > Conor. > . >
On 02/09/2021 14:13, Chengwen Feng wrote: > This patch add dmadev API test which based on 'dma_skeleton' vdev. The > test cases could be executed using 'dmadev_autotest' command in test > framework. > > Signed-off-by: Chengwen Feng <fengchengwen@huawei.com> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > --- > MAINTAINERS | 1 + > app/test/meson.build | 4 + > app/test/test_dmadev.c | 45 ++++ > app/test/test_dmadev_api.c | 532 +++++++++++++++++++++++++++++++++++++++++++++ > 4 files changed, 582 insertions(+) > create mode 100644 app/test/test_dmadev.c > create mode 100644 app/test/test_dmadev_api.c > <snip> > + > +REGISTER_TEST_COMMAND(dmadev_autotest, test_dmadev); > diff --git a/app/test/test_dmadev_api.c b/app/test/test_dmadev_api.c > new file mode 100644 > index 0000000..a7795eb > --- /dev/null > +++ b/app/test/test_dmadev_api.c > @@ -0,0 +1,532 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(c) 2021 HiSilicon Limited. > + */ > + > +#include <stdint.h> > +#include <string.h> > + > +#include <rte_common.h> > +#include <rte_cycles.h> > +#include <rte_malloc.h> > +#include <rte_test.h> > +#include <rte_dmadev.h> > + > +extern int test_dmadev_api(uint16_t dev_id); > + > +#define SKELDMA_TEST_RUN(test) \ > + testsuite_run_test(test, #test) > + > +#define TEST_MEMCPY_SIZE 1024 > +#define TEST_WAIT_US_VAL 50000 > + > +#define TEST_SUCCESS 0 > +#define TEST_FAILED -1 > + > +static uint16_t test_dev_id; > +static uint16_t invalid_dev_id; > + > +static int total; > +static int passed; > +static int failed; > +static char *src; > +static char *dst; > + > +static int > +testsuite_setup(uint16_t dev_id) > +{ > + test_dev_id = dev_id; > + invalid_dev_id = RTE_DMADEV_MAX_DEVS; > + > + src = rte_malloc("dmadev_test_src", TEST_MEMCPY_SIZE, 0); > + if (src == NULL) > + return -ENOMEM; > + dst = rte_malloc("dmadev_test_dst", TEST_MEMCPY_SIZE, 0); > + if (dst == NULL) > + return -ENOMEM; > + > + total = 0; > + passed = 0; > + failed = 0; > + > + return 0; > +} > + > +static void > +testsuite_teardown(void) > +{ > + rte_free(src); > + rte_free(dst); These should be set to NULL after free. > + /* Ensure the dmadev is stopped. */ > + rte_dmadev_stop(test_dev_id); > +} > + <snip> > + > +int > +test_dmadev_api(uint16_t dev_id) > +{ > + int ret = testsuite_setup(dev_id); > + if (ret) { If testsuite setup fails, src/dst potentially need to be free'd, so "testsuite_teardown()" should be called here. > + printf("testsuite setup fail!\n"); > + return -1; > + } > + > + /* If the testcase exit successfully, ensure that the test dmadev exist > + * and the dmadev is in the stopped state. > + */ > + SKELDMA_TEST_RUN(test_dmadev_get_dev_id); > + SKELDMA_TEST_RUN(test_dmadev_is_valid_dev); > + SKELDMA_TEST_RUN(test_dmadev_count); > + SKELDMA_TEST_RUN(test_dmadev_info_get); > + SKELDMA_TEST_RUN(test_dmadev_configure); > + SKELDMA_TEST_RUN(test_dmadev_vchan_setup); > + SKELDMA_TEST_RUN(test_dmadev_start_stop); > + SKELDMA_TEST_RUN(test_dmadev_stats); > + SKELDMA_TEST_RUN(test_dmadev_dump); > + SKELDMA_TEST_RUN(test_dmadev_completed); > + SKELDMA_TEST_RUN(test_dmadev_completed_status); > + > + testsuite_teardown(); > + > + printf("Total tests : %d\n", total); > + printf("Passed : %d\n", passed); > + printf("Failed : %d\n", failed); > + > + if (failed) > + return -1; > + > + return 0; > +}; With the above comments addressed, Reviewed-by: Kevin Laatz <kevin.laatz@intel.com>
> This is a tradeoff point. If we changed the log level of dmadev, it is difficult to > know where the test case fails. > > So I prefer add more meaningful information, at least print out the function > name. > > And V19 add format function name in log default, so the rte_dmadev's log > will show like: > > rte_dmadev_configure(): Invalid dev_id=64 > rte_dmadev_configure(): Device 4 configure zero vchans > rte_dmadev_configure(): Device 4 configure too many vchans > rte_dmadev_configure(): Device 4 don't support silent > test_dmadev_configure Passed > rte_dmadev_vchan_setup(): Invalid dev_id=64 > rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid > rte_dmadev_vchan_setup(): Device 4 vchan out range! > rte_dmadev_vchan_setup(): Device 4 direction invalid! > rte_dmadev_vchan_setup(): Device 4 direction invalid! > rte_dmadev_vchan_setup(): Device 4 don't support mem2dev transfer > rte_dmadev_vchan_setup(): Device 4 don't support dev2mem transfer > rte_dmadev_vchan_setup(): Device 4 don't support dev2dev transfer > rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid > rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid > rte_dmadev_vchan_setup(): Device 4 source port type invalid > rte_dmadev_vchan_setup(): Device 4 destination port type invalid I thought it would be cleaner, could you suppress the output only before and re-enable after negative testing? It's a lot of output to print to a user, in the driver tests extra information is only printed in the case of an error or failure. The line of code shown below will only suppress output from the dmadev lib not EAL or test. I don’t have a very strong opinion either way, I wanted to improve usability. With or without this change/cleanup: Reviewed-by: Conor Walsh <conor.walsh@intel.com> > > > On 2021/9/2 22:11, Walsh, Conor wrote: > > Hi Chengwen, > > > > The output from the API tests is not very straightforward to interpret if you > are not familiar with these tests. > > Could we change the log level of the dmadev library before and after the > API tests using something similar to > > The code I have included inline below? > > > >> +static int > >> +testsuite_setup(uint16_t dev_id) > >> +{ > >> + test_dev_id = dev_id; > >> + invalid_dev_id = RTE_DMADEV_MAX_DEVS; > >> + > >> + src = rte_malloc("dmadev_test_src", TEST_MEMCPY_SIZE, 0); > >> + if (src == NULL) > >> + return -ENOMEM; > >> + dst = rte_malloc("dmadev_test_dst", TEST_MEMCPY_SIZE, 0); > >> + if (dst == NULL) > >> + return -ENOMEM; > > > > /* Set dmadev log level to critical to suppress unnecessary output > during API tests. */ > > rte_log_set_level_pattern("lib.dmadev", RTE_LOG_CRIT); > >> + > >> + total = 0; > >> + passed = 0; > >> + failed = 0; > >> + > >> + return 0; > >> +} > >> + > >> +static void > >> +testsuite_teardown(void) > >> +{ > >> + rte_free(src); > >> + rte_free(dst); > >> + /* Ensure the dmadev is stopped. */ > >> + rte_dmadev_stop(test_dev_id); > > rte_log_set_level_pattern("lib.dmadev", RTE_LOG_INFO); > >> +} > > > > This change would bring your output down from: > > > > ### Test dmadev infrastructure using skeleton driver > > test_dmadev_get_dev_id Passed > > test_dmadev_is_valid_dev Passed > > test_dmadev_count Passed > > Invalid dev_id=64 > > test_dmadev_info_get Passed > > Invalid dev_id=64 > > Device 1 configure zero vchans > > Device 1 configure too many vchans > > Device 1 don't support silent > > test_dmadev_configure Passed > > Invalid dev_id=64 > > Device 1 number of descriptors invalid > > Device 1 vchan out range! > > Device 1 direction invalid! > > Device 1 direction invalid! > > Device 1 don't support mem2dev transfer > > Device 1 don't support dev2mem transfer > > Device 1 don't support dev2dev transfer > > Device 1 number of descriptors invalid > > Device 1 number of descriptors invalid > > Device 1 source port type invalid > > Device 1 destination port type invalid > > test_dmadev_vchan_setup Passed > > Invalid dev_id=64 > > Invalid dev_id=64 > > Device 1 must be stopped to allow configuration > > Device 1 must be stopped to allow configuration > > test_dmadev_start_stop Passed > > Invalid dev_id=64 > > Invalid dev_id=64 > > Invalid dev_id=64 > > Device 1 vchan 1 out of range > > Device 1 vchan 1 out of range > > test_dmadev_stats Passed > > test_dmadev_completed Passed > > test_dmadev_completed_status Passed > > Device 1 already stopped > > Total tests : 10 > > Passed : 10 > > Failed : 0 > > skeldma_remove(): Remove dma_skeleton dmadev > > > > To: > > > > ### Test dmadev infrastructure using skeleton driver > > test_dmadev_get_dev_id Passed > > test_dmadev_is_valid_dev Passed > > test_dmadev_count Passed > > test_dmadev_info_get Passed > > test_dmadev_configure Passed > > test_dmadev_vchan_setup Passed > > test_dmadev_start_stop Passed > > test_dmadev_stats Passed > > test_dmadev_completed Passed > > test_dmadev_completed_status Passed > > Total tests : 10 > > Passed : 10 > > Failed : 0 > > skeldma_remove(): Remove dma_skeleton dmadev > > > > Thanks, > > Conor. > > . > >
On 2021/9/3 23:38, Walsh, Conor wrote: > >> This is a tradeoff point. If we changed the log level of dmadev, it is difficult to >> know where the test case fails. >> >> So I prefer add more meaningful information, at least print out the function >> name. >> >> And V19 add format function name in log default, so the rte_dmadev's log >> will show like: >> >> rte_dmadev_configure(): Invalid dev_id=64 >> rte_dmadev_configure(): Device 4 configure zero vchans >> rte_dmadev_configure(): Device 4 configure too many vchans >> rte_dmadev_configure(): Device 4 don't support silent >> test_dmadev_configure Passed >> rte_dmadev_vchan_setup(): Invalid dev_id=64 >> rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid >> rte_dmadev_vchan_setup(): Device 4 vchan out range! >> rte_dmadev_vchan_setup(): Device 4 direction invalid! >> rte_dmadev_vchan_setup(): Device 4 direction invalid! >> rte_dmadev_vchan_setup(): Device 4 don't support mem2dev transfer >> rte_dmadev_vchan_setup(): Device 4 don't support dev2mem transfer >> rte_dmadev_vchan_setup(): Device 4 don't support dev2dev transfer >> rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid >> rte_dmadev_vchan_setup(): Device 4 number of descriptors invalid >> rte_dmadev_vchan_setup(): Device 4 source port type invalid >> rte_dmadev_vchan_setup(): Device 4 destination port type invalid > > I thought it would be cleaner, could you suppress the output only before and re-enable after negative testing? > It's a lot of output to print to a user, in the driver tests extra information is only printed in the case > of an error or failure. The line of code shown below will only suppress output from the dmadev lib not > EAL or test. Yes, you're right, UT is fine enough to quickly pinpoint which error. will use your method to modify, thanks > > I don’t have a very strong opinion either way, I wanted to improve usability. > > With or without this change/cleanup: > > Reviewed-by: Conor Walsh <conor.walsh@intel.com> > >> >> >> On 2021/9/2 22:11, Walsh, Conor wrote: >>> Hi Chengwen, >>> >>> The output from the API tests is not very straightforward to interpret if you >> are not familiar with these tests. >>> Could we change the log level of the dmadev library before and after the >> API tests using something similar to >>> The code I have included inline below? >>> >>>> +static int >>>> +testsuite_setup(uint16_t dev_id) >>>> +{ >>>> + test_dev_id = dev_id; >>>> + invalid_dev_id = RTE_DMADEV_MAX_DEVS; >>>> + >>>> + src = rte_malloc("dmadev_test_src", TEST_MEMCPY_SIZE, 0); >>>> + if (src == NULL) >>>> + return -ENOMEM; >>>> + dst = rte_malloc("dmadev_test_dst", TEST_MEMCPY_SIZE, 0); >>>> + if (dst == NULL) >>>> + return -ENOMEM; >>> >>> /* Set dmadev log level to critical to suppress unnecessary output >> during API tests. */ >>> rte_log_set_level_pattern("lib.dmadev", RTE_LOG_CRIT); >>>> + >>>> + total = 0; >>>> + passed = 0; >>>> + failed = 0; >>>> + >>>> + return 0; >>>> +} >>>> + >>>> +static void >>>> +testsuite_teardown(void) >>>> +{ >>>> + rte_free(src); >>>> + rte_free(dst); >>>> + /* Ensure the dmadev is stopped. */ >>>> + rte_dmadev_stop(test_dev_id); >>> rte_log_set_level_pattern("lib.dmadev", RTE_LOG_INFO); >>>> +} >>> >>> This change would bring your output down from: >>> >>> ### Test dmadev infrastructure using skeleton driver >>> test_dmadev_get_dev_id Passed >>> test_dmadev_is_valid_dev Passed >>> test_dmadev_count Passed >>> Invalid dev_id=64 >>> test_dmadev_info_get Passed >>> Invalid dev_id=64 >>> Device 1 configure zero vchans >>> Device 1 configure too many vchans >>> Device 1 don't support silent >>> test_dmadev_configure Passed >>> Invalid dev_id=64 >>> Device 1 number of descriptors invalid >>> Device 1 vchan out range! >>> Device 1 direction invalid! >>> Device 1 direction invalid! >>> Device 1 don't support mem2dev transfer >>> Device 1 don't support dev2mem transfer >>> Device 1 don't support dev2dev transfer >>> Device 1 number of descriptors invalid >>> Device 1 number of descriptors invalid >>> Device 1 source port type invalid >>> Device 1 destination port type invalid >>> test_dmadev_vchan_setup Passed >>> Invalid dev_id=64 >>> Invalid dev_id=64 >>> Device 1 must be stopped to allow configuration >>> Device 1 must be stopped to allow configuration >>> test_dmadev_start_stop Passed >>> Invalid dev_id=64 >>> Invalid dev_id=64 >>> Invalid dev_id=64 >>> Device 1 vchan 1 out of range >>> Device 1 vchan 1 out of range >>> test_dmadev_stats Passed >>> test_dmadev_completed Passed >>> test_dmadev_completed_status Passed >>> Device 1 already stopped >>> Total tests : 10 >>> Passed : 10 >>> Failed : 0 >>> skeldma_remove(): Remove dma_skeleton dmadev >>> >>> To: >>> >>> ### Test dmadev infrastructure using skeleton driver >>> test_dmadev_get_dev_id Passed >>> test_dmadev_is_valid_dev Passed >>> test_dmadev_count Passed >>> test_dmadev_info_get Passed >>> test_dmadev_configure Passed >>> test_dmadev_vchan_setup Passed >>> test_dmadev_start_stop Passed >>> test_dmadev_stats Passed >>> test_dmadev_completed Passed >>> test_dmadev_completed_status Passed >>> Total tests : 10 >>> Passed : 10 >>> Failed : 0 >>> skeldma_remove(): Remove dma_skeleton dmadev >>> >>> Thanks, >>> Conor. >>> . >>>
diff --git a/MAINTAINERS b/MAINTAINERS index e69fb28..17763c8 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -500,6 +500,7 @@ DMA device API - EXPERIMENTAL M: Chengwen Feng <fengchengwen@huawei.com> F: lib/dmadev/ F: drivers/dma/skeleton/ +F: app/test/test_dmadev* F: doc/guides/prog_guide/dmadev.rst diff --git a/app/test/meson.build b/app/test/meson.build index a761168..9027eba 100644 --- a/app/test/meson.build +++ b/app/test/meson.build @@ -43,6 +43,8 @@ test_sources = files( 'test_debug.c', 'test_distributor.c', 'test_distributor_perf.c', + 'test_dmadev.c', + 'test_dmadev_api.c', 'test_eal_flags.c', 'test_eal_fs.c', 'test_efd.c', @@ -162,6 +164,7 @@ test_deps = [ 'cmdline', 'cryptodev', 'distributor', + 'dmadev', 'efd', 'ethdev', 'eventdev', @@ -333,6 +336,7 @@ driver_test_names = [ 'cryptodev_sw_mvsam_autotest', 'cryptodev_sw_snow3g_autotest', 'cryptodev_sw_zuc_autotest', + 'dmadev_autotest', 'eventdev_selftest_octeontx', 'eventdev_selftest_sw', 'rawdev_autotest', diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c new file mode 100644 index 0000000..bb01e86 --- /dev/null +++ b/app/test/test_dmadev.c @@ -0,0 +1,45 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(c) 2021 HiSilicon Limited. + * Copyright(c) 2021 Intel Corporation. + */ + +#include <rte_common.h> +#include <rte_dev.h> +#include <rte_dmadev.h> +#include <rte_bus_vdev.h> + +#include "test.h" + +/* from test_dmadev_api.c */ +extern int test_dmadev_api(uint16_t dev_id); + +static int +test_apis(void) +{ + const char *pmd = "dma_skeleton"; + int id; + int ret; + + if (rte_vdev_init(pmd, NULL) < 0) + return TEST_SKIPPED; + id = rte_dmadev_get_dev_id(pmd); + if (id < 0) + return TEST_SKIPPED; + printf("\n### Test dmadev infrastructure using skeleton driver\n"); + ret = test_dmadev_api(id); + rte_vdev_uninit(pmd); + + return ret; +} + +static int +test_dmadev(void) +{ + /* basic sanity on dmadev infrastructure */ + if (test_apis() < 0) + return -1; + + return 0; +} + +REGISTER_TEST_COMMAND(dmadev_autotest, test_dmadev); diff --git a/app/test/test_dmadev_api.c b/app/test/test_dmadev_api.c new file mode 100644 index 0000000..a7795eb --- /dev/null +++ b/app/test/test_dmadev_api.c @@ -0,0 +1,532 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright(c) 2021 HiSilicon Limited. + */ + +#include <stdint.h> +#include <string.h> + +#include <rte_common.h> +#include <rte_cycles.h> +#include <rte_malloc.h> +#include <rte_test.h> +#include <rte_dmadev.h> + +extern int test_dmadev_api(uint16_t dev_id); + +#define SKELDMA_TEST_RUN(test) \ + testsuite_run_test(test, #test) + +#define TEST_MEMCPY_SIZE 1024 +#define TEST_WAIT_US_VAL 50000 + +#define TEST_SUCCESS 0 +#define TEST_FAILED -1 + +static uint16_t test_dev_id; +static uint16_t invalid_dev_id; + +static int total; +static int passed; +static int failed; +static char *src; +static char *dst; + +static int +testsuite_setup(uint16_t dev_id) +{ + test_dev_id = dev_id; + invalid_dev_id = RTE_DMADEV_MAX_DEVS; + + src = rte_malloc("dmadev_test_src", TEST_MEMCPY_SIZE, 0); + if (src == NULL) + return -ENOMEM; + dst = rte_malloc("dmadev_test_dst", TEST_MEMCPY_SIZE, 0); + if (dst == NULL) + return -ENOMEM; + + total = 0; + passed = 0; + failed = 0; + + return 0; +} + +static void +testsuite_teardown(void) +{ + rte_free(src); + rte_free(dst); + /* Ensure the dmadev is stopped. */ + rte_dmadev_stop(test_dev_id); +} + +static void +testsuite_run_test(int (*test)(void), const char *name) +{ + int ret = 0; + + if (test) { + ret = test(); + if (ret < 0) { + failed++; + printf("%s Failed\n", name); + } else { + passed++; + printf("%s Passed\n", name); + } + } + + total++; +} + +static int +test_dmadev_get_dev_id(void) +{ + int ret = rte_dmadev_get_dev_id("invalid_dmadev_device"); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + return TEST_SUCCESS; +} + +static int +test_dmadev_is_valid_dev(void) +{ + int ret; + ret = rte_dmadev_is_valid_dev(invalid_dev_id); + RTE_TEST_ASSERT(ret == false, "Expected false for invalid dev id"); + ret = rte_dmadev_is_valid_dev(test_dev_id); + RTE_TEST_ASSERT(ret == true, "Expected true for valid dev id"); + return TEST_SUCCESS; +} + +static int +test_dmadev_count(void) +{ + uint16_t count = rte_dmadev_count(); + RTE_TEST_ASSERT(count > 0, "Invalid dmadev count %u", count); + return TEST_SUCCESS; +} + +static int +test_dmadev_info_get(void) +{ + struct rte_dmadev_info info = { 0 }; + int ret; + + ret = rte_dmadev_info_get(invalid_dev_id, &info); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_info_get(test_dev_id, NULL); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_info_get(test_dev_id, &info); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to obtain device info"); + + return TEST_SUCCESS; +} + +static int +test_dmadev_configure(void) +{ + struct rte_dmadev_conf conf = { 0 }; + struct rte_dmadev_info info = { 0 }; + int ret; + + /* Check for invalid parameters */ + ret = rte_dmadev_configure(invalid_dev_id, &conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_configure(test_dev_id, NULL); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check for nb_vchans == 0 */ + memset(&conf, 0, sizeof(conf)); + ret = rte_dmadev_configure(test_dev_id, &conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check for conf.nb_vchans > info.max_vchans */ + ret = rte_dmadev_info_get(test_dev_id, &info); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to obtain device info"); + memset(&conf, 0, sizeof(conf)); + conf.nb_vchans = info.max_vchans + 1; + ret = rte_dmadev_configure(test_dev_id, &conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check enable silent mode */ + memset(&conf, 0, sizeof(conf)); + conf.nb_vchans = info.max_vchans; + conf.enable_silent = true; + ret = rte_dmadev_configure(test_dev_id, &conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Configure success */ + memset(&conf, 0, sizeof(conf)); + conf.nb_vchans = info.max_vchans; + ret = rte_dmadev_configure(test_dev_id, &conf); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to configure dmadev, %d", ret); + + /* Check configure success */ + ret = rte_dmadev_info_get(test_dev_id, &info); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to obtain device info"); + RTE_TEST_ASSERT_EQUAL(conf.nb_vchans, info.nb_vchans, + "Configure nb_vchans not match"); + + return TEST_SUCCESS; +} + +static int +test_dmadev_vchan_setup(void) +{ + struct rte_dmadev_vchan_conf vchan_conf = { 0 }; + struct rte_dmadev_conf dev_conf = { 0 }; + struct rte_dmadev_info dev_info = { 0 }; + int ret; + + /* Check for invalid parameters */ + ret = rte_dmadev_vchan_setup(invalid_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_vchan_setup(test_dev_id, 0, NULL); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Make sure configure success */ + ret = rte_dmadev_info_get(test_dev_id, &dev_info); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to obtain device info"); + dev_conf.nb_vchans = dev_info.max_vchans; + ret = rte_dmadev_configure(test_dev_id, &dev_conf); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to configure dmadev, %d", ret); + + /* Check for invalid vchan */ + ret = rte_dmadev_vchan_setup(test_dev_id, dev_conf.nb_vchans, + &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check for direction */ + memset(&vchan_conf, 0, sizeof(vchan_conf)); + vchan_conf.direction = RTE_DMA_DIR_DEV_TO_DEV + 1; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_MEM - 1; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check for direction and dev_capa combination */ + memset(&vchan_conf, 0, sizeof(vchan_conf)); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_DEV; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + vchan_conf.direction = RTE_DMA_DIR_DEV_TO_MEM; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + vchan_conf.direction = RTE_DMA_DIR_DEV_TO_DEV; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check for nb_desc validation */ + memset(&vchan_conf, 0, sizeof(vchan_conf)); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_MEM; + vchan_conf.nb_desc = dev_info.min_desc - 1; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + vchan_conf.nb_desc = dev_info.max_desc + 1; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check src port type validation */ + memset(&vchan_conf, 0, sizeof(vchan_conf)); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_MEM; + vchan_conf.nb_desc = dev_info.min_desc; + vchan_conf.src_port.port_type = RTE_DMADEV_PORT_PCIE; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check dst port type validation */ + memset(&vchan_conf, 0, sizeof(vchan_conf)); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_MEM; + vchan_conf.nb_desc = dev_info.min_desc; + vchan_conf.dst_port.port_type = RTE_DMADEV_PORT_PCIE; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check vchan setup success */ + memset(&vchan_conf, 0, sizeof(vchan_conf)); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_MEM; + vchan_conf.nb_desc = dev_info.min_desc; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to setup vchan, %d", ret); + + return TEST_SUCCESS; +} + +static int +setup_one_vchan(void) +{ + struct rte_dmadev_vchan_conf vchan_conf = { 0 }; + struct rte_dmadev_info dev_info = { 0 }; + struct rte_dmadev_conf dev_conf = { 0 }; + int ret; + + ret = rte_dmadev_info_get(test_dev_id, &dev_info); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to obtain device info, %d", ret); + dev_conf.nb_vchans = dev_info.max_vchans; + ret = rte_dmadev_configure(test_dev_id, &dev_conf); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to configure, %d", ret); + vchan_conf.direction = RTE_DMA_DIR_MEM_TO_MEM; + vchan_conf.nb_desc = dev_info.min_desc; + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to setup vchan, %d", ret); + + return TEST_SUCCESS; +} + +static int +test_dmadev_start_stop(void) +{ + struct rte_dmadev_vchan_conf vchan_conf = { 0 }; + struct rte_dmadev_conf dev_conf = { 0 }; + int ret; + + /* Check for invalid parameters */ + ret = rte_dmadev_start(invalid_dev_id); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_stop(invalid_dev_id); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Setup one vchan for later test */ + ret = setup_one_vchan(); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to setup one vchan, %d", ret); + + ret = rte_dmadev_start(test_dev_id); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to start, %d", ret); + + /* Check reconfigure and vchan setup when device started */ + ret = rte_dmadev_configure(test_dev_id, &dev_conf); + RTE_TEST_ASSERT(ret == -EBUSY, "Failed to configure, %d", ret); + ret = rte_dmadev_vchan_setup(test_dev_id, 0, &vchan_conf); + RTE_TEST_ASSERT(ret == -EBUSY, "Failed to setup vchan, %d", ret); + + ret = rte_dmadev_stop(test_dev_id); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to stop, %d", ret); + + return TEST_SUCCESS; +} + +static int +test_dmadev_stats(void) +{ + struct rte_dmadev_info dev_info = { 0 }; + struct rte_dmadev_stats stats = { 0 }; + int ret; + + /* Check for invalid parameters */ + ret = rte_dmadev_stats_get(invalid_dev_id, 0, &stats); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_stats_get(invalid_dev_id, 0, NULL); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_stats_reset(invalid_dev_id, 0); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Setup one vchan for later test */ + ret = setup_one_vchan(); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to setup one vchan, %d", ret); + + /* Check for invalid vchan */ + ret = rte_dmadev_info_get(test_dev_id, &dev_info); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to obtain device info, %d", ret); + ret = rte_dmadev_stats_get(test_dev_id, dev_info.max_vchans, &stats); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + ret = rte_dmadev_stats_reset(test_dev_id, dev_info.max_vchans); + RTE_TEST_ASSERT(ret == -EINVAL, "Expected -EINVAL, %d", ret); + + /* Check for valid vchan */ + ret = rte_dmadev_stats_get(test_dev_id, 0, &stats); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to get stats, %d", ret); + ret = rte_dmadev_stats_get(test_dev_id, RTE_DMADEV_ALL_VCHAN, &stats); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to get all stats, %d", ret); + ret = rte_dmadev_stats_reset(test_dev_id, 0); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to reset stats, %d", ret); + ret = rte_dmadev_stats_reset(test_dev_id, RTE_DMADEV_ALL_VCHAN); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to reset all stats, %d", ret); + + return TEST_SUCCESS; +} + +static int +test_dmadev_dump(void) +{ + int ret; + + /* Check for invalid parameters */ + ret = rte_dmadev_dump(invalid_dev_id, stderr); + RTE_TEST_ASSERT(ret == -EINVAL, "Excepted -EINVAL, %d", ret); + ret = rte_dmadev_dump(test_dev_id, NULL); + RTE_TEST_ASSERT(ret == -EINVAL, "Excepted -EINVAL, %d", ret); + + return TEST_SUCCESS; +} + +static int +test_dmadev_completed(void) +{ + uint16_t last_idx = 1; + bool has_error = true; + uint16_t cpl_ret; + int ret, i; + + /* Setup one vchan for later test */ + ret = setup_one_vchan(); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to setup one vchan, %d", ret); + + ret = rte_dmadev_start(test_dev_id); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to start, %d", ret); + + /* Setup test memory */ + for (i = 0; i < TEST_MEMCPY_SIZE; i++) + src[i] = (char)i; + memset(dst, 0, TEST_MEMCPY_SIZE); + + /* Check enqueue without submit */ + ret = rte_dmadev_copy(test_dev_id, 0, (rte_iova_t)src, (rte_iova_t)dst, + TEST_MEMCPY_SIZE, 0); + RTE_TEST_ASSERT_EQUAL(ret, 0, "Failed to enqueue copy, %d", ret); + rte_delay_us_sleep(TEST_WAIT_US_VAL); + cpl_ret = rte_dmadev_completed(test_dev_id, 0, 1, &last_idx, + &has_error); + RTE_TEST_ASSERT_EQUAL(cpl_ret, 0, "Failed to get completed"); + + /* Check add submit */ + ret = rte_dmadev_submit(test_dev_id, 0); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to submit, %d", ret); + rte_delay_us_sleep(TEST_WAIT_US_VAL); + cpl_ret = rte_dmadev_completed(test_dev_id, 0, 1, &last_idx, + &has_error); + RTE_TEST_ASSERT_EQUAL(cpl_ret, 1, "Failed to get completed"); + RTE_TEST_ASSERT_EQUAL(last_idx, 0, "Last idx should be zero, %u", + last_idx); + RTE_TEST_ASSERT_EQUAL(has_error, false, "Should have no error"); + for (i = 0; i < TEST_MEMCPY_SIZE; i++) { + if (src[i] != dst[i]) { + RTE_TEST_ASSERT_EQUAL(src[i], dst[i], + "Failed to copy memory, %d %d", src[i], dst[i]); + break; + } + } + + /* Setup test memory */ + for (i = 0; i < TEST_MEMCPY_SIZE; i++) + src[i] = (char)i; + memset(dst, 0, TEST_MEMCPY_SIZE); + + /* Check for enqueue with submit */ + ret = rte_dmadev_copy(test_dev_id, 0, (rte_iova_t)src, (rte_iova_t)dst, + TEST_MEMCPY_SIZE, RTE_DMA_OP_FLAG_SUBMIT); + RTE_TEST_ASSERT_EQUAL(ret, 1, "Failed to enqueue copy, %d", ret); + rte_delay_us_sleep(TEST_WAIT_US_VAL); + cpl_ret = rte_dmadev_completed(test_dev_id, 0, 1, &last_idx, + &has_error); + RTE_TEST_ASSERT_EQUAL(cpl_ret, 1, "Failed to get completed"); + RTE_TEST_ASSERT_EQUAL(last_idx, 1, "Last idx should be 1, %u", + last_idx); + RTE_TEST_ASSERT_EQUAL(has_error, false, "Should have no error"); + for (i = 0; i < TEST_MEMCPY_SIZE; i++) { + if (src[i] != dst[i]) { + RTE_TEST_ASSERT_EQUAL(src[i], dst[i], + "Failed to copy memory, %d %d", src[i], dst[i]); + break; + } + } + + /* Stop dmadev to make sure dmadev to a known state */ + ret = rte_dmadev_stop(test_dev_id); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to stop, %d", ret); + + return TEST_SUCCESS; +} + +static int +test_dmadev_completed_status(void) +{ + enum rte_dma_status_code status[1] = { 1 }; + uint16_t last_idx = 1; + uint16_t cpl_ret, i; + int ret; + + /* Setup one vchan for later test */ + ret = setup_one_vchan(); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to setup one vchan, %d", ret); + + ret = rte_dmadev_start(test_dev_id); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to start, %d", ret); + + /* Check for enqueue with submit */ + ret = rte_dmadev_copy(test_dev_id, 0, (rte_iova_t)src, (rte_iova_t)dst, + TEST_MEMCPY_SIZE, RTE_DMA_OP_FLAG_SUBMIT); + RTE_TEST_ASSERT_EQUAL(ret, 0, "Failed to enqueue copy, %d", ret); + rte_delay_us_sleep(TEST_WAIT_US_VAL); + cpl_ret = rte_dmadev_completed_status(test_dev_id, 0, 1, &last_idx, + status); + RTE_TEST_ASSERT_EQUAL(cpl_ret, 1, "Failed to completed status"); + RTE_TEST_ASSERT_EQUAL(last_idx, 0, "Last idx should be zero, %u", + last_idx); + for (i = 0; i < RTE_DIM(status); i++) + RTE_TEST_ASSERT_EQUAL(status[i], 0, + "Failed to completed status, %d", status[i]); + + /* Check do completed status again */ + cpl_ret = rte_dmadev_completed_status(test_dev_id, 0, 1, &last_idx, + status); + RTE_TEST_ASSERT_EQUAL(cpl_ret, 0, "Failed to completed status"); + + /* Check for enqueue with submit again */ + ret = rte_dmadev_copy(test_dev_id, 0, (rte_iova_t)src, (rte_iova_t)dst, + TEST_MEMCPY_SIZE, RTE_DMA_OP_FLAG_SUBMIT); + RTE_TEST_ASSERT_EQUAL(ret, 1, "Failed to enqueue copy, %d", ret); + rte_delay_us_sleep(TEST_WAIT_US_VAL); + cpl_ret = rte_dmadev_completed_status(test_dev_id, 0, 1, &last_idx, + status); + RTE_TEST_ASSERT_EQUAL(cpl_ret, 1, "Failed to completed status"); + RTE_TEST_ASSERT_EQUAL(last_idx, 1, "Last idx should be 1, %u", + last_idx); + for (i = 0; i < RTE_DIM(status); i++) + RTE_TEST_ASSERT_EQUAL(status[i], 0, + "Failed to completed status, %d", status[i]); + + /* Stop dmadev to make sure dmadev to a known state */ + ret = rte_dmadev_stop(test_dev_id); + RTE_TEST_ASSERT_SUCCESS(ret, "Failed to stop, %d", ret); + + return TEST_SUCCESS; +} + +int +test_dmadev_api(uint16_t dev_id) +{ + int ret = testsuite_setup(dev_id); + if (ret) { + printf("testsuite setup fail!\n"); + return -1; + } + + /* If the testcase exit successfully, ensure that the test dmadev exist + * and the dmadev is in the stopped state. + */ + SKELDMA_TEST_RUN(test_dmadev_get_dev_id); + SKELDMA_TEST_RUN(test_dmadev_is_valid_dev); + SKELDMA_TEST_RUN(test_dmadev_count); + SKELDMA_TEST_RUN(test_dmadev_info_get); + SKELDMA_TEST_RUN(test_dmadev_configure); + SKELDMA_TEST_RUN(test_dmadev_vchan_setup); + SKELDMA_TEST_RUN(test_dmadev_start_stop); + SKELDMA_TEST_RUN(test_dmadev_stats); + SKELDMA_TEST_RUN(test_dmadev_dump); + SKELDMA_TEST_RUN(test_dmadev_completed); + SKELDMA_TEST_RUN(test_dmadev_completed_status); + + testsuite_teardown(); + + printf("Total tests : %d\n", total); + printf("Passed : %d\n", passed); + printf("Failed : %d\n", failed); + + if (failed) + return -1; + + return 0; +};