[v2,2/5] test: add quick run tests under test-fast suite
Checks
Commit Message
From: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
Added test cases that runs quickly under test fast category
Signed-off-by: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
---
test/test/meson.build | 102 +++++++++++++++++++++++++++++++++++++++++++++++---
1 file changed, 97 insertions(+), 5 deletions(-)
Comments
> -----Original Message-----
> From: Vemula, Hari KumarX
> Sent: Friday, October 12, 2018 3:33 PM
> To: dev@dpdk.org
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; Pattan, Reshma
> <reshma.pattan@intel.com>; Vemula, Hari KumarX
> <hari.kumarx.vemula@intel.com>
> Subject: [PATCH v2 2/5] test: add quick run tests under test-fast suite
>
> From: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
>
>
> + test(arg, dpdk_test,
> + env : ['DPDK_TEST=' + arg],
> + args : ['-c f','-n 4', '--file-prefix=test@0@'.format(itr)],
Can we use test name instead of itr here? And remove itr?
Thanks,
Reshma
> -----Original Message-----
> From: Vemula, Hari KumarX
> Sent: Friday, October 12, 2018 3:33 PM
> To: dev@dpdk.org
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; Pattan, Reshma
> <reshma.pattan@intel.com>; Vemula, Hari KumarX
> <hari.kumarx.vemula@intel.com>
> Subject: [PATCH v2 2/5] test: add quick run tests under test-fast suite
>
> +
> + itr = 0
> + foreach arg : fast_parallel_test_names
> + test(arg, dpdk_test,
> + env : ['DPDK_TEST=' + arg],
> + args : ['-c f','-n 4', '--file-prefix=test@0@'.format(itr)],
i.e. '--file-prefix=@0@'.format(arg)], ?
> -----Original Message-----
> From: Vemula, Hari KumarX
> Sent: Friday, October 12, 2018 3:33 PM
> To: dev@dpdk.org
> Cc: Richardson, Bruce <bruce.richardson@intel.com>; Pattan, Reshma
> <reshma.pattan@intel.com>; Vemula, Hari KumarX
> <hari.kumarx.vemula@intel.com>
> Subject: [PATCH v2 2/5] test: add quick run tests under test-fast suite
>
> From: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
> + is_parallel : true,
Meson by default run the UTs in parallel by default, so u can remove the above line.
Thanks,
Reshma
@@ -122,6 +122,75 @@ test_deps = ['acl',
'timer'
]
+#All test cases in fast_parallel_test_names list are parallel
+fast_parallel_test_names =[
+ 'acl_autotest',
+ 'alarm_autotest',
+ 'atomic_autotest',
+ 'byteorder_autotest',
+ 'cmdline_autotest',
+ 'common_autotest',
+ 'cpuflags_autotest',
+ 'cycles_autotest',
+ 'debug_autotest',
+ 'eal_flags_autotest',
+ 'eal_fs_autotest',
+ 'errno_autotest',
+ 'event_ring_autotest',
+ 'func_reentrancy_autotest',
+ 'flow_classify_autotest',
+ 'hash_autotest',
+ 'interrupt_autotest',
+ 'logs_autotest',
+ 'lpm6_autotest',
+ 'lpm_autotest',
+ 'malloc_autotest',
+ 'mbuf_autotest',
+ 'memcpy_autotest',
+ 'memory_autotest',
+ 'mempool_autotest',
+ 'memzone_autotest',
+ 'meter_autotest',
+ 'multiprocess_autotest',
+ 'per_lcore_autotest',
+ 'prefetch_autotest',
+ 'red_autotest',
+ 'ring_autotest',
+ 'ring_pmd_autotest',
+ 'rwlock_autotest',
+ 'sched_autotest',
+ 'spinlock_autotest',
+ 'string_autotest',
+ 'table_autotest',
+ 'tailq_autotest',
+ 'timer_autotest',
+ 'user_delay_us',
+ 'version_autotest',
+]
+
+#All test cases in fast_non_parallel_test_names list are non-parallel
+fast_non_parallel_test_names =[
+ 'cryptodev_sw_armv8_autotest',
+ 'crc_autotest',
+ 'cryptodev_openssl_asym_autotest',
+ 'cryptodev_sw_mvsam_autotest',
+ 'devargs_autotest',
+ 'distributor_autotest',
+ 'eventdev_common_autotest',
+ 'eventdev_octeontx_autotest',
+ 'eventdev_sw_autotest',
+ 'hash_scaling_autotest',
+ 'kni_autotest',
+ 'kvargs_autotest',
+ 'member_autotest',
+ 'power_acpi_cpufreq_autotest',
+ 'power_autotest',
+ 'power_kvm_vm_autotest',
+ 'reorder_autotest',
+ 'service_autotest',
+ 'thash_autotest',
+]
+
if dpdk_conf.has('RTE_LIBRTE_PDUMP')
test_deps += 'pdump'
endif
@@ -150,11 +219,14 @@ if cc.has_argument('-Wno-format-truncation')
endif
test_dep_objs = []
-compress_test_dep = dependency('zlib', required: false)
-if compress_test_dep.found()
- test_dep_objs += compress_test_dep
- test_sources += 'test_compressdev.c'
- test_deps += 'compressdev'
+if dpdk_conf.has('RTE_LIBRTE_COMPRESSDEV')
+ compress_test_dep = dependency('zlib', required: false)
+ if compress_test_dep.found()
+ test_dep_objs += compress_test_dep
+ test_sources += 'test_compressdev.c'
+ test_deps += 'compressdev'
+ fast_non_parallel_test_names += 'compressdev_autotest'
+ endif
endif
foreach d:test_deps
@@ -180,5 +252,25 @@ if get_option('tests')
# some perf tests (eg: memcpy perf autotest)take very long
# to complete, so timeout to 10 minutes
timeout_seconds = 600
+ timeout_seconds_fast = 10
+
+ itr = 0
+ foreach arg : fast_parallel_test_names
+ test(arg, dpdk_test,
+ env : ['DPDK_TEST=' + arg],
+ args : ['-c f','-n 4', '--file-prefix=test@0@'.format(itr)],
+ timeout : timeout_seconds_fast,
+ is_parallel : true,
+ suite : 'test-fast')
+ itr += 1
+ endforeach
+
+ foreach arg : fast_non_parallel_test_names
+ test(arg, dpdk_test,
+ env : ['DPDK_TEST=' + arg],
+ timeout : timeout_seconds_fast,
+ is_parallel : false,
+ suite : 'test-fast')
+ endforeach
endif