diff mbox series

[v4,5/5] test: add dump test cases under test-dump suite

Message ID 1540372032-28642-6-git-send-email-hari.kumarx.vemula@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series None |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Hari Kumar Vemula Oct. 24, 2018, 9:07 a.m. UTC 
  From: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>

Grouped logging or dump related test cases to test-dump category

Signed-off-by: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
---
 test/test/meson.build | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

Comments

Bruce Richardson Oct. 24, 2018, 10:16 a.m. UTC | #1 
On Wed, Oct 24, 2018 at 10:07:12AM +0100, Hari kumar Vemula wrote:
> From: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
> 
> Grouped logging or dump related test cases to test-dump category
> 
> Signed-off-by: Hari Kumar Vemula <hari.kumarx.vemula@intel.com>
> ---
>  test/test/meson.build | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/test/test/meson.build b/test/test/meson.build
> index da1dbfac4..266e1bfa6 100644
> --- a/test/test/meson.build
> +++ b/test/test/meson.build
> @@ -236,6 +236,20 @@ driver_test_names = [
>          'cryptodev_sw_zuc_autotest',
>  ]
>  
> +#All test cases in dump_test_names list are non-parallel
> +dump_test_names = [
> +        'dump_struct_sizes',
> +        'dump_mempool',
> +        'dump_malloc_stats',
> +        'dump_devargs',
> +        'dump_log_types',
> +        'dump_ring',
> +        'quit_autotest',
> +        'dump_physmem',
> +        'dump_memzone',
> +        'devargs_autotest',
> +]
> +
"quit_autotest" and "devargs_autotest" don't look like they belong here.
Why are they included?
diff mbox series

Patch

diff --git a/test/test/meson.build b/test/test/meson.build
index da1dbfac4..266e1bfa6 100644
--- a/test/test/meson.build
+++ b/test/test/meson.build
@@ -236,6 +236,20 @@  driver_test_names = [
         'cryptodev_sw_zuc_autotest',
 ]
 
+#All test cases in dump_test_names list are non-parallel
+dump_test_names = [
+        'dump_struct_sizes',
+        'dump_mempool',
+        'dump_malloc_stats',
+        'dump_devargs',
+        'dump_log_types',
+        'dump_ring',
+        'quit_autotest',
+        'dump_physmem',
+        'dump_memzone',
+        'devargs_autotest',
+]
+
 if dpdk_conf.has('RTE_LIBRTE_PDUMP')
 	test_deps += 'pdump'
 endif
@@ -334,4 +348,11 @@  if get_option('tests')
                         suite : 'test-driver')
         endforeach
 
+        foreach arg : dump_test_names
+                test(arg, dpdk_test,
+                        env : ['DPDK_TEST=' + arg],
+                        timeout : timeout_seconds,
+                        is_parallel : false,
+                        suite : 'test-dump')
+        endforeach
 endif