[dpdk-dev] [PATCH v2 10/11] test: register eventdev selftest

Van Haaren, Harry harry.van.haaren at intel.com
Tue Dec 19 16:27:25 CET 2017


> From: Pavan Nikhilesh [mailto:pbhagavatula at caviumnetworks.com]
> Sent: Thursday, December 14, 2017 3:02 PM
> To: jerin.jacob at caviumnetworks.com; santosh.shukla at caviumnetworks.com;
> Richardson, Bruce <bruce.richardson at intel.com>; Van Haaren, Harry
> <harry.van.haaren at intel.com>; Eads, Gage <gage.eads at intel.com>;
> hemant.agrawal at nxp.com; nipun.gupta at nxp.com; Ma, Liang J
> <liang.j.ma at intel.com>

[Side note: we shouldn't put everybody on CC all the time..]

> Cc: dev at dpdk.org; Pavan Nikhilesh <pbhagavatula at caviumnetworks.com>
> Subject: [dpdk-dev] [PATCH v2 10/11] test: register eventdev selftest
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula at caviumnetworks.com>


> diff --git a/test/test/test_eventdev.c b/test/test/test_eventdev.c
> index 1ed2a1ddd..b86efab01 100644
> --- a/test/test/test_eventdev.c
> +++ b/test/test/test_eventdev.c
> @@ -1009,4 +1009,11 @@ test_eventdev_common(void)
>  	return unit_test_suite_runner(&eventdev_common_testsuite);
>  }
> 
> +static int
> +test_eventdev_selftest(void)
> +{
> +	return rte_event_dev_selftest(TEST_DEV_ID);
> +}
> +
>  REGISTER_TEST_COMMAND(eventdev_common_autotest, test_eventdev_common);
> +REGISTER_TEST_COMMAND(eventdev_selftest, test_eventdev_selftest);


Currently when running the test app, we don't pass any arguments. Running the "eventdev_sw_autotest" command, it will create the required event_sw0 PMD vdev, and launch the tests then.

Given the selftest is PMD agnostic, does it makes sense to have a single string "sw" or "octeontx" to run the tests against? Right now it requires that we pass ./app/test --vdev event_sw0  which I think is a burden, particularly when automating this with the meson test infrastructure down the line.

Summary; Please add a string parameter that indicates the PMD to run the self-test on.



More information about the dev mailing list