diff options
| author | Jesse Gross <jesse@nicira.com> | 2011-09-20 16:29:38 -0700 |
|---|---|---|
| committer | Jesse Gross <jesse@nicira.com> | 2011-09-21 10:24:02 -0700 |
| commit | 646b8753e467399966d1ed7bb56c81a84589029f (patch) | |
| tree | 2a0bfa44e974abd488f07a43af83968268b26b92 | |
| parent | d48c88ecdbab481a210a1d02a616a30ee261cabf (diff) | |
| download | openvswitch-646b8753e467399966d1ed7bb56c81a84589029f.tar.gz | |
datapath: Send to userspace errors shouldn't halt processing.
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 <jesse@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
| -rw-r--r-- | datapath/actions.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/datapath/actions.c b/datapath/actions.c index 83b531adf..deaa4dc92 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: |