On Wed, May 20, 2015 at 5:06 PM, Ethan Jackson <[email protected]> wrote: > When executing actions, it's possible a recirculation will occur > causing dp_netdev_input() to be called multiple times. If the batch > pointers embedded in dp_netdev_flow aren't cleared, it's possible > packets after the recirculation will be reinserted into a batch > associated with the original lookup. This could be very bad. > > This patch fixes the problem by zeroing out flow batch pointers before > calling packet_batch_execute(). This probably has a slightly negative > performance impact, though I haven't tried it. > > Signed-off-by: Ethan Jackson <[email protected]>
Good catch. Looks good to me too. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
