]> git.proxmox.com Git - ovs.git/commitdiff
netdev-tc-offloads: Fix incorrect mask in probe_multi_mask_per_prio().
authorBen Pfaff <blp@ovn.org>
Fri, 11 May 2018 16:10:01 +0000 (09:10 -0700)
committerBen Pfaff <blp@ovn.org>
Fri, 11 May 2018 17:45:28 +0000 (10:45 -0700)
Presumably this was meant to be all-one-bits but it wasn't.  It also didn't
have the right endianness for an ovs_be16, so "sparse" complained.

CC: Paul Blakey <paulb@mellanox.com>
CC: Simon Horman <simon.horman@netronome.com>
Fixes: d00eeded6a9b ("netdev-tc-offloads: Probe for allowing multiple masks on single priority")
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Justin Pettit <jpettit@ovn.org>
lib/netdev-tc-offloads.c

index 7bac62ac3a7d5fda19fb2d9b2fa293410bf2405f..69d6533e4b333f29a56bb8a317c95ebdb9c415db 100644 (file)
@@ -1213,7 +1213,7 @@ probe_multi_mask_per_prio(int ifindex)
     memset(&flower, 0, sizeof flower);
 
     flower.key.eth_type = htons(ETH_P_IP);
-    flower.mask.eth_type = 0xfff;
+    flower.mask.eth_type = OVS_BE16_MAX;
     memset(&flower.key.dst_mac, 0x11, sizeof flower.key.dst_mac);
     memset(&flower.mask.dst_mac, 0xff, sizeof flower.mask.dst_mac);