On Fri, Nov 15, 2013 at 03:12:18PM -0800, Jarno Rajahalme wrote:
> Allowing the packet to be modified by execution allows less data
> copying for userspace action execution.  Some users of the
> dpif_execute already expect that the packet may be modified.  This
> patch makes this behavior uniform and makes the userspace datapath and
> the execution helpers modify the packet as it is being executed.
> Userspace action now steals the packet if given permission, as the
> packet is normally not needed after it.  The only exception is the
> sample action, and this is accounted for my keeping track of any
> actions that could be following the userspace action.
> 
> The packet in dpif_upcall is changed from a pointer to a struct,
> allowing the packet to be honest about it's headroom.  After this
> change the packet can safely be pushed on over the precarious 4 byte
> limit earlier allowed by the netlink data preceding the packet.
> 
> Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>

Acked-by: Ben Pfaff <b...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to