summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorYannick Vignon <yannick.vignon@nxp.com>2021-07-30 18:53:21 +0200
committerDavid S. Miller <davem@davemloft.net>2021-08-02 12:47:35 +0200
commitebca25ead0711729e0aeeec45062e7ac4df3e158 (patch)
treeb44f5924efbb951631eebccbe98711a7a8291a41 /net
parentnet, gro: Set inner transport header offset in tcp/udp GRO hook (diff)
downloadlinux-ebca25ead0711729e0aeeec45062e7ac4df3e158.tar.xz
linux-ebca25ead0711729e0aeeec45062e7ac4df3e158.zip
net/sched: taprio: Fix init procedure
Commit 13511704f8d759 ("net: taprio offload: enforce qdisc to netdev queue mapping") resulted in duplicate entries in the qdisc hash. While this did not impact the overall operation of the qdisc and taprio code paths, it did result in an infinite loop when dumping the qdisc properties, at least on one target (NXP LS1028 ARDB). Removing the duplicate call to qdisc_hash_add() solves the problem. Fixes: 13511704f8d759 ("net: taprio offload: enforce qdisc to netdev queue mapping") Signed-off-by: Yannick Vignon <yannick.vignon@nxp.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/sched/sch_taprio.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/net/sched/sch_taprio.c b/net/sched/sch_taprio.c
index 07b30d0601d7..9c79374457a0 100644
--- a/net/sched/sch_taprio.c
+++ b/net/sched/sch_taprio.c
@@ -1739,8 +1739,6 @@ static void taprio_attach(struct Qdisc *sch)
if (FULL_OFFLOAD_IS_ENABLED(q->flags)) {
qdisc->flags |= TCQ_F_ONETXQUEUE | TCQ_F_NOPARENT;
old = dev_graft_qdisc(qdisc->dev_queue, qdisc);
- if (ntx < dev->real_num_tx_queues)
- qdisc_hash_add(qdisc, false);
} else {
old = dev_graft_qdisc(qdisc->dev_queue, sch);
qdisc_refcount_inc(sch);