netdev - Re: [patch iproute2-next 0/6] devlink: implement dump selector for devlink objects show commands

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: <ZPWkbOHuqBm13A7t@nanopsycho>
Date: Mon, 4 Sep 2023 11:33:32 +0200
From: Jiri Pirko <jiri@...nulli.us>
To: netdev@...r.kernel.org
Cc: stephen@...workplumber.org, dsahern@...il.com
Subject: Re: [patch iproute2-next 0/6] devlink: implement dump selector for
 devlink objects show commands

Thu, Aug 31, 2023 at 03:22:23PM CEST, jiri@...nulli.us wrote:
>From: Jiri Pirko <jiri@...dia.com>
>
>First 5 patches are preparations for the last one.
>
>Motivation:
>
>For SFs, one devlink instance per SF is created. There might be
>thousands of these on a single host. When a user needs to know port
>handle for specific SF, he needs to dump all devlink ports on the host
>which does not scale good.
>
>Solution:
>
>Allow user to pass devlink handle (and possibly other attributes)
>alongside the dump command and dump only objects which are matching
>the selection.
>
>Example:
>$ devlink port show
>auxiliary/mlx5_core.eth.0/65535: type eth netdev eth2 flavour physical port 0 splittable false
>auxiliary/mlx5_core.eth.1/131071: type eth netdev eth3 flavour physical port 1 splittable false
>
>$ devlink port show auxiliary/mlx5_core.eth.0
>auxiliary/mlx5_core.eth.0/65535: type eth netdev eth2 flavour physical port 0 splittable false
>
>$ devlink port show auxiliary/mlx5_core.eth.1
>auxiliary/mlx5_core.eth.1/131071: type eth netdev eth3 flavour physical port 1 splittable false
>
>Jiri Pirko (6):
>  devlink: move DL_OPT_SB into required options
>  devlink: make parsing of handle non-destructive to argv
>  devlink: implement command line args dry parsing
>  devlink: return -ENOENT if argument is missing
>  mnl_utils: introduce a helper to check if dump policy exists for
>    command
>  devlink: implement dump selector for devlink objects show commands
>
> devlink/devlink.c   | 376 ++++++++++++++++++++++++++------------------
> include/mnl_utils.h |   1 +
> lib/mnl_utils.c     | 121 +++++++++++++-
> 3 files changed, 342 insertions(+), 156 deletions(-)
>
>-- 
>2.41.0
>

There is an issue with the first patch, fixing and sending v2.

pw-bot: changes-requested