net: sched: mq: add simple offload notification
authorJakub Kicinski <jakub.kicinski@netronome.com>
Sat, 26 May 2018 04:53:35 +0000 (21:53 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 29 May 2018 13:49:16 +0000 (09:49 -0400)
mq offload is trivial, we just need to let the device know
that the root qdisc is mq.  Alternative approach would be
to export qdisc_lookup() and make drivers check the root
type themselves, but notification via ndo_setup_tc is more
in line with other qdiscs.

Note that mq doesn't hold any stats on it's own, it just
adds up stats of its children.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netdevice.h
include/net/pkt_cls.h
net/sched/sch_mq.c

index f45b1a4..6b863ed 100644 (file)
@@ -791,6 +791,7 @@ enum tc_setup_type {
        TC_SETUP_QDISC_CBS,
        TC_SETUP_QDISC_RED,
        TC_SETUP_QDISC_PRIO,
+       TC_SETUP_QDISC_MQ,
 };
 
 /* These structures hold the attributes of bpf state that are being passed
index f3ec437..942f839 100644 (file)
@@ -778,6 +778,16 @@ struct tc_qopt_offload_stats {
        struct gnet_stats_queue *qstats;
 };
 
+enum tc_mq_command {
+       TC_MQ_CREATE,
+       TC_MQ_DESTROY,
+};
+
+struct tc_mq_qopt_offload {
+       enum tc_mq_command command;
+       u32 handle;
+};
+
 enum tc_red_command {
        TC_RED_REPLACE,
        TC_RED_DESTROY,
index f062a18..6ccf6da 100644 (file)
@@ -16,6 +16,7 @@
 #include <linux/errno.h>
 #include <linux/skbuff.h>
 #include <net/netlink.h>
+#include <net/pkt_cls.h>
 #include <net/pkt_sched.h>
 #include <net/sch_generic.h>
 
@@ -23,12 +24,28 @@ struct mq_sched {
        struct Qdisc            **qdiscs;
 };
 
+static int mq_offload(struct Qdisc *sch, enum tc_mq_command cmd)
+{
+       struct net_device *dev = qdisc_dev(sch);
+       struct tc_mq_qopt_offload opt = {
+               .command = cmd,
+               .handle = sch->handle,
+       };
+
+       if (!tc_can_offload(dev) || !dev->netdev_ops->ndo_setup_tc)
+               return -EOPNOTSUPP;
+
+       return dev->netdev_ops->ndo_setup_tc(dev, TC_SETUP_QDISC_MQ, &opt);
+}
+
 static void mq_destroy(struct Qdisc *sch)
 {
        struct net_device *dev = qdisc_dev(sch);
        struct mq_sched *priv = qdisc_priv(sch);
        unsigned int ntx;
 
+       mq_offload(sch, TC_MQ_DESTROY);
+
        if (!priv->qdiscs)
                return;
        for (ntx = 0; ntx < dev->num_tx_queues && priv->qdiscs[ntx]; ntx++)
@@ -70,6 +87,8 @@ static int mq_init(struct Qdisc *sch, struct nlattr *opt,
        }
 
        sch->flags |= TCQ_F_MQROOT;
+
+       mq_offload(sch, TC_MQ_CREATE);
        return 0;
 }