Re: [PATCH 0/2] Display adjacent switch port's attributes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2013-12-12 at 13:47 -0800, John Fastabend wrote:
> [...]
> 
> >> 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.

I agree with this - rtnetlink seems to make more sense.

Ben.

-- 
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.

--
To unsubscribe from this list: send the line "unsubscribe linux-s390" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Kernel Development]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Info]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Linux Media]     [Device Mapper]

  Powered by Linux