[ovs-dev] [PATCH] bridge: Consistently use miimon status if miimon is configured.

Ben Pfaff blp at nicira.com
Fri Mar 18 23:50:50 UTC 2011


Thanks.  I pushed this one out, after resolving the conflicts.

On Fri, Mar 18, 2011 at 03:37:06PM -0700, Ethan Jackson wrote:
> Looks Good.  Minor changes may be required since the LACP series was
> been merged, but the approach looks fine.  Go ahead and merge.
> 
> Ethan
> 
> On Thu, Mar 17, 2011 at 10:48 AM, Ben Pfaff <blp at nicira.com> wrote:
> > A port can be configured to use miimon reporting as the criterion for
> > enabling or disabling an interface, but in some cases (such as for reading
> > the initial link status) the code was reading the carrier status instead.
> > This commit fixes the problem.
> >
> > This changes the meaning of the link_status column in the Interface table.
> > I don't think that the old meaning was useful to the controller in the
> > case of a bond configured for miimon monitoring, because the controller
> > could not use it to detect which interfaces the bond considered to be up
> > or down.



More information about the dev mailing list