On 04.02.2016 14:26, Jiri Pirko wrote:
Thu, Feb 04, 2016 at 02:22:17PM CET, han...@stressinduktion.org wrote:
On 03.02.2016 16:17, Daniel Borkmann wrote:
On 02/03/2016 02:33 PM, Jiri Pirko wrote:
Wed, Feb 03, 2016 at 02:31:33PM CET, bro...@redhat.com wrote:
On Wed, 3 Feb 2016 11:47:56 +0100
Jiri Pirko <j...@resnulli.us> wrote:
From: Jiri Pirko <j...@mellanox.com>
[...]
It is available here:
https://github.com/jpirko/devlink
IHMO this very short command name "dl" is not an advantage. It is
simply too unspecific and short for a good Google search. E.g. when
searching for good examples for using "dl". I think "devlink" would be
better. If you like short commands do: alias dl="devlink"
I was thinking about using "devlink". Decided to go with shortened
version so this is in-line with "ip". But you have a point.
Btw, if you add this tool into iproute2 (which would be preferred), then
probably dl should be ok (and easier retrievable in that relation).
This doesn't seem to be too much related to networking? Why can't something
like this be in sysfs?
It is related to networking quite bit. There has been couple of
iteration of this, including sysfs and configfs implementations. There
has been a consensus reached that this should be done by netlink. I
believe netlink is really the best for this purpose. Sysfs is not a good
idea.
Okay. I see it more as changing mode of operation of hardware and thus
has not really anything to do with networking. If you say you change
ethernet to infiniband it has something to do with networking, sure. But
I am fine with this, I just thought the code size could be reduced by
adding this to sysfs quite a lot. I don't have a strong opinion on this.
The attributes seem to reassemble sysfs quite a lot and at the same time
implement a dependency on the ifindex which in a relational model is not
really possible? Do infiniband devices have ifindexes? Names for
infiniband and netdev are hardcoded? Shouldn't an interface to deliver
hw msg be better in terms of streaming than using netlink?
Thanks,
Hannes
