diff mbox series

conf: Added skips to TestVirtioUserInterrupt

Message ID 20220110171941.23183-1-ohilyard@iol.unh.edu (mailing list archive)
State Accepted
Headers
Series conf: Added skips to TestVirtioUserInterrupt |

Checks

Context Check Description
ci/Intel-dts-suite-test warning SKIPPED

Commit Message

Owen Hilyard Jan. 10, 2022, 5:19 p.m. UTC 
  From: Owen Hilyard <ohilyard@iol.unh.edu>

Some tests in TestVirtioUserInterrupt only support the igb_uio.
Previously they emitted a failure for the test when it was run with the
wrong driver. This update will make the test cases be skipped if a
supported driver is not used.

Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>
---
 conf/test_case_supportlist.json | 91 ++++++++++++++++++++++++++-------
 1 file changed, 73 insertions(+), 18 deletions(-)

Comments

Tu, Lijuan Jan. 25, 2022, 6:27 a.m. UTC | #1 
> -----Original Message-----
> From: ohilyard@iol.unh.edu <ohilyard@iol.unh.edu>
> Sent: 2022年1月11日 1:20
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; lylavoie@iol.unh.edu; Owen Hilyard
> <ohilyard@iol.unh.edu>
> Subject: [PATCH] conf: Added skips to TestVirtioUserInterrupt
> 
> From: Owen Hilyard <ohilyard@iol.unh.edu>
> 
> Some tests in TestVirtioUserInterrupt only support the igb_uio.
> Previously they emitted a failure for the test when it was run with the wrong
> driver. This update will make the test cases be skipped if a supported driver is not
> used.
> 
> Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>
> ---
>  conf/test_case_supportlist.json | 91 ++++++++++++++++++++++++++-------
>  1 file changed, 73 insertions(+), 18 deletions(-)
> 
> diff --git a/conf/test_case_supportlist.json b/conf/test_case_supportlist.json
> index e3eefd40..4206356a 100644
> --- a/conf/test_case_supportlist.json
> +++ b/conf/test_case_supportlist.json
> @@ -1826,23 +1826,6 @@
>              "Comments": "This case currently support for x550/x540"
>          }
>      ],
> -    "perf_kernel_pf_dpdk_iavf_perf_host_only": [
> -        {
> -            "OS": [
> -                "ALL"
> -            ],
> -            "NIC": [
> -                "fortville_eagle",
> -                "fortville_spirit",
> -                "fortville_25g"
> -            ],
> -            "Target": [
> -                "ALL"
> -            ],
> -            "Bug ID": "",
> -            "Comments": "this case only support on fortville"
> -        }
> -    ],

I am confused why removes perf_kernel_pf_dpdk_iavf_perf_host_only here.
Owen Hilyard Jan. 25, 2022, 8:44 p.m. UTC | #2 
There were duplicate entries in the list. Sorry, I should have clarified
that.
Tu, Lijuan Jan. 26, 2022, 5:51 a.m. UTC | #3 
> -----Original Message-----
> From: ohilyard@iol.unh.edu <ohilyard@iol.unh.edu>
> Sent: 2022年1月11日 1:20
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; lylavoie@iol.unh.edu; Owen Hilyard
> <ohilyard@iol.unh.edu>
> Subject: [PATCH] conf: Added skips to TestVirtioUserInterrupt
> 
> From: Owen Hilyard <ohilyard@iol.unh.edu>
> 
> Some tests in TestVirtioUserInterrupt only support the igb_uio.
> Previously they emitted a failure for the test when it was run with the wrong
> driver. This update will make the test cases be skipped if a supported driver is not
> used.
> 
> Signed-off-by: Owen Hilyard <ohilyard@iol.unh.edu>

Applied, thanks
diff mbox series

Patch

diff --git a/conf/test_case_supportlist.json b/conf/test_case_supportlist.json
index e3eefd40..4206356a 100644
--- a/conf/test_case_supportlist.json
+++ b/conf/test_case_supportlist.json
@@ -1826,23 +1826,6 @@ 
             "Comments": "This case currently support for x550/x540"
         }
     ],
-    "perf_kernel_pf_dpdk_iavf_perf_host_only": [
-        {
-            "OS": [
-                "ALL"
-            ],
-            "NIC": [
-                "fortville_eagle",
-                "fortville_spirit",
-                "fortville_25g"
-            ],
-            "Target": [
-                "ALL"
-            ],
-            "Bug ID": "",
-            "Comments": "this case only support on fortville"
-        }
-    ],
     "fdir_L2_mac_filter_basic_for_ipv4_udp": [
         {
             "OS": [
@@ -2297,5 +2280,77 @@ 
             "Bug ID": "",
             "Comments": "This case only support ice"
         }
-    ] 
+    ],
+    "lsc_event_between_vhost_user_and_virtio_user_with_packed_ring_and_cbdma_enabled": [
+        {
+            "OS": [
+                "ALL"
+            ],
+            "NIC": [
+                "ALL"
+            ],
+            "Target": [
+                "ALL"
+            ],
+            "host_driver": [
+                "igb_uio"
+            ],
+            "Bug ID": "",
+            "Comments": "This case only supports the igb_uio driver"
+        }
+    ],
+    "lsc_event_between_vhost_user_and_virtio_user_with_split_ring_and_cbdma_enabled": [
+        {
+            "OS": [
+                "ALL"
+            ],
+            "NIC": [
+                "ALL"
+            ],
+            "Target": [
+                "ALL"
+            ],
+            "host_driver": [
+                "igb_uio"
+            ],
+            "Bug ID": "",
+            "Comments": "This case only supports the igb_uio driver"
+        }
+    ],
+    "packed_ring_virtio_user_interrupt_test_with_vhost_user_as_backend_and_cbdma_enabled": [
+        {
+            "OS": [
+                "ALL"
+            ],
+            "NIC": [
+                "ALL"
+            ],
+            "Target": [
+                "ALL"
+            ],
+            "host_driver": [
+                "igb_uio"
+            ],
+            "Bug ID": "",
+            "Comments": "This case only supports the igb_uio driver"
+        }
+    ],
+    "split_ring_virtio_user_interrupt_test_with_vhost_user_as_backend_and_cbdma_enabled": [
+        {
+            "OS": [
+                "ALL"
+            ],
+            "NIC": [
+                "ALL"
+            ],
+            "Target": [
+                "ALL"
+            ],
+            "host_driver": [
+                "igb_uio"
+            ],
+            "Bug ID": "",
+            "Comments": "This case only supports the igb_uio driver"
+        }
+    ]
 }