netdev - Re: ynl ethtool subscribe issue

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAD4GDZzQc1M+b_um-VH1zfq5RSMNwoWNp+8yT=D-e4_pHSpBjQ@mail.gmail.com>
Date: Thu, 17 Oct 2024 23:05:14 +0100
From: Donald Hunter <donald.hunter@...il.com>
To: Kory Maincent <kory.maincent@...tlin.com>
Cc: netdev@...r.kernel.org, kuba@...nel.org, 
	Maxime Chevallier <maxime.chevallier@...tlin.com>
Subject: Re: ynl ethtool subscribe issue

On Thu, 17 Oct 2024 at 17:05, Kory Maincent <kory.maincent@...tlin.com> wrote:
>
> Hello,
>
> It seems there is an issue with ynl tool and ethtool specs:
>
> # ./ynl/cli.py --spec netlink/specs/ethtool.yaml --no-schema --subscribe monitor
>  --sleep 10 &
> # ethtool -K eth0 rx-gro-list on
> # Traceback (most recent call last):
>   File "/root/ynl/lib/ynl.py", line 723, in _decode
>     attr_spec = attr_space.attrs_by_val[attr.type]
>                 ~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^
> KeyError: 4
>
> During handling of the above exception, another exception occurred:
>
> Traceback (most recent call last):
>   File "/root/./ynl/cli.py", line 114, in <module>
>     main()
>   File "/root/./ynl/cli.py", line 109, in main
>     ynl.check_ntf()
>   File "/root/ynl/lib/ynl.py", line 930, in check_ntf
>     self.handle_ntf(decoded)
>   File "/root/ynl/lib/ynl.py", line 898, in handle_ntf
>     attrs = self._decode(decoded.raw_attrs, op.attr_set.name)
>             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>   File "/root/ynl/lib/ynl.py", line 732, in _decode
>     subdict = self._decode(NlAttrs(attr.raw), attr_spec['nested-attributes'],
> search_attrs)
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> File "/root/ynl/lib/ynl.py", line 726, in _decode raise Exception(f"Space
> '{space}' has no attribute with value '{attr.type}'") Exception: Space 'bitset'
> has no attribute with value '4'

The 'bitset' attribute-set is missing these attributes which are
defined in include/uapi/linux/ethtool_netlink.h

ETHTOOL_A_BITSET_VALUE, /* binary */
ETHTOOL_A_BITSET_MASK, /* binary */

This patch fixes it:

diff --git a/Documentation/netlink/specs/ethtool.yaml
b/Documentation/netlink/specs/ethtool.yaml
index 6a050d755b9c..f6c5d8214c7e 100644
--- a/Documentation/netlink/specs/ethtool.yaml
+++ b/Documentation/netlink/specs/ethtool.yaml
@@ -96,7 +96,12 @@ attribute-sets:
         name: bits
         type: nest
         nested-attributes: bitset-bits
-
+      -
+        name: value
+        type: binary
+      -
+        name: mask
+        type: binary
   -
     name: string
     attributes:

./tools/net/ynl/cli.py --spec Documentation/netlink/specs/ethtool.yaml
--subscribe monitor --sleep 10
[{'msg': {'active': {'nomask': True,
                     'size': 64,
                     'value': b'\xb2A\x00\x00\x00\x01\x00\x00'},
          'header': {'dev-index': 2, 'dev-name': 'enp42s0'},
          'hw': {'mask': b'\xff\xff\xff\xff\xff\xff\xff\xff',
                 'size': 64,
                 'value': b'\x93I\x19\x00\x00\x1b\x00\n'},
          'nochange': {'nomask': True,
                       'size': 64,
                       'value': b'\x004\x00\x00\x00\x00\x00\x00'},
          'wanted': {'nomask': True,
                     'size': 64,
                     'value': b'\x92I\x00\x00\x00\x01\x00\x00'}},
  'name': 'features-ntf'}]

> I used set_features netlink command but all of ethtool commands are not working.
> I am not a ynl expert so someone may find the issue before I do.
> That would be kind!

I haven't tried anything else from the ethtool spec, but plan to check
it out tomorrow.

I can submit a patch for this tomorrow, unless you want to?

Cheers,
Donald.