flow_dissector: Add flags argument to skb_flow_dissector functions
authorTom Herbert <tom@herbertland.com>
Tue, 1 Sep 2015 16:24:27 +0000 (09:24 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Sep 2015 22:06:22 +0000 (15:06 -0700)
The flags argument will allow control of the dissection process (for
instance whether to parse beyond L3).

Signed-off-by: Tom Herbert <tom@herbertland.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c
drivers/net/ethernet/cisco/enic/enic_clsf.c
drivers/net/hyperv/netvsc_drv.c
include/linux/skbuff.h
net/core/flow_dissector.c
net/ethernet/eth.c
net/sched/cls_flow.c
net/sched/cls_flower.c
net/sched/sch_choke.c

index 06e2d01f0b4e1ad60d4aa79d3512c550219d2c25..771a449d2f56384bae4a5409ffb588dfb40e1f61 100644 (file)
@@ -3095,7 +3095,7 @@ static bool bond_flow_dissect(struct bonding *bond, struct sk_buff *skb,
        int noff, proto = -1;
 
        if (bond->params.xmit_policy > BOND_XMIT_POLICY_LAYER23)
-               return skb_flow_dissect_flow_keys(skb, fk);
+               return skb_flow_dissect_flow_keys(skb, fk, 0);
 
        fk->ports.ports = 0;
        noff = skb_network_offset(skb);
index d106186f4f4abfbcf5b662bc80870b4ae4ab4216..3c677ed3c29e722d6e375afae0ee928e2e10838a 100644 (file)
@@ -177,7 +177,7 @@ int enic_rx_flow_steer(struct net_device *dev, const struct sk_buff *skb,
        int res, i;
 
        enic = netdev_priv(dev);
-       res = skb_flow_dissect_flow_keys(skb, &keys);
+       res = skb_flow_dissect_flow_keys(skb, &keys, 0);
        if (!res || keys.basic.n_proto != htons(ETH_P_IP) ||
            (keys.basic.ip_proto != IPPROTO_TCP &&
             keys.basic.ip_proto != IPPROTO_UDP))
index 2990024b90f972e3e22c56f21d2cce278da651ab..409b48e1e589dfdfa795a56b7369d835fa4643da 100644 (file)
@@ -239,7 +239,7 @@ static bool netvsc_set_hash(u32 *hash, struct sk_buff *skb)
        struct flow_keys flow;
        int data_len;
 
-       if (!skb_flow_dissect_flow_keys(skb, &flow) ||
+       if (!skb_flow_dissect_flow_keys(skb, &flow, 0) ||
            !(flow.basic.n_proto == htons(ETH_P_IP) ||
              flow.basic.n_proto == htons(ETH_P_IPV6)))
                return false;
index bbe41bccfc5f297a037946a7a5055b3ea8e1f4c6..9e62687c70f32ea606cd23bc36501698fc2f7e91 100644 (file)
@@ -991,31 +991,34 @@ void skb_flow_dissector_init(struct flow_dissector *flow_dissector,
 bool __skb_flow_dissect(const struct sk_buff *skb,
                        struct flow_dissector *flow_dissector,
                        void *target_container,
-                       void *data, __be16 proto, int nhoff, int hlen);
+                       void *data, __be16 proto, int nhoff, int hlen,
+                       unsigned int flags);
 
 static inline bool skb_flow_dissect(const struct sk_buff *skb,
                                    struct flow_dissector *flow_dissector,
-                                   void *target_container)
+                                   void *target_container, unsigned int flags)
 {
        return __skb_flow_dissect(skb, flow_dissector, target_container,
-                                 NULL, 0, 0, 0);
+                                 NULL, 0, 0, 0, flags);
 }
 
 static inline bool skb_flow_dissect_flow_keys(const struct sk_buff *skb,
-                                             struct flow_keys *flow)
+                                             struct flow_keys *flow,
+                                             unsigned int flags)
 {
        memset(flow, 0, sizeof(*flow));
        return __skb_flow_dissect(skb, &flow_keys_dissector, flow,
-                                 NULL, 0, 0, 0);
+                                 NULL, 0, 0, 0, flags);
 }
 
 static inline bool skb_flow_dissect_flow_keys_buf(struct flow_keys *flow,
                                                  void *data, __be16 proto,
-                                                 int nhoff, int hlen)
+                                                 int nhoff, int hlen,
+                                                 unsigned int flags)
 {
        memset(flow, 0, sizeof(*flow));
        return __skb_flow_dissect(NULL, &flow_keys_buf_dissector, flow,
-                                 data, proto, nhoff, hlen);
+                                 data, proto, nhoff, hlen, flags);
 }
 
 static inline __u32 skb_get_hash(struct sk_buff *skb)
