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

[Gurucharan Shetty]

On Tue, Nov 20, 2012 at 12:40 PM, Ben Pfaff <blp at nicira.com> wrote:
> 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.
I corrected (made author as gshetty at nicira.com) and pushed to master.
>
>> 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!