summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorBruce Richardson <bruce.richardson@intel.com>2018-10-12 16:34:04 +0100
committerThomas Monjalon <thomas@monjalon.net>2018-11-06 03:20:26 +0100
commita5563b85f6fab9cf81d27827e0146f621127047a (patch)
tree699fdf7bef09f2351a93d3b173d41089662d58d1 /test
parent9af4eb5657107b05b0e06141291b7c6e6162bb36 (diff)
downloaddpdk-a5563b85f6fab9cf81d27827e0146f621127047a.zip
dpdk-a5563b85f6fab9cf81d27827e0146f621127047a.tar.gz
dpdk-a5563b85f6fab9cf81d27827e0146f621127047a.tar.xz
test: allow taking extra arguments from environment
When running unit tests automatically, either via script, from meson, or otherwise, the same set of options may be used for each run, for example to set a standard coremask to be used for all tests. To facilitate this, this patch adds support for the test binary taking additional EAL parameters from the environment and appending them to the argc/argv list passed to eal init. This allows parameter modification without having to edit test scripts etc. There are now two environment variables which can be used for running tests: * DPDK_TEST - (added previously) passes the test name to be run automatically rather than running the app interactively. Used by "meson test" when running tests individually or as part of a suite. * DPDK_TEST_PARAMS - new parameter to specify the commandline arguments to use with the test binary. For example to run a test, or tests, on only 16 lcores, and to skip pci scan we can set this to "-l 0-15 --no-pci". Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> Tested-by: Luca Boccassi <bluca@debian.org>
Diffstat (limited to 'test')
-rw-r--r--test/test/test.c34
1 files changed, 33 insertions, 1 deletions
diff --git a/test/test/test.c b/test/test/test.c
index ffa9c36..24df629 100644
--- a/test/test/test.c
+++ b/test/test/test.c
@@ -75,15 +75,47 @@ do_recursive_call(void)
int last_test_result;
+#define MAX_EXTRA_ARGS 32
+
int
main(int argc, char **argv)
{
#ifdef RTE_LIBRTE_CMDLINE
struct cmdline *cl;
#endif
+ char *extra_args;
int ret;
- ret = rte_eal_init(argc, argv);
+ extra_args = getenv("DPDK_TEST_PARAMS");
+ if (extra_args != NULL && strlen(extra_args) > 0) {
+ char **all_argv;
+ char *eargv[MAX_EXTRA_ARGS];
+ int all_argc;
+ int eargc;
+ int i;
+
+ RTE_LOG(INFO, APP, "Using additional DPDK_TEST_PARAMS: '%s'\n",
+ extra_args);
+ eargc = rte_strsplit(extra_args, strlen(extra_args),
+ eargv, MAX_EXTRA_ARGS, ' ');
+
+ /* merge argc/argv and the environment args */
+ all_argc = argc + eargc;
+ all_argv = malloc(sizeof(*all_argv) * (all_argc + 1));
+ if (all_argv == NULL)
+ return -1;
+
+ for (i = 0; i < argc; i++)
+ all_argv[i] = argv[i];
+ for (i = 0; i < eargc; i++)
+ all_argv[argc + i] = eargv[i];
+ all_argv[all_argc] = NULL;
+
+ /* call eal_init with combined args */
+ ret = rte_eal_init(all_argc, all_argv);
+ free(all_argv);
+ } else
+ ret = rte_eal_init(argc, argv);
if (ret < 0) {
ret = -1;
goto out;