test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts]  [PATCH V2]tests/shutdown_api: fix can't create vf
@ 2020-03-20  8:37 Zeng Xiaoxiao
  2020-03-20  9:02 ` Zeng, XiaoxiaoX
  2020-03-24  6:51 ` Tu, Lijuan
  0 siblings, 2 replies; 4+ messages in thread
From: Zeng Xiaoxiao @ 2020-03-20  8:37 UTC (permalink / raw)
  To: dts; +Cc: Zeng Xiaoxiao

Signed-off-by: Zeng Xiaoxiao <xiaoxiaox.zeng@intel.com>
---
 tests/TestSuite_shutdown_api.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_shutdown_api.py b/tests/TestSuite_shutdown_api.py
index 11d451b..8699e79 100644
--- a/tests/TestSuite_shutdown_api.py
+++ b/tests/TestSuite_shutdown_api.py
@@ -260,6 +260,7 @@ class TestShutdownApi(TestCase):
             self.vf_assign_method = 'vfio-pci'
             self.dut.send_expect('modprobe vfio-pci', '#')
 
+        self.dut.setup_modules_linux(self.target, 'igb_uio', '')
         self.bind_nic_driver(self.dut_ports[:1], driver="igb_uio")
         self.used_dut_port = self.dut_ports[0]
         tester_port = self.tester.get_local_port(self.used_dut_port)
@@ -296,6 +297,7 @@ class TestShutdownApi(TestCase):
 
         self.vm_env_done = True
         self.dut_testpmd.quit()
+        self.bind_nic_driver(self.dut_ports[:1], driver=self.vf_driver)
 
     def destroy_vm_env(self):
         if not self.vm_env_done:
@@ -539,6 +541,7 @@ class TestShutdownApi(TestCase):
         self.dut_testpmd.quit()
         time.sleep(3)
         self.vm0_dut.kill_all()
+        self.destroy_vm_env()
 
     def test_enable_disablejumbo(self):
         """
@@ -764,5 +767,6 @@ class TestShutdownApi(TestCase):
         """
         Run after each test suite.
         """
+        if self.vm_env_done:
+            self.destroy_vm_env()
         self.dut.kill_all()
-        self.destroy_vm_env()
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dts] [PATCH V2]tests/shutdown_api: fix can't create vf
  2020-03-20  8:37 [dts] [PATCH V2]tests/shutdown_api: fix can't create vf Zeng Xiaoxiao
@ 2020-03-20  9:02 ` Zeng, XiaoxiaoX
  2020-03-24  6:51 ` Tu, Lijuan
  1 sibling, 0 replies; 4+ messages in thread
From: Zeng, XiaoxiaoX @ 2020-03-20  9:02 UTC (permalink / raw)
  To: dts

[-- Attachment #1: Type: text/plain, Size: 1952 bytes --]


Testd-By: Zeng, XiaoxiaoX <xiaoxiaox.zeng@intel.com>


Best regards,
Zeng,xiaoxiao


> -----Original Message-----
> From: Zeng, XiaoxiaoX
> Sent: Friday, March 20, 2020 4:38 PM
> To: dts@dpdk.org
> Cc: Zeng, XiaoxiaoX <xiaoxiaox.zeng@intel.com>
> Subject: [dts] [PATCH V2]tests/shutdown_api: fix can't create vf
> 
> Signed-off-by: Zeng Xiaoxiao <xiaoxiaox.zeng@intel.com>
> ---
>  tests/TestSuite_shutdown_api.py | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/TestSuite_shutdown_api.py
> b/tests/TestSuite_shutdown_api.py index 11d451b..8699e79 100644
> --- a/tests/TestSuite_shutdown_api.py
> +++ b/tests/TestSuite_shutdown_api.py
> @@ -260,6 +260,7 @@ class TestShutdownApi(TestCase):
>              self.vf_assign_method = 'vfio-pci'
>              self.dut.send_expect('modprobe vfio-pci', '#')
> 
> +        self.dut.setup_modules_linux(self.target, 'igb_uio', '')
>          self.bind_nic_driver(self.dut_ports[:1], driver="igb_uio")
>          self.used_dut_port = self.dut_ports[0]
>          tester_port = self.tester.get_local_port(self.used_dut_port)
> @@ -296,6 +297,7 @@ class TestShutdownApi(TestCase):
> 
>          self.vm_env_done = True
>          self.dut_testpmd.quit()
> +        self.bind_nic_driver(self.dut_ports[:1], driver=self.vf_driver)
> 
>      def destroy_vm_env(self):
>          if not self.vm_env_done:
> @@ -539,6 +541,7 @@ class TestShutdownApi(TestCase):
>          self.dut_testpmd.quit()
>          time.sleep(3)
>          self.vm0_dut.kill_all()
> +        self.destroy_vm_env()
> 
>      def test_enable_disablejumbo(self):
>          """
> @@ -764,5 +767,6 @@ class TestShutdownApi(TestCase):
>          """
>          Run after each test suite.
>          """
> +        if self.vm_env_done:
> +            self.destroy_vm_env()
>          self.dut.kill_all()
> -        self.destroy_vm_env()
> --
> 1.8.3.1


