net_sch: red: Add offload ability to RED qdisc
authorNogah Frankel <nogahf@mellanox.com>
Mon, 6 Nov 2017 06:23:41 +0000 (07:23 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 8 Nov 2017 03:23:37 +0000 (12:23 +0900)
Add the ability to offload RED qdisc by using ndo_setup_tc.
There are four commands for RED offloading:
* TC_RED_SET: handles set and change.
* TC_RED_DESTROY: handle qdisc destroy.
* TC_RED_STATS: update the qdiscs counters (given as reference)
* TC_RED_XSTAT: returns red xstats.

Whether RED is being offloaded is being determined every time dump action
is being called because parent change of this qdisc could change its
offload state but doesn't require any RED function to be called.

Signed-off-by: Nogah Frankel <nogahf@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netdevice.h
include/net/pkt_cls.h
include/uapi/linux/pkt_sched.h
net/sched/sch_red.c

index fda527ccb2637154e004008f4519ca0bc1d2ffda..71968a2ca9f39062782a6d067bfe71f82e68a2b1 100644 (file)
@@ -777,6 +777,7 @@ enum tc_setup_type {
        TC_SETUP_CLSBPF,
        TC_SETUP_BLOCK,
        TC_SETUP_CBS,
+       TC_SETUP_QDISC_RED,
 };
 
 /* These structures hold the attributes of bpf state that are being passed
index 98fef322122763e2c98b06d8dc54fbc50531bc37..03c208d3c9224c3da45c1aa60d5b83d3431038d7 100644 (file)
@@ -703,4 +703,34 @@ struct tc_cookie {
        u8  *data;
        u32 len;
 };
+
+enum tc_red_command {
+       TC_RED_REPLACE,
+       TC_RED_DESTROY,
+       TC_RED_STATS,
+       TC_RED_XSTATS,
+};
+
+struct tc_red_qopt_offload_params {
+       u32 min;
+       u32 max;
+       u32 probability;
+       bool is_ecn;
+};
+struct tc_red_qopt_offload_stats {
+       struct gnet_stats_basic_packed *bstats;
+       struct gnet_stats_queue *qstats;
+};
+
+struct tc_red_qopt_offload {
+       enum tc_red_command command;
+       u32 handle;
+       u32 parent;
+       union {
+               struct tc_red_qopt_offload_params set;
+               struct tc_red_qopt_offload_stats stats;
+               struct red_stats *xstats;
+       };
+};
+
 #endif
index 5002562868ccf896be8702bec73ef3932c4222ee..6a2c5ea7e9c4e04264ddd80de963dc7ba1f8fede 100644 (file)
@@ -256,6 +256,7 @@ struct tc_red_qopt {
 #define TC_RED_ECN             1
 #define TC_RED_HARDDROP                2
 #define TC_RED_ADAPTATIVE      4
+#define TC_RED_OFFLOADED       8
 };
 
 struct tc_red_xstats {
index fdfdb56aaae263e4042e1e343f9d36fd8cc105cc..007dd8ef8aacdfa7249422a91b74d2724a5c2c7a 100644 (file)
@@ -19,6 +19,7 @@
 #include <linux/kernel.h>
 #include <linux/skbuff.h>
 #include <net/pkt_sched.h>
+#include <net/pkt_cls.h>
 #include <net/inet_ecn.h>
 #include <net/red.h>
 
@@ -148,11 +149,37 @@ static void red_reset(struct Qdisc *sch)
        red_restart(&q->vars);
 }
 
+static int red_offload(struct Qdisc *sch, bool enable)
+{
+       struct red_sched_data *q = qdisc_priv(sch);
+       struct net_device *dev = qdisc_dev(sch);
+       struct tc_red_qopt_offload opt = {
+               .handle = sch->handle,
+               .parent = sch->parent,
+       };
+
+       if (!tc_can_offload(dev) || !dev->netdev_ops->ndo_setup_tc)
+               return -EOPNOTSUPP;
+
+       if (enable) {
+               opt.command = TC_RED_REPLACE;
+               opt.set.min = q->parms.qth_min >> q->parms.Wlog;
+               opt.set.max = q->parms.qth_max >> q->parms.Wlog;
+               opt.set.probability = q->parms.max_P;
+               opt.set.is_ecn = red_use_ecn(q);
+       } else {
+               opt.command = TC_RED_DESTROY;
+       }
+
+       return dev->netdev_ops->ndo_setup_tc(dev, TC_SETUP_QDISC_RED, &opt);
+}
+
 static void red_destroy(struct Qdisc *sch)
 {
        struct red_sched_data *q = qdisc_priv(sch);
 
        del_timer_sync(&q->adapt_timer);
+       red_offload(sch, false);
        qdisc_destroy(q->qdisc);
 }
 
@@ -219,6 +246,7 @@ static int red_change(struct Qdisc *sch, struct nlattr *opt)
                red_start_of_idle_period(&q->vars);
 
        sch_tree_unlock(sch);
+       red_offload(sch, true);
        return 0;
 }
 
@@ -244,6 +272,33 @@ static int red_init(struct Qdisc *sch, struct nlattr *opt)
        return red_change(sch, opt);
 }
 
+static int red_dump_offload(struct Qdisc *sch, struct tc_red_qopt *opt)
+{
+       struct net_device *dev = qdisc_dev(sch);
+       struct tc_red_qopt_offload hw_stats = {
+               .handle = sch->handle,
+               .parent = sch->parent,
+               .command = TC_RED_STATS,
+               .stats.bstats = &sch->bstats,
+               .stats.qstats = &sch->qstats,
+       };
+       int err;
+
+       opt->flags &= ~TC_RED_OFFLOADED;
+       if (!tc_can_offload(dev) || !dev->netdev_ops->ndo_setup_tc)
+               return 0;
+
+       err = dev->netdev_ops->ndo_setup_tc(dev, TC_SETUP_QDISC_RED,
+                                           &hw_stats);
+       if (err == -EOPNOTSUPP)
+               return 0;
+
+       if (!err)
+               opt->flags |= TC_RED_OFFLOADED;
+
+       return err;
+}
+
 static int red_dump(struct Qdisc *sch, struct sk_buff *skb)
 {
        struct red_sched_data *q = qdisc_priv(sch);
@@ -257,8 +312,13 @@ static int red_dump(struct Qdisc *sch, struct sk_buff *skb)
                .Plog           = q->parms.Plog,
                .Scell_log      = q->parms.Scell_log,
        };
+       int err;
 
        sch->qstats.backlog = q->qdisc->qstats.backlog;
+       err = red_dump_offload(sch, &opt);
+       if (err)
+               goto nla_put_failure;
+
        opts = nla_nest_start(skb, TCA_OPTIONS);
        if (opts == NULL)
                goto nla_put_failure;
@@ -275,6 +335,7 @@ nla_put_failure:
 static int red_dump_stats(struct Qdisc *sch, struct gnet_dump *d)
 {
        struct red_sched_data *q = qdisc_priv(sch);
+       struct net_device *dev = qdisc_dev(sch);
        struct tc_red_xstats st = {
                .early  = q->stats.prob_drop + q->stats.forced_drop,
                .pdrop  = q->stats.pdrop,
@@ -282,6 +343,24 @@ static int red_dump_stats(struct Qdisc *sch, struct gnet_dump *d)
                .marked = q->stats.prob_mark + q->stats.forced_mark,
        };
 
+       if (tc_can_offload(dev) &&  dev->netdev_ops->ndo_setup_tc) {
+               struct red_stats hw_stats = {0};
+               struct tc_red_qopt_offload hw_stats_request = {
+                       .handle = sch->handle,
+                       .parent = sch->parent,
+                       .command = TC_RED_XSTATS,
+                       .xstats = &hw_stats,
+               };
+               if (!dev->netdev_ops->ndo_setup_tc(dev,
+                                                  TC_SETUP_QDISC_RED,
+                                                  &hw_stats_request)) {
+                       st.early += hw_stats.prob_drop + hw_stats.forced_drop;
+                       st.pdrop += hw_stats.pdrop;
+                       st.other += hw_stats.other;
+                       st.marked += hw_stats.prob_mark + hw_stats.forced_mark;
+               }
+       }
+
        return gnet_stats_copy_app(d, &st, sizeof(st));
 }