[RFC PATCH 01/17] net: sched: cleanup qdisc_run and __qdisc_run semantics

[John Fastabend]

Currently __qdisc_run calls qdisc_run_end() but does not call
qdisc_run_begin(). This makes it hard to track pairs of
qdisc_run_{begin,end} across function calls.

To simplify reading these code paths this patch moves begin/end calls
into qdisc_run().

Acked-by: Eric Dumazet <eduma...@google.com>
Signed-off-by: John Fastabend <john.fastab...@gmail.com>
---
 0 files changed

diff --git a/include/net/pkt_sched.h b/include/net/pkt_sched.h
index d1f413f..4eea719 100644
--- a/include/net/pkt_sched.h
+++ b/include/net/pkt_sched.h
@@ -113,8 +113,10 @@ int sch_direct_xmit(struct sk_buff *skb, struct Qdisc *q,
 
 static inline void qdisc_run(struct Qdisc *q)
 {
-       if (qdisc_run_begin(q))
+       if (qdisc_run_begin(q)) {
                __qdisc_run(q);
+               qdisc_run_end(q);
+       }
 }
 
 static inline __be16 tc_skb_protocol(const struct sk_buff *skb)
diff --git a/net/core/dev.c b/net/core/dev.c
index 30b5fe3..9f5fa78 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -3209,9 +3209,9 @@ static inline int __dev_xmit_skb(struct sk_buff *skb, 
struct Qdisc *q,
                                contended = false;
                        }
                        __qdisc_run(q);
-               } else
-                       qdisc_run_end(q);
+               }
 
+               qdisc_run_end(q);
                rc = NET_XMIT_SUCCESS;
        } else {
                rc = q->enqueue(skb, q, &to_free) & NET_XMIT_MASK;
@@ -3221,6 +3221,7 @@ static inline int __dev_xmit_skb(struct sk_buff *skb, 
struct Qdisc *q,
                                contended = false;
                        }
                        __qdisc_run(q);
+                       qdisc_run_end(q);
                }
        }
        spin_unlock(root_lock);
diff --git a/net/sched/sch_generic.c b/net/sched/sch_generic.c
index 3839cbb..f6803e1 100644
--- a/net/sched/sch_generic.c
+++ b/net/sched/sch_generic.c
@@ -266,8 +266,6 @@ void __qdisc_run(struct Qdisc *q)
                        break;
                }
        }
-
-       qdisc_run_end(q);
 }
 
 unsigned long dev_trans_start(struct net_device *dev)