On Wed, Oct 19, 2011 at 9:46 PM, Ben Pfaff <[email protected]> wrote:
> Signed-off-by: Ben Pfaff <[email protected]>

Acked-by: Jesse Gross <[email protected]>

> Arguably the call to dp_upcall() in dp_process_received_packet()
> should also be followed by consume_skb() instead of kfree_skb().

Yes, it definitely should be (that one went in at the same time as this one).

> And I see that fragment() in vport-capwap.c uses kfree_skb() from
> its "out" label that is used for both success and failure paths.

It's probably worth separating them out since it is easy enough.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to