datapath: Properly initialize ovs_skb_cb of packet from userspace.
authorBen Pfaff <blp@nicira.com>
Sat, 19 Feb 2011 04:53:51 +0000 (20:53 -0800)
committerBen Pfaff <blp@nicira.com>
Sat, 19 Feb 2011 04:53:51 +0000 (20:53 -0800)
The ovs_skb_cb in 'packet' in this function is initially a clone of the
corresponding area in 'skb', which came from the Netlink layer and thus
isn't necessarily all-zeros.  This commit initializes it properly before
passing it along to execute_actions().

The most common problem caused by failing to initialize the ovs_skb_cb
properly was that on Linux 2.6.26 and earlier, where Open vSwitch keeps
its own vlan_tci field inside ovs_skb_cb, the first packet of a flow would
get sent out tagged with a random VLAN (usually 0x0001 or 0xffff in our
testing).  This commit should fix that problem.

Another likely problem would be for turning on sFlow to randomly panic the
kernel.  That problem would not be kernel version dependent.  We haven't
been testing sFlow so we haven't noticed this problem.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Justin Pettit <jpettit@nicira.com>
Reported-by: Pankaj Thakkar <thakkar@nicira.com>
datapath/datapath.c

index 940a58142dbef97ad0881fdfc308f6640122d361..dcff05f2986dbc23749c01063eb47ce3f10b120c 100644 (file)
@@ -709,6 +709,15 @@ static int odp_packet_cmd_execute(struct sk_buff *skb, struct genl_info *info)
        if (err)
                goto exit;
 
+       /* Initialize OVS_CB (it came from Netlink so might not be zeroed). */
+       OVS_CB(packet)->vport = NULL;
+       OVS_CB(packet)->flow = NULL;
+       /* execute_actions() will reset tun_id to 0 anyhow. */
+#ifdef NEED_CSUM_NORMALIZE
+       OVS_CB(packet)->ip_summed = OVS_CSUM_NONE;
+#endif
+       vlan_copy_skb_tci(packet);
+
        rcu_read_lock();
        dp = get_dp(odp_header->dp_ifindex);
        err = -ENODEV;