diff options
author | Andrea Parri <parri.andrea@gmail.com> | 2015-06-16 18:16:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-06-21 12:47:24 -0400 |
commit | a55e1c5c2640549ff3beb843f7ecaa242d69c351 (patch) | |
tree | cd48a1cd740e1c4d3fc58eb88a3c63d9a4c64613 /net/sched | |
parent | 68946159da1b0b6791c5990242940950b9383cfc (diff) |
pkt_sched: sch_qfq: remove redundant -if- control statement
The control !hlist_unhashed() in qfq_destroy_agg() is unnecessary
because already performed in hlist_del_init(), so remove it.
Signed-off-by: Andrea Parri <parri.andrea@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
-rw-r--r-- | net/sched/sch_qfq.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/sched/sch_qfq.c b/net/sched/sch_qfq.c index 3ec7e88a43ca..b8d73bca683c 100644 --- a/net/sched/sch_qfq.c +++ b/net/sched/sch_qfq.c | |||
@@ -339,8 +339,7 @@ static struct qfq_aggregate *qfq_choose_next_agg(struct qfq_sched *); | |||
339 | 339 | ||
340 | static void qfq_destroy_agg(struct qfq_sched *q, struct qfq_aggregate *agg) | 340 | static void qfq_destroy_agg(struct qfq_sched *q, struct qfq_aggregate *agg) |
341 | { | 341 | { |
342 | if (!hlist_unhashed(&agg->nonfull_next)) | 342 | hlist_del_init(&agg->nonfull_next); |
343 | hlist_del_init(&agg->nonfull_next); | ||
344 | q->wsum -= agg->class_weight; | 343 | q->wsum -= agg->class_weight; |
345 | if (q->wsum != 0) | 344 | if (q->wsum != 0) |
346 | q->iwsum = ONE_FP / q->wsum; | 345 | q->iwsum = ONE_FP / q->wsum; |