[ovs-dev] [PATCH] ofproto: Fix ofport allocation.

Ben Pfaff blp at nicira.com
Tue Nov 20 20:40:27 UTC 2012


On Tue, Nov 20, 2012 at 03:29:03AM -0800, Gurucharan Shetty wrote:
> From: Gurucharan Shetty <shettyg at nicira.com>

You used shettyg as author and committer but signed off as gshetty.  I
think that the repository hook will reject that, requiring
consistency.

> Currently running the following 3 commands results
> in vswitchd running in an infinite loop.
> 
> ovs-vsctl add-port br1 port1 -- set interface port1 \
> type=internal ofport_request=1
> ovs-vsctl add-port br1 port2 -- set interface port2 \
> type=internal ofport_request=2
> ovs-vsctl add-port br1 port3 -- set interface port3 \
> type=internal
> 
> This patch lets us go through all the possible ofport
> values to find a free ofport.
> 
> Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>

Good catch, thank you!



More information about the dev mailing list