netdev - Re: [BUG?] bonding, slave selection, carrier loss, etc.

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: <20120213104810.158d714a@nehalam.linuxnetplumber.net>
Date:	Mon, 13 Feb 2012 10:48:10 -0800
From:	Stephen Hemminger <shemminger@...tta.com>
To:	Chris Friesen <chris.friesen@...band.com>
Cc:	Ben Hutchings <bhutchings@...arflare.com>,
	Jay Vosburgh <fubar@...ibm.com>, andy@...yhouse.net,
	netdev <netdev@...r.kernel.org>
Subject: Re: [BUG?] bonding, slave selection, carrier loss, etc.

On Mon, 13 Feb 2012 12:16:59 -0600
Chris Friesen <chris.friesen@...band.com> wrote:

> On 02/11/2012 12:52 PM, Ben Hutchings wrote:
> > On Fri, 2012-02-10 at 17:53 -0800, Jay Vosburgh wrote:
> >> Chris Friesen<chris.friesen@...band.com>  wrote:
> 
> >>> The best solution would be for bonding to just register for notification
> >>> of the link going down.  Presumably most drivers should be doing that
> >>> properly by now, and for devices that get interrupt-driven notification
> >>> of link status changes this would allow the bonding code to react much
> >>> quicker.
> >>
> >> 	A quick look at some drivers shows that at least acenic still
> >> doesn't do netif_carrier_off, so converting entirely to a notifier-based
> >> failover mechanism would break drivers that work today.
> > [...]
> >
> > It might be worth having some sort of feature flag (in priv_flags) that
> > indicates whether the driver updates the link state.  Alternately,
> > disable polling of a device once you see a notification.

Just fix the drivers to update link state.
The whole mii polling method of bonding is really leftover from the era of
10 years ago when network drivers were stupid and didn't handle carrier.
--
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