netdev - Re: [PATCH net-next] tools: ynl: add --list-ops and --list-msgs to CLI

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: <m2fruzfbcu.fsf@gmail.com>
Date: Fri, 03 May 2024 11:17:37 +0100
From: Donald Hunter <donald.hunter@...il.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: davem@...emloft.net,  netdev@...r.kernel.org,  edumazet@...gle.com,
  pabeni@...hat.com,  jiri@...nulli.us,  alessandromarcolini99@...il.com
Subject: Re: [PATCH net-next] tools: ynl: add --list-ops and --list-msgs to CLI

Jakub Kicinski <kuba@...nel.org> writes:

> I often forget the exact naming of ops and have to look at
> the spec to find it. Add support for listing the operations:
>
>   $ ./cli.py --spec .../netdev.yaml --list-ops
>   dev-get  [ do, dump ]
>   page-pool-get  [ do, dump ]
>   page-pool-stats-get  [ do, dump ]
>   queue-get  [ do, dump ]
>   napi-get  [ do, dump ]
>   qstats-get  [ dump ]
>
> For completeness also support listing all ops (including
> notifications:
>
>   # ./cli.py --spec .../netdev.yaml --list-msgs
>   dev-get  [ dump, do ]
>   dev-add-ntf  [ notify ]
>   dev-del-ntf  [ notify ]
>   dev-change-ntf  [ notify ]
>   page-pool-get  [ dump, do ]
>   page-pool-add-ntf  [ notify ]
>   page-pool-del-ntf  [ notify ]
>   page-pool-change-ntf  [ notify ]
>   page-pool-stats-get  [ dump, do ]
>   queue-get  [ dump, do ]
>   napi-get  [ dump, do ]
>   qstats-get  [ dump ]
>
> Use double space after the name for slightly easier to read
> output.
>
> Signed-off-by: Jakub Kicinski <kuba@...nel.org>

Reviewed-by: Donald Hunter <donald.hunter@...il.com>