Thread (42 messages) 42 messages, 6 authors, 2022-02-09

Re: [PATCH net-next 1/2] devlink: Add support to set port function as trusted

From: Jakub Kicinski <kuba@kernel.org>
Date: 2022-01-15 02:10:36

On Fri, 14 Jan 2022 10:15:49 +0100 Jiri Pirko wrote:
quoted
quoted
It was implicit that a driver API callback addition for both types of features is not good.
Devlink port function params enables to achieve both generic and device specific features.
Shall we proceed with port function params? What do you think?
I already addressed this. I don't like devlink params. They muddy the
water between vendor specific gunk and bona fide Linux uAPI. Build a
normal dedicated API.
Well, that is indeed true. But on the other hand, what is the alternative
solution? There are still going to be things wich are generic and driver-
specific. Params or no params. Or do you say we need some new well
defined enum-based api for generic stuff and driver-speficic will just
go to params?
The latter is where my thinking is right now. I think devlink params
are attracting too much vendor attention, when they should really be
more of control for quirks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help