net_sched: add the ability to defer skb freeing
authorEric Dumazet <edumazet@google.com>
Tue, 14 Jun 2016 03:21:50 +0000 (20:21 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 15 Jun 2016 21:08:34 +0000 (14:08 -0700)
qdisc are changed under RTNL protection and often
while blocking BH and root qdisc spinlock.

When lots of skbs need to be dropped, we free
them under these locks causing TX/RX freezes,
and more generally latency spikes.

This commit adds rtnl_kfree_skbs(), used to queue
skbs for deferred freeing.

Actual freeing happens right after RTNL is released,
with appropriate scheduling points.

rtnl_qdisc_drop() can also be used in place
of disc_drop() when RTNL is held.

qdisc_reset_queue() and __qdisc_reset_queue() get
the new behavior, so standard qdiscs like pfifo, pfifo_fast...
have their ->reset() method automatically handled.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/rtnetlink.h
include/net/sch_generic.h
net/core/rtnetlink.c
net/sched/sch_generic.c

index c006cc900c44540deb1ca9004d14106a5e8629e6..2daece8979f75a953c24fdde0fabeff81f039fad 100644 (file)
@@ -89,8 +89,9 @@ void net_inc_egress_queue(void);
 void net_dec_egress_queue(void);
 #endif
 
-extern void rtnetlink_init(void);
-extern void __rtnl_unlock(void);
+void rtnetlink_init(void);
+void __rtnl_unlock(void);
+void rtnl_kfree_skbs(struct sk_buff *head, struct sk_buff *tail);
 
 #define ASSERT_RTNL() do { \
        if (unlikely(!rtnl_is_locked())) { \
index 9a0d177884c6472a06cb9670da9b5561e387376e..4f7cee8344c4c2180ee24de421da392ac1ce36d4 100644 (file)
@@ -683,19 +683,21 @@ static inline struct sk_buff *qdisc_dequeue_peeked(struct Qdisc *sch)
        return skb;
 }
 
-static inline void __qdisc_reset_queue(struct Qdisc *sch,
-                                      struct sk_buff_head *list)
+static inline void __qdisc_reset_queue(struct sk_buff_head *list)
 {
        /*
         * We do not know the backlog in bytes of this list, it
         * is up to the caller to correct it
         */
-       __skb_queue_purge(list);
+       if (!skb_queue_empty(list)) {
+               rtnl_kfree_skbs(list->next, list->prev);
+               __skb_queue_head_init(list);
+       }
 }
 
 static inline void qdisc_reset_queue(struct Qdisc *sch)
 {
-       __qdisc_reset_queue(sch, &sch->q);
+       __qdisc_reset_queue(&sch->q);
        sch->qstats.backlog = 0;
 }
 
@@ -716,6 +718,12 @@ static inline struct Qdisc *qdisc_replace(struct Qdisc *sch, struct Qdisc *new,
        return old;
 }
 
+static inline void rtnl_qdisc_drop(struct sk_buff *skb, struct Qdisc *sch)
+{
+       rtnl_kfree_skbs(skb, skb);
+       qdisc_qstats_drop(sch);
+}
+
 static inline int qdisc_drop(struct sk_buff *skb, struct Qdisc *sch)
 {
        kfree_skb(skb);
index d69c4644f8f2ca5a4e1bda711672af2dafcf604c..eb49ca24274abc9238f6594f681cb9999543c1ec 100644 (file)
@@ -71,9 +71,31 @@ void rtnl_lock(void)
 }
 EXPORT_SYMBOL(rtnl_lock);
 
+static struct sk_buff *defer_kfree_skb_list;
+void rtnl_kfree_skbs(struct sk_buff *head, struct sk_buff *tail)
+{
+       if (head && tail) {
+               tail->next = defer_kfree_skb_list;
+               defer_kfree_skb_list = head;
+       }
+}
+EXPORT_SYMBOL(rtnl_kfree_skbs);
+
 void __rtnl_unlock(void)
 {
+       struct sk_buff *head = defer_kfree_skb_list;
+
+       defer_kfree_skb_list = NULL;
+
        mutex_unlock(&rtnl_mutex);
+
+       while (head) {
+               struct sk_buff *next = head->next;
+
+               kfree_skb(head);
+               cond_resched();
+               head = next;
+       }
 }
 
 void rtnl_unlock(void)
index 0c9cb516f2e31c3a1ddaa42e3474656b51548e91..773b632e1e333f7b2492b6d288599266d6f71c3e 100644 (file)
@@ -493,7 +493,7 @@ static void pfifo_fast_reset(struct Qdisc *qdisc)
        struct pfifo_fast_priv *priv = qdisc_priv(qdisc);
 
        for (prio = 0; prio < PFIFO_FAST_BANDS; prio++)
-               __qdisc_reset_queue(qdisc, band2list(priv, prio));
+               __qdisc_reset_queue(band2list(priv, prio));
 
        priv->bitmap = 0;
        qdisc->qstats.backlog = 0;