If we encounter an error when sending a packet to userspace due to an explicit action we stop processing further actions. This makes sense for things like push vlan, where to continue means outputting an incorrect packet. However, sending to userspace is more akin to outputting to a port, which does not halt further processing. For consistency, ignore errors in this case as well.
Signed-off-by: Jesse Gross <[email protected]> --- datapath/actions.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/datapath/actions.c b/datapath/actions.c index 83b531a..deaa4dc 100644 --- a/datapath/actions.c +++ b/datapath/actions.c @@ -290,7 +290,7 @@ static int do_execute_actions(struct datapath *dp, struct sk_buff *skb, break; case OVS_ACTION_ATTR_USERSPACE: - err = output_userspace(dp, skb, nla_get_u64(a)); + output_userspace(dp, skb, nla_get_u64(a)); break; case OVS_ACTION_ATTR_SET_TUNNEL: -- 1.7.4.1 _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
