From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Wed, 7 Mar 2007 16:03:55 -0800
> One change introduced by the workqueue removal patch is that adding an > interface that is up to a bridge which is also up does not ever call > br_stp_enable_port(), leaving the port in DISABLED state until we do > ifconfig down and up or link events occur. > > The following patch to the br_add_if function fixes it. > This is a regression introduced in 2.6.21. > > Submitted-by: [EMAIL PROTECTED] > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> Applied, but: > + if ((dev->flags & IFF_UP) && netif_carrier_ok(dev) && > + (br->dev->flags & IFF_UP)) I had to fix up the trailing whitespace on that second line. Please use "git apply --check --whitespace=error-all" or similar to validate the patches you send me. Thanks. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
