netdev - Re: [PATCH ethtool-next v4 00/16] Add JSON output to --module-info

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] [day] [month] [year] [list]

Message-ID: <20250206163949.370011ae@kernel.org>
Date: Thu, 6 Feb 2025 16:39:49 -0800
From: Jakub Kicinski <kuba@...nel.org>
To: Danielle Ratson <danieller@...dia.com>
Cc: <netdev@...r.kernel.org>, <mkubecek@...e.cz>, <matt@...verse.com.au>,
 <daniel.zahka@...il.com>, <amcohen@...dia.com>,
 <nbu-mlxsw@...hange.nvidia.com>
Subject: Re: [PATCH ethtool-next v4 00/16] Add JSON output to --module-info

On Wed, 5 Feb 2025 17:54:20 +0200 Danielle Ratson wrote:
> Add JSON output for 'ethtool -m' / --module-info, following the
> guideline below:
> 
> 1. Fields with description, will have a separate description field.
> 2. Units will be documented in a separate module-info.json file.
> 3. ASCII fields will be presented as strings.
> 4. On/Off is rendered as true/false.
> 5. Yes/no is rendered as true/false.
> 6. Per-channel fields will be presented as array, when each element
>    represents a channel.
> 7. Fields that hold version, will be split to major and minor sub
>    fields.
> 
> This patchset suppose to extend [1] to cover all types of modules.
> 
> Patchset overview:
> Patches #1-#7: Preparations.
> Patches #8-#9: Add JSON output support for CMIS compliant modules.
> Patches #10-#11: Add JSON output support for SFF8636 modules.
> Patches #12-#14: Add JSON output support for SFF8079 and SFF8472 modules.
> Patch #15: Add a new schema JSON file for units documentation.
> Patches #16: Add '-j' support to ethtool

Acked-by: Jakub Kicinski <kuba@...nel.org>

Thanks a lot for doing this work!