[ovs-dev] [PATCH] dpif-netdev: Update max_mtu correctly.

Ben Pfaff blp at nicira.com
Mon Apr 29 21:11:28 UTC 2013


Thanks, applied to master.

On Fri, Apr 26, 2013 at 03:50:26PM -0700, Justin Pettit wrote:
> Looks good.
> 
> --Justin
> 
> 
> On Apr 26, 2013, at 11:37 AM, Ben Pfaff <blp at nicira.com> wrote:
> 
> > What a dumb bug.
> > 
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> > lib/dpif-netdev.c |    2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
> > index e4a2f75..6838800 100644
> > --- a/lib/dpif-netdev.c
> > +++ b/lib/dpif-netdev.c
> > @@ -412,7 +412,7 @@ do_add_port(struct dp_netdev *dp, const char *devname, const char *type,
> >     port->type = xstrdup(type);
> > 
> >     error = netdev_get_mtu(netdev, &mtu);
> > -    if (!error) {
> > +    if (!error && mtu > max_mtu) {
> >         max_mtu = mtu;
> >     }
> > 
> > -- 
> > 1.7.2.5
> > 
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> 



More information about the dev mailing list