[-- Attachment #2: TestShutdownApi.log --]
[-- Type: application/octet-stream, Size: 27170 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dts] [PATCH V2]tests/shutdown_api: fix can't create vf
  2020-03-20  8:37 [dts] [PATCH V2]tests/shutdown_api: fix can't create vf Zeng Xiaoxiao
  2020-03-20  9:02 ` Zeng, XiaoxiaoX
@ 2020-03-24  6:51 ` Tu, Lijuan
  1 sibling, 0 replies; 4+ messages in thread
From: Tu, Lijuan @ 2020-03-24  6:51 UTC (permalink / raw)
  To: Zeng, XiaoxiaoX, dts; +Cc: Zeng, XiaoxiaoX

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Zeng Xiaoxiao
> Sent: Friday, March 20, 2020 4:38 PM
> To: dts@dpdk.org
> Cc: Zeng, XiaoxiaoX <xiaoxiaox.zeng@intel.com>
> Subject: [dts] [PATCH V2]tests/shutdown_api: fix can't create vf
> 
> Signed-off-by: Zeng Xiaoxiao <xiaoxiaox.zeng@intel.com>
> ---
>  tests/TestSuite_shutdown_api.py | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/TestSuite_shutdown_api.py
> b/tests/TestSuite_shutdown_api.py index 11d451b..8699e79 100644
> --- a/tests/TestSuite_shutdown_api.py
> +++ b/tests/TestSuite_shutdown_api.py
> @@ -260,6 +260,7 @@ class TestShutdownApi(TestCase):
>              self.vf_assign_method = 'vfio-pci'
>              self.dut.send_expect('modprobe vfio-pci', '#')
> 
> +        self.dut.setup_modules_linux(self.target, 'igb_uio', '')
>          self.bind_nic_driver(self.dut_ports[:1], driver="igb_uio")
>          self.used_dut_port = self.dut_ports[0]
>          tester_port = self.tester.get_local_port(self.used_dut_port)
> @@ -296,6 +297,7 @@ class TestShutdownApi(TestCase):
> 
>          self.vm_env_done = True
>          self.dut_testpmd.quit()
> +        self.bind_nic_driver(self.dut_ports[:1], driver=self.vf_driver)
> 
>      def destroy_vm_env(self):
>          if not self.vm_env_done:
> @@ -539,6 +541,7 @@ class TestShutdownApi(TestCase):
>          self.dut_testpmd.quit()
>          time.sleep(3)
>          self.vm0_dut.kill_all()
> +        self.destroy_vm_env()
> 
>      def test_enable_disablejumbo(self):
>          """
> @@ -764,5 +767,6 @@ class TestShutdownApi(TestCase):
>          """
>          Run after each test suite.
>          """
> +        if self.vm_env_done:
> +            self.destroy_vm_env()
>          self.dut.kill_all()
> -        self.destroy_vm_env()
> --
> 1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [dts]  [PATCH V2]tests/shutdown_api: fix can't create vf
@ 2020-03-20  8:37 Zeng Xiaoxiao
  0 siblings, 0 replies; 4+ messages in thread
From: Zeng Xiaoxiao @ 2020-03-20  8:37 UTC (permalink / raw)
  To: dts; +Cc: Zeng Xiaoxiao

Signed-off-by: Zeng Xiaoxiao <xiaoxiaox.zeng@intel.com>
---
 tests/TestSuite_shutdown_api.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/tests/TestSuite_shutdown_api.py b/tests/TestSuite_shutdown_api.py
index 11d451b..8699e79 100644
--- a/tests/TestSuite_shutdown_api.py
+++ b/tests/TestSuite_shutdown_api.py
@@ -260,6 +260,7 @@ class TestShutdownApi(TestCase):
             self.vf_assign_method = 'vfio-pci'
             self.dut.send_expect('modprobe vfio-pci', '#')
 
+        self.dut.setup_modules_linux(self.target, 'igb_uio', '')
         self.bind_nic_driver(self.dut_ports[:1], driver="igb_uio")
         self.used_dut_port = self.dut_ports[0]
         tester_port = self.tester.get_local_port(self.used_dut_port)
@@ -296,6 +297,7 @@ class TestShutdownApi(TestCase):
 
         self.vm_env_done = True
         self.dut_testpmd.quit()
+        self.bind_nic_driver(self.dut_ports[:1], driver=self.vf_driver)
 
     def destroy_vm_env(self):
         if not self.vm_env_done:
@@ -539,6 +541,7 @@ class TestShutdownApi(TestCase):
         self.dut_testpmd.quit()
         time.sleep(3)
         self.vm0_dut.kill_all()
+        self.destroy_vm_env()
 
     def test_enable_disablejumbo(self):
         """
@@ -764,5 +767,6 @@ class TestShutdownApi(TestCase):
         """
         Run after each test suite.
         """
+        if self.vm_env_done:
+            self.destroy_vm_env()
         self.dut.kill_all()
-        self.destroy_vm_env()
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-03-24  6:51 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-20  8:37 [dts] [PATCH V2]tests/shutdown_api: fix can't create vf Zeng Xiaoxiao
2020-03-20  9:02 ` Zeng, XiaoxiaoX
2020-03-24  6:51 ` Tu, Lijuan
  -- strict thread matches above, loose matches on Subject: below --
2020-03-20  8:37 Zeng Xiaoxiao

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).