netdev - Re: [PATCH iproute2-next] bash-completion: devlink: fix port param name show completion

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: <20241119084547.4285a0dd@hermes.local>
Date: Tue, 19 Nov 2024 08:45:47 -0800
From: Stephen Hemminger <stephen@...workplumber.org>
To: Jiri Pirko <jiri@...nulli.us>
Cc: Saeed Mahameed <saeedm@...dia.com>, Saeed Mahameed <saeed@...nel.org>,
 David Ahern <dsahern@...il.com>, netdev@...r.kernel.org
Subject: Re: [PATCH iproute2-next] bash-completion: devlink: fix port param
 name show completion

On Sat, 16 Nov 2024 08:58:05 +0100
Jiri Pirko <jiri@...nulli.us> wrote:

> Fri, Nov 15, 2024 at 07:44:01PM CET, saeedm@...dia.com wrote:
> >On 15 Nov 13:58, Jiri Pirko wrote:  
> >> Fri, Nov 15, 2024 at 06:58:48AM CET, saeed@...nel.org wrote:  
> >> > From: Saeed Mahameed <saeedm@...dia.com>
> >> > 
> >> > Port param names are found with "devlink port param show", and not
> >> > "devlink param show", fix that.
> >> > 
> >> > Port dev name can be a netdev, so find the actual port dev before
> >> > querying "devlink port params show | jq '... [$dev] ...'",
> >> > since "devlink port params show" doesn't return the netdev name,
> >> > but the actual port dev name.
> >> > 
> >> > Signed-off-by: Saeed Mahameed <saeedm@...dia.com>
> >> > ---
> >> > bash-completion/devlink | 11 ++++++++++-
> >> > 1 file changed, 10 insertions(+), 1 deletion(-)
> >> > 
> >> > diff --git a/bash-completion/devlink b/bash-completion/devlink
> >> > index 52dc82b3..ac5ea62c 100644
> >> > --- a/bash-completion/devlink
> >> > +++ b/bash-completion/devlink
> >> > @@ -43,6 +43,15 @@ _devlink_direct_complete()
> >> >             value=$(devlink -j dev param show 2>/dev/null \
> >> >                     | jq ".param[\"$dev\"][].name")
> >> >             ;;
> >> > +        port_param_name)
> >> > +            dev=${words[4]}
> >> > +            # dev could be a port or a netdev so find the port
> >> > +            portdev=$(devlink -j port show dev $dev 2>/dev/null \
> >> > +                    | jq '.port as $ports | $ports | keys[] as $keys | keys[0] ')
> >> > +
> >> > +            value=$(devlink -j port param show 2>/dev/null \  
> >> 
> >> As you only care about params for specific port, you should pass it as
> >> cmdline option here. And you can pass netdev directly, devlink knows how
> >> to handle that. If I'm not missing anything in the code, should work
> >> right now.
> >>   
> >
> >Nope doesn't work:
> >
> >$ devlink -j port param show mlx5_1
> >Parameter name expected.
> >
> >$ devlink -j port param show auxiliary/mlx5_core.eth.0/65535
> >Parameter name expected.  
> 
> Okay, so fix it :)

Holding off on applying this until some conclusion is reached.