Pushed, thanks! I corrected a typo in a comment (“alloed” -> “allowed”), which breaks the next patch in the series. The resolution is trivial, so I’d rather not send out a new version if not needed otherwise.
Jarno On Dec 13, 2013, at 3:59 PM, Ben Pfaff <b...@nicira.com> wrote: > 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
