]> git.proxmox.com Git - ovs.git/commit
ofproto: Always set MTU for new internal ports.
authorDaniele Di Proietto <diproiettod@vmware.com>
Tue, 30 Aug 2016 16:58:37 +0000 (09:58 -0700)
committerDaniele Di Proietto <diproiettod@vmware.com>
Wed, 31 Aug 2016 00:49:35 +0000 (17:49 -0700)
commit47bf118665a3d0f3c153d1fe80e9af02ac9a4e9c
tree85f3f260102548ecc8bc8aa6c24a98a3f4bb7373
parent7c12e20006118e69a98f6840731ebf0c87582b4a
ofproto: Always set MTU for new internal ports.

We only change the MTU of new internal ports if it is bigger than the
bridge minimum.  But when the minimum MTU of the bridge is updated we
change the MTU of all internal ports no matter what.

The behavior is inconsistent, because now the internal ports MTU depends
on the order in which the ports were added.

This commit fixes the problem by _always_ setting the MTU of new
internal ports to the bridge minimum.  I'm not sure what was the logic
behind only adjusting the mtu if it was too big.

A testcase is improved to detect the problem.

VMware-BZ: #1718776
Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com>
Acked-by: Ben Pfaff <blp@ovn.org>
ofproto/ofproto.c
tests/ofproto-dpif.at