On Thu, Feb 2, 2017 at 10:56 AM, Ben Pfaff <b...@ovn.org> wrote: > On Wed, Jan 11, 2017 at 04:00:04PM -0800, Andy Zhou wrote: >> User space implementation of the sample action is not consistent with >> kernel datapath. In kernel datapath, the side effects of actions >> within the sample actions are not visible to the subsequent actions. >> Current user space handling does not follow the same logic. This patch >> makes them consistent. >> >> Signed-off-by: Andy Zhou <az...@ovn.org> > > Thanks! > > This looks like a bug fix, but I suspect that there's no way for > existing code to actually trigger the bug, because I think that OVS at > least up to 2.6 only ever puts actions that do not modify the packet > into a sample action. > > Acked-by: Ben Pfaff <b...@ovn.org>
This patch was sent out before we have decided to add the userspace datapath 'clone' action. Now that we have the clone action. The sample action will continue to be used for the current use case, so your comment will continue to apply. The fix adds an extra packet copy. The current use cases are not performance critical so may it is not a big deal. In case this becomes an issue, we then should revisit the logic here an see if we can avoid packet copy in case it is not required. Pushed to master. Thanks for the review and comment. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
