test: load drivers when required
authorDavid Marchand <david.marchand@redhat.com>
Mon, 23 Mar 2020 19:59:22 +0000 (20:59 +0100)
committerDavid Marchand <david.marchand@redhat.com>
Tue, 24 Mar 2020 07:37:21 +0000 (08:37 +0100)
Concatenating to test_args if event_eth_tx_adapter_autotest is executed
makes all subsequent tests inherit from the drivers loading while this
is unneeded.

Fixes: 207b1c813f39 ("test: fix build without ring PMD")
Cc: stable@dpdk.org
Signed-off-by: David Marchand <david.marchand@redhat.com>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
Acked-by: Aaron Conole <aconole@redhat.com>
app/test/meson.build

index 0a2ce71..7ab2b58 100644 (file)
@@ -403,37 +403,33 @@ timeout_seconds_fast = 10
 get_coremask = find_program('get-coremask.sh')
 num_cores_arg = '-l ' + run_command(get_coremask).stdout().strip()
 
-test_args = [num_cores_arg]
+default_test_args = [num_cores_arg]
 
 foreach arg : fast_test_names
+       test_args = default_test_args
+
        if (get_option('default_library') == 'shared' and
                arg == 'event_eth_tx_adapter_autotest')
                foreach drv:dpdk_drivers
                        test_args += ['-d', drv.full_path().split('.a')[0] + '.so']
                endforeach
        endif
-       if host_machine.system() == 'linux'
-               test(arg, dpdk_test,
-                         env : ['DPDK_TEST=' + arg],
-                         args : test_args +
-                                ['--file-prefix=@0@'.format(arg)],
-               timeout : timeout_seconds_fast,
-               is_parallel : false,
-               suite : 'fast-tests')
-       else
-               test(arg, dpdk_test,
-                       env : ['DPDK_TEST=' + arg],
-                       args : test_args,
+       if is_linux
+               test_args += ['--file-prefix=@0@'.format(arg)]
+       endif
+
+       test(arg, dpdk_test,
+               env : ['DPDK_TEST=' + arg],
+               args : test_args,
                timeout : timeout_seconds_fast,
                is_parallel : false,
                suite : 'fast-tests')
-       endif
 endforeach
 
 foreach arg : perf_test_names
        test(arg, dpdk_test,
        env : ['DPDK_TEST=' + arg],
-       args : test_args,
+       args : default_test_args,
        timeout : timeout_seconds,
        is_parallel : false,
        suite : 'perf-tests')
@@ -442,7 +438,7 @@ endforeach
 foreach arg : driver_test_names
        test(arg, dpdk_test,
                env : ['DPDK_TEST=' + arg],
-               args : test_args,
+               args : default_test_args,
                timeout : timeout_seconds,
                is_parallel : false,
                suite : 'driver-tests')
@@ -451,7 +447,7 @@ endforeach
 foreach arg : dump_test_names
        test(arg, dpdk_test,
                env : ['DPDK_TEST=' + arg],
-               args : test_args,
+               args : default_test_args,
                timeout : timeout_seconds,
                is_parallel : false,
                suite : 'debug-tests')