@@ -2046,7 +2049,7 @@ static inline void skb_probe_transport_header(struct sk_buff *skb,
 
        if (skb_transport_header_was_set(skb))
                return;
-       else if (skb_flow_dissect_flow_keys(skb, &keys))
+       else if (skb_flow_dissect_flow_keys(skb, &keys, 0))
                skb_set_transport_header(skb, keys.control.thoff);
        else
                skb_set_transport_header(skb, offset_hint);
index 22f3d768b45925d03beecb5715c419b25fde09bf..c3d9807cb34ec348f74be7dee6c6a597ba84f6c8 100644 (file)
@@ -121,7 +121,8 @@ EXPORT_SYMBOL(__skb_flow_get_ports);
 bool __skb_flow_dissect(const struct sk_buff *skb,
                        struct flow_dissector *flow_dissector,
                        void *target_container,
-                       void *data, __be16 proto, int nhoff, int hlen)
+                       void *data, __be16 proto, int nhoff, int hlen,
+                       unsigned int flags)
 {
        struct flow_dissector_key_control *key_control;
        struct flow_dissector_key_basic *key_basic;
@@ -556,7 +557,7 @@ EXPORT_SYMBOL(flow_hash_from_keys);
 static inline u32 ___skb_get_hash(const struct sk_buff *skb,
                                  struct flow_keys *keys, u32 keyval)
 {
-       if (!skb_flow_dissect_flow_keys(skb, keys))
+       if (!skb_flow_dissect_flow_keys(skb, keys, 0))
                return 0;
 
        return __flow_hash_from_keys(keys, keyval);
@@ -726,7 +727,7 @@ u32 skb_get_poff(const struct sk_buff *skb)
 {
        struct flow_keys keys;
 
-       if (!skb_flow_dissect_flow_keys(skb, &keys))
+       if (!skb_flow_dissect_flow_keys(skb, &keys, 0))
                return 0;
 
        return __skb_get_poff(skb, skb->data, &keys, skb_headlen(skb));
index 217127c3a3ef346475a8d0f4c41e8b9da2328b0c..d850fdc828f932c20191741a75bf90acffb9f55f 100644 (file)
@@ -132,7 +132,7 @@ u32 eth_get_headlen(void *data, unsigned int len)
 
        /* parse any remaining L2/L3 headers, check for L4 */
        if (!skb_flow_dissect_flow_keys_buf(&keys, data, eth->h_proto,
-                                           sizeof(*eth), len))
+                                           sizeof(*eth), len, 0))
                return max_t(u32, keys.control.thoff, sizeof(*eth));
 
        /* parse for any L4 headers */
index bb2a0f529c1f519f79f22ffa046013d7ac863eb5..536838b657bfcfb8e33596b7a7d5aec76b72e4f4 100644 (file)
@@ -301,7 +301,7 @@ static int flow_classify(struct sk_buff *skb, const struct tcf_proto *tp,
 
                keymask = f->keymask;
                if (keymask & FLOW_KEYS_NEEDED)
-                       skb_flow_dissect_flow_keys(skb, &flow_keys);
+                       skb_flow_dissect_flow_keys(skb, &flow_keys, 0);
 
                for (n = 0; n < f->nkeys; n++) {
                        key = ffs(keymask) - 1;
index 2f3d03f99487ed35c4af1e5c7ede590e4b0e3721..57692947ebbe61461ca0958cdee531a6b7b81153 100644 (file)
@@ -129,7 +129,7 @@ static int fl_classify(struct sk_buff *skb, const struct tcf_proto *tp,
         * so do it rather here.
         */
        skb_key.basic.n_proto = skb->protocol;
-       skb_flow_dissect(skb, &head->dissector, &skb_key);
+       skb_flow_dissect(skb, &head->dissector, &skb_key, 0);
 
        fl_set_masked_key(&skb_mkey, &skb_key, &head->mask);
 
index 665bde07916b833955a872fc2deff503f85226cf..02bfd3d1c4f078e8b7da08791ea1069218095d1a 100644 (file)
@@ -170,13 +170,13 @@ static bool choke_match_flow(struct sk_buff *skb1,
 
        if (!choke_skb_cb(skb1)->keys_valid) {
                choke_skb_cb(skb1)->keys_valid = 1;
-               skb_flow_dissect_flow_keys(skb1, &temp);
+               skb_flow_dissect_flow_keys(skb1, &temp, 0);
                make_flow_keys_digest(&choke_skb_cb(skb1)->keys, &temp);
        }
 
        if (!choke_skb_cb(skb2)->keys_valid) {
                choke_skb_cb(skb2)->keys_valid = 1;
-               skb_flow_dissect_flow_keys(skb2, &temp);
+               skb_flow_dissect_flow_keys(skb2, &temp, 0);
                make_flow_keys_digest(&choke_skb_cb(skb2)->keys, &temp);
        }