diff options
| author | David Wei <dw@davidwei.uk> | 2024-05-07 19:32:28 +0300 | 
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2024-05-09 04:59:47 +0300 | 
| commit | 1cf2704242180351d156fb48c334b319ae6b0759 (patch) | |
| tree | 4b170d7487ac21ac5ae53a7308a678478b99e6a4 /tools/testing/selftests/net/lib/py/utils.py | |
| parent | 3762ec05a9fbda16aaaa2568df679ab8ad13f38d (diff) | |
| download | linux-1cf2704242180351d156fb48c334b319ae6b0759.tar.xz | |
net: selftest: add test for netdev netlink queue-get API
Add a selftest for netdev generic netlink. For now there is only a
single test that exercises the `queue-get` API.
The test works with netdevsim by default or with a real device by
setting NETIF.
Add a timeout param to cmd() since ethtool -L can take a long time on
real devices.
Signed-off-by: David Wei <dw@davidwei.uk>
Link: https://lore.kernel.org/r/20240507163228.2066817-3-dw@davidwei.uk
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/testing/selftests/net/lib/py/utils.py')
| -rw-r--r-- | tools/testing/selftests/net/lib/py/utils.py | 8 | 
1 files changed, 4 insertions, 4 deletions
| diff --git a/tools/testing/selftests/net/lib/py/utils.py b/tools/testing/selftests/net/lib/py/utils.py index ec8b086b4fcb..0540ea24921d 100644 --- a/tools/testing/selftests/net/lib/py/utils.py +++ b/tools/testing/selftests/net/lib/py/utils.py @@ -8,7 +8,7 @@ import time  class cmd: -    def __init__(self, comm, shell=True, fail=True, ns=None, background=False, host=None): +    def __init__(self, comm, shell=True, fail=True, ns=None, background=False, host=None, timeout=5):          if ns:              comm = f'ip netns exec {ns} ' + comm @@ -23,15 +23,15 @@ class cmd:              self.proc = subprocess.Popen(comm, shell=shell, stdout=subprocess.PIPE,                                           stderr=subprocess.PIPE)          if not background: -            self.process(terminate=False, fail=fail) +            self.process(terminate=False, fail=fail, timeout=timeout) -    def process(self, terminate=True, fail=None): +    def process(self, terminate=True, fail=None, timeout=5):          if fail is None:              fail = not terminate          if terminate:              self.proc.terminate() -        stdout, stderr = self.proc.communicate(timeout=5) +        stdout, stderr = self.proc.communicate(timeout)          self.stdout = stdout.decode("utf-8")          self.stderr = stderr.decode("utf-8")          self.proc.stdout.close() | 
