On Tue, Dec 23, 2014 at 7:53 PM, Pravin Shelar <pshe...@nicira.com> wrote: > On Mon, Dec 22, 2014 at 8:32 PM, Jesse Gross <je...@nicira.com> wrote: >> On Mon, Dec 22, 2014 at 6:24 PM, Pravin B Shelar <pshe...@nicira.com> wrote: >>> Today vport-send has complex error handling because it involves >>> freeing skb and updating stats depending on return value from >>> vport send implementation. >>> This can be simplified by delegating responsibility of freeing >>> skb to the vport implementation for all cases. So that >>> vport-send needs just update stats. >>> >>> Signed-off-by: Pravin B Shelar <pshe...@nicira.com> >> >> Acked-by: Jesse Gross <je...@nicira.com> > > I pushed both patch to master. > I also posted this patch to netdev mailing list, I could not add your > acked-by since I merged Geneve changed in same patch. Can you have a > look at that patch as well.
It looks fine to me modulo the bug that the other guy pointed out. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
