[V1] tests/userspace_ethtool: optimization script

Message ID 20220428143924.15886-1-songx.jiale@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/userspace_ethtool: optimization script |

Checks

Context Check Description
ci/Intel-dts-format-test fail Testing issues
ci/Intel-dts-pylama-test fail Testing issues
ci/Intel-dts-suite-test success Testing OK

Commit Message

Jiale, SongX April 28, 2022, 2:39 p.m. UTC
  In some case tests, the port is bound to the kernel driver. when these cases fail, 
the port will always be bound to the kernel driver, affecting the tests of later cases.

Signed-off-by: Jiale Song <songx.jiale@intel.com>
---
 tests/TestSuite_userspace_ethtool.py | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Huang, ZhiminX April 28, 2022, 7:09 a.m. UTC | #1
> -----Original Message-----
> From: Jiale Song <songx.jiale@intel.com>
> Sent: Thursday, April 28, 2022 10:39 PM
> To: dts@dpdk.org
> Cc: Jiale, SongX <songx.jiale@intel.com>
> Subject: [dts] [PATCH V1] tests/userspace_ethtool: optimization script
> 
> In some case tests, the port is bound to the kernel driver. when these cases fail,
> the port will always be bound to the kernel driver, affecting the tests of later
> cases.
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>
> ---
Tested-by: Zhimin Huang <zhiminx.huang@intel.com >
  
Tu, Lijuan May 5, 2022, 6:12 a.m. UTC | #2
On Thu, 28 Apr 2022 14:39:24 +0000, Jiale Song <songx.jiale@intel.com> wrote:
> In some case tests, the port is bound to the kernel driver. when these cases fail, 
> the port will always be bound to the kernel driver, affecting the tests of later cases.
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>


Applied, thanks
  

Patch

diff --git a/tests/TestSuite_userspace_ethtool.py b/tests/TestSuite_userspace_ethtool.py
index 2b0269f4..c1818722 100755
--- a/tests/TestSuite_userspace_ethtool.py
+++ b/tests/TestSuite_userspace_ethtool.py
@@ -699,6 +699,7 @@  class TestUserspaceEthtool(TestCase):
         """
         Run after each test case.
         """
+        self.dut.bind_interfaces_linux(self.drivername)
         self.dut.kill_all()
         pass