[ovs-dev] [PATCH 1/2] revalidator: Eliminate duplicate flow handling.

Joe Stringer joestringer at nicira.com
Mon Jun 2 23:55:51 UTC 2014


On 3 June 2014 08:41, Ethan Jackson <ethan at nicira.com> wrote:

> The comment over the trylock exceeds the line limit.
>

I'll fix that up.


> I'm not sure if this is the right patch to fix it or not, but I don't
> like how we're overloading the meaning of the 'mark'.  I think the
> code would be a lot clearer if we had both a "already_dumped" flag and
> a "delete" flag  on every ukey.  What do you think?
>

I was planning to replace the mark by making use of the udpif->dump_seq, as
per the series here (Will rebase that series soon):
http://openvswitch.org/pipermail/dev/2014-May/040292.html


> Acked-by: Ethan Jackson <ethan at nicira.com>


Thanks, pushed to master.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140603/2da8207b/attachment-0005.html>


More information about the dev mailing list