[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 12 Dec 2013 13:47:32 -0800
From: John Fastabend <john.r.fastabend@...el.com>
To: Florian Fainelli <f.fainelli@...il.com>
CC: raspl@...ux.vnet.ibm.com, David Miller <davem@...emloft.net>,
Stephen Hemminger <stephen@...workplumber.org>,
Ben Hutchings <bhutchings@...arflare.com>,
blaschka@...ux.vnet.ibm.com, netdev <netdev@...r.kernel.org>,
linux-s390@...r.kernel.org
Subject: Re: [PATCH 0/2] Display adjacent switch port's attributes
[...]
>> Just to elaborate...
>>
>> Any application using lldp information will want to get events when
>> TLVs change. Maybe you can contrive ethtool to do this but its going to
>> be ugly. Netlink can support multicast events and applications can
>> register for them. Also netlink's TLV format matches nicely with LLDPs
>> TLV format.
>
> ethtool and netlink usually intersect for a few bits of information
> such as link status for instance. It is useful to have this
> information twice, with ethtool as a debugging aid, and via netlink to
> take appropriate actions.
>
> Maybe we just need to be clear on what needs to be present in ethtool
> only (configuration, static information) and see on a case-by-case
> what needs to be present in both ethtool and netlink?
>
OK if there is an enable/disable bit in ethtool that might make some
sense. Or an error flag that is helpful to have.
In this case we are dealing with peer attributes which are dynamic and
in my opinion should go into netlink and duplicating them in ethtool
although possible doesn't seem very useful to me.
.John
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists