Tested-by: xizhan4x -----Original Message----- From: Zhang, XiX Sent: Friday, July 3, 2020 11:08 AM To: dts@dpdk.org Cc: Zhang, XiX Subject: [dts][PATCH V1] tests/TestSuite_shutdown_api:Modify the execution order of shutting down the VM environment Modify the execution order of shutting down the VM environment, The correct step should be to shut down the VM environment before unbind Vf Signed-off-by: xizhan4x --- tests/TestSuite_shutdown_api.py | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/tests/TestSuite_shutdown_api.py b/tests/TestSuite_shutdown_api.py index 650ccfc..9fc3e3a 100644 --- a/tests/TestSuite_shutdown_api.py +++ b/tests/TestSuite_shutdown_api.py @@ -296,6 +296,16 @@ class TestShutdownApi(TestCase): def destroy_vm_env(self): + if getattr(self, 'self.vm0_testpmd', None): + self.vm0_testpmd.quit() + + if getattr(self, 'vm0', None): + if getattr(self, 'vm0_dut', None): + self.vm0_dut.kill_all() + self.vm0_testpmd = None + self.vm0_dut_ports = None + # destroy vm0 + self.vm0.stop() if getattr(self, 'used_dut_port', None) is not None: self.dut.destroy_sriov_vfs_by_port(self.used_dut_port) self.used_dut_port = None @@ -308,18 +318,11 @@ class TestShutdownApi(TestCase): if not self.vm_env_done: return - if getattr(self, 'self.vm0_testpmd', None): - self.vm0_testpmd.quit() - - if getattr(self, 'vm0', None): - if getattr(self, 'vm0_dut', None): - self.vm0_dut.kill_all() - self.vm0_testpmd = None - self.vm0_dut_ports = None - # destroy vm0 - self.vm0.stop() self.vm_env_done = False + + + def test_stop_restart(self): """ Stop and Restar. -- 1.8.3.1