netdev - Re: [PATCH net-next 00/11] mlxsw: extend line card model by devices and 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] [thread-next>] [day] [month] [year] [list]

Message-ID: <Yox/TkxkTUtd0RMM@nanopsycho>
Date:   Tue, 24 May 2022 08:46:38 +0200
From:   Jiri Pirko <jiri@...nulli.us>
To:     Jakub Kicinski <kuba@...nel.org>
Cc:     Ido Schimmel <idosch@...sch.org>, Ido Schimmel <idosch@...dia.com>,
        netdev@...r.kernel.org, davem@...emloft.net, pabeni@...hat.com,
        jiri@...dia.com, petrm@...dia.com, dsahern@...il.com,
        andrew@...n.ch, mlxsw@...dia.com
Subject: Re: [PATCH net-next 00/11] mlxsw: extend line card model by devices
 and info

Mon, May 23, 2022 at 07:56:40PM CEST, kuba@...nel.org wrote:
>On Mon, 23 May 2022 11:42:07 +0200 Jiri Pirko wrote:
>> Mon, May 02, 2022 at 04:39:33PM CEST, kuba@...nel.org wrote:
>> >On Sat, 30 Apr 2022 08:27:35 +0200 Jiri Pirko wrote:  
>> >> Now I just want to use this component name to target individual line
>> >> cards. I see it is a nice fit. Don't you think?  
>> >
>> >Still on the fence.  
>> 
>> Why?
>
>IIRC my concern was mixing objects. We have component name coming from
>lc info, but then use it in dev flash.

Sure. I considered that. The thing is, even if you put the lc component
names to output of "devlink dev info", you would need to provide lc
objects as well (somehow) - to contain the versions.

But the component name is related to lc object listed in "devlink lc",
so "devlink lc info" sounds line the correct place to put it.

If you are concern about "devlink dev flash" using component name from
"devlink lc info", I would rather introduce "devlink lc flash" so you
have a match. But from what I see, I don't really see the necessity for
this match. Do you?


>
>> >> I see that the manpage is mentioning "the component names from devlink dev info"
>> >> which is not actually implemented, but exactly what I proposed.  
>> >
>> >How do you tie the line card to the component name? lc8_dev0 from 
>> >the flashing example is not present in the lc info output.  
>> 
>> Okay, I will move it there. Makes sense.
>
>FWIW I think I meant my comment as a way to underline that what you
>argue for is not what's implemented (assuming your "not actually
>implemented" referred to the flashing). I was trying to send you back 
>to the drawing board rather than break open a box of band-aides.

Sure, lets do this right, I don't want to band-aide anything...