diff options
| author |   Eric Dumazet <edumazet@google.com> | 2016-06-10 16:41:37 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-06-10 23:58:20 -0700 |
| commit | cca605dd4b3b2bfa381250b7dbbe16b124916f24 (patch) | |
| tree | a1dde5e043514772fcb539d1b76d7d2567d952de /net/sched | |
| parent | net_sched: sch_plug: use a private throttled status (diff) | |
| download | linux-dev-cca605dd4b3b2bfa381250b7dbbe16b124916f24.tar.xz linux-dev-cca605dd4b3b2bfa381250b7dbbe16b124916f24.zip | |
net_sched: cbq: remove a flaky use of qdisc_is_throttled()
So far no qdisc ever unset the throttled bit at enqueue() time,
so CBQ usage of qdisc_is_throttled() was flaky.
Since __QDISC_STATE_THROTTLED set/unset is way too expensive
considering that only CBQ was eventually caring for this status,
it would make sense to implement a Qdisc ops ->is_throttled()
if we find that this is needed.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
| -rw-r--r-- | net/sched/sch_cbq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/sch_cbq.c b/net/sched/sch_cbq.c index f2af31be6370..6e61f9aa8783 100644 --- a/net/sched/sch_cbq.c +++ b/net/sched/sch_cbq.c @@ -345,7 +345,7 @@ cbq_mark_toplevel(struct cbq_sched_data *q, struct cbq_class *cl) { int toplevel = q->toplevel; - if (toplevel > cl->level && !(qdisc_is_throttled(cl->q))) { + if (toplevel > cl->level) { psched_time_t now = psched_get_time(); do { |