[ovs-dev] [PATCH] vxlan: Create and delete tnl_backers in type_run()

Ethan Jackson ethan at nicira.com
Fri Feb 15 22:00:59 UTC 2013


> I don't think so. If you look, the code which is assigning iter->odp_port first
> checks if node is NULL.

Actually no I think he's right.  In the condition where the node is in
tmp_simap, we need to re-look it up after we've put it into
tnl_backers.  Otherwise later node will be non-null, but deleted, in
which case we will dereference it incorrectly.

>> I think that ofproto destruction will no longer delete tunnel ports.
>> I don't know whether that matters.

I'm not completely sure what the correct behavior is here.  If you
have multiple ofprotos sharing a backer, and only one is deleted.  I
think our current behavior is correct because we'll end up garbage
collecting the old backers.  If we're deleting the last ofproto using
a backer we'll end up leaving junk tnl backers around.  We could
special case, this, but I'm not sure if it matters.  Thoughts?

Ethan


>>
> I think the idea is for them to be deleted in type_run() instead.
>
>> Thanks,
>>
> One other thing I found with this code is that odp_port was not
> initialized to UINT32_MAX. I just fixed that and am testing it again.
>
> Thanks,
> Kyle
>
>> Ben.
>> _______________________________________________
>> dev mailing list
>> dev at openvswitch.org
>> http://openvswitch.org/mailman/listinfo/dev
>
>



More information about the dev mailing list