summaryrefslogtreecommitdiffstats
path: root/net/dsa/slave.c
diff options
context:
space:
mode:
authorJiri Pirko <jiri@mellanox.com>2017-11-01 11:47:39 +0100
committerDavid S. Miller <davem@davemloft.net>2017-11-02 16:10:39 +0900
commit44ae12a768b7212976a362c590075716a77e8f28 (patch)
tree02eeee68258d3f0b1c03c260852f51c4fadb2a24 /net/dsa/slave.c
parent0b5a89caee5c9958c18cd933c7f8891e35b21781 (diff)
downloadlinux-stable-44ae12a768b7212976a362c590075716a77e8f28.tar.gz
linux-stable-44ae12a768b7212976a362c590075716a77e8f28.tar.bz2
linux-stable-44ae12a768b7212976a362c590075716a77e8f28.zip
net: sched: move the can_offload check from binding phase to rule insertion phase
This restores the original behaviour before the block callbacks were introduced. Allow the drivers to do binding of block always, no matter if the NETIF_F_HW_TC feature is on or off. Move the check to the block callback which is called for rule insertion. Reported-by: Alexander Duyck <alexander.duyck@gmail.com> Signed-off-by: Jiri Pirko <jiri@mellanox.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa/slave.c')
-rw-r--r--net/dsa/slave.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index 48b954a76b0d..9b75d0ac4092 100644
--- a/net/dsa/slave.c
+++ b/net/dsa/slave.c
@@ -799,6 +799,9 @@ static int dsa_slave_setup_tc_block_cb(enum tc_setup_type type, void *type_data,
{
struct net_device *dev = cb_priv;
+ if (!tc_can_offload(dev))
+ return -EOPNOTSUPP;
+
switch (type) {
case TC_SETUP_CLSMATCHALL:
return dsa_slave_setup_tc_cls_matchall(dev, type_data, ingress);