On Tue, Jul 30, 2013 at 05:29:10PM -0700, Ethan Jackson wrote: > This patch reverts commit 5559942 (ofproto-dpif: GOTO_TABLE recursion > removal.) by reintroducing the recursion through xlate_table_action(). > The main reason to do this is the introduction of new rule locking in > future patches. The code before this patch was relatively difficult > to lock in a clean straight-forward manner. > > Signed-off-by: Ethan Jackson <[email protected]>
I'm OK with changing the implementation, but I don't like the idea of the externally visible behavior changing. What if, instead of doing this iteratively, we simply don't penalize goto_table actions with taking up a level of resubmit? We limit the levels of resubmit to avoid loops, but goto_table can't loop (it can only go to a table with a higher number), so it's not a problem there. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
