From e087eb215b346ac42fc9d7b96faeb272aa5d68e7 Mon Sep 17 00:00:00 2001 From: Jesse Gross Date: Mon, 1 Aug 2011 00:35:20 -0700 Subject: [PATCH] datapath: Set vport in skb when executed from userspace. Currently, the OVS_CB(skb)->vport member is never initialized for packets coming from userspace. This means that they can never be sampled by sFlow and generally violates our principle that userspace packets should be made to look the same as others. Signed-off-by: Jesse Gross Acked-by: Pravin Shelar --- datapath/datapath.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/datapath/datapath.c b/datapath/datapath.c index 2f30bdcf8..ccdcb754d 100644 --- a/datapath/datapath.c +++ b/datapath/datapath.c @@ -713,6 +713,11 @@ static int odp_packet_cmd_execute(struct sk_buff *skb, struct genl_info *info) err = -ENODEV; if (!dp) goto err_unlock; + + if (flow->key.eth.in_port < DP_MAX_PORTS) + OVS_CB(packet)->vport = get_vport_protected(dp, + flow->key.eth.in_port); + err = execute_actions(dp, packet); rcu_read_unlock(); -- 2.43.0