[PATCH net-next 01/15] sch_htb: Remove local SKB queue handling code.

[David Miller]

Instead, adjust __qdisc_enqueue_tail() such that HTB can use it
instead.

The only other caller of __qdisc_enqueue_tail() is
qdisc_enqueue_tail() so we can move the backlog and return value
handling (which HTB doesn't need/want) to the latter.

Signed-off-by: David S. Miller <da...@davemloft.net>
---
 include/net/sch_generic.h | 11 +++++------
 net/sched/sch_htb.c       | 18 +-----------------
 2 files changed, 6 insertions(+), 23 deletions(-)

diff --git a/include/net/sch_generic.h b/include/net/sch_generic.h
index a6d00093f35e..bc8f6b0b6610 100644
--- a/include/net/sch_generic.h
+++ b/include/net/sch_generic.h
@@ -828,8 +828,8 @@ static inline void qdisc_skb_head_init(struct 
qdisc_skb_head *qh)
        qh->qlen = 0;
 }
 
-static inline int __qdisc_enqueue_tail(struct sk_buff *skb, struct Qdisc *sch,
-                                      struct qdisc_skb_head *qh)
+static inline void __qdisc_enqueue_tail(struct sk_buff *skb,
+                                       struct qdisc_skb_head *qh)
 {
        struct sk_buff *last = qh->tail;
 
@@ -842,14 +842,13 @@ static inline int __qdisc_enqueue_tail(struct sk_buff 
*skb, struct Qdisc *sch,
                qh->head = skb;
        }
        qh->qlen++;
-       qdisc_qstats_backlog_inc(sch, skb);
-
-       return NET_XMIT_SUCCESS;
 }
 
 static inline int qdisc_enqueue_tail(struct sk_buff *skb, struct Qdisc *sch)
 {
-       return __qdisc_enqueue_tail(skb, sch, &sch->q);
+       __qdisc_enqueue_tail(skb, &sch->q);
+       qdisc_qstats_backlog_inc(sch, skb);
+       return NET_XMIT_SUCCESS;
 }
 
 static inline struct sk_buff *__qdisc_dequeue_head(struct qdisc_skb_head *qh)
diff --git a/net/sched/sch_htb.c b/net/sched/sch_htb.c
index 43c4bfe625a9..cf23829cbede 100644
--- a/net/sched/sch_htb.c
+++ b/net/sched/sch_htb.c
@@ -577,22 +577,6 @@ static inline void htb_deactivate(struct htb_sched *q, 
struct htb_class *cl)
        cl->prio_activity = 0;
 }
 
-static void htb_enqueue_tail(struct sk_buff *skb, struct Qdisc *sch,
-                            struct qdisc_skb_head *qh)
-{
-       struct sk_buff *last = qh->tail;
-
-       if (last) {
-               skb->next = NULL;
-               last->next = skb;
-               qh->tail = skb;
-       } else {
-               qh->tail = skb;
-               qh->head = skb;
-       }
-       qh->qlen++;
-}
-
 static int htb_enqueue(struct sk_buff *skb, struct Qdisc *sch,
                       struct sk_buff **to_free)
 {
@@ -603,7 +587,7 @@ static int htb_enqueue(struct sk_buff *skb, struct Qdisc 
*sch,
        if (cl == HTB_DIRECT) {
                /* enqueue to helper queue */
                if (q->direct_queue.qlen < q->direct_qlen) {
-                       htb_enqueue_tail(skb, sch, &q->direct_queue);
+                       __qdisc_enqueue_tail(skb, &q->direct_queue);
                        q->direct_pkts++;
                } else {
                        return qdisc_drop(skb, sch, to_free);
-- 
2.17.1