diff options
author | John Fastabend <john.fastabend@gmail.com> | 2016-02-26 07:54:13 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-03-01 16:05:39 -0500 |
commit | 2b6ab0d3aae6bf1e08118060b0c5565778cd6b21 (patch) | |
tree | 58a2208c1b403db5309020117e984a8c3c12fd12 /include/net/pkt_cls.h | |
parent | 6843e7a2abe7cac10c19702ffec90018df6f040d (diff) | |
download | lwn-2b6ab0d3aae6bf1e08118060b0c5565778cd6b21.tar.gz lwn-2b6ab0d3aae6bf1e08118060b0c5565778cd6b21.zip |
net: cls_u32: move TC offload feature bit into cls_u32 offload logic
In the original series drivers would get offload requests for cls_u32
rules even if the feature bit is disabled. This meant the driver had
to do a boiler plate check on the feature bit before adding/deleting
the rule.
This patch lifts the check into the core code and removes it from the
driver specific case.
Signed-off-by: John Fastabend <john.r.fastabend@intel.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/pkt_cls.h')
-rw-r--r-- | include/net/pkt_cls.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/include/net/pkt_cls.h b/include/net/pkt_cls.h index e64d20b81047..6096e96fb78b 100644 --- a/include/net/pkt_cls.h +++ b/include/net/pkt_cls.h @@ -394,6 +394,9 @@ struct tc_cls_u32_offload { static inline bool tc_should_offload(struct net_device *dev) { + if (!(dev->features & NETIF_F_HW_TC)) + return false; + return dev->netdev_ops->ndo_setup_tc; } |