Shouldn't you now also revert the commits from https://git.openwrt.org/?p=openwrt/staging/dangole.git;a=commitdiff;h=de1c58a64bd66319e770d2587da07d8c9c90174a since they caused throughput regression? On 20/12/2018, Stanislaw Gruszka <sgruszka@xxxxxxxxxx> wrote: > For unknown reasons printk() on some context can cause CPU hung on > embedded MT7620 AP/router MIPS platforms. What can result on wifi > disconnects. > > This patch move queue full messages to debug level what is consistent > with other mac80211 drivers which drop packet silently if tx queue is > full. This make MT7620 OpenWRT routers more stable, what was reported > by various users. > > Signed-off-by: Stanislaw Gruszka <sgruszka@xxxxxxxxxx> > --- > drivers/net/wireless/ralink/rt2x00/rt2x00queue.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/ralink/rt2x00/rt2x00queue.c > b/drivers/net/wireless/ralink/rt2x00/rt2x00queue.c > index 92ddc19e7bf7..947fc8964e9a 100644 > --- a/drivers/net/wireless/ralink/rt2x00/rt2x00queue.c > +++ b/drivers/net/wireless/ralink/rt2x00/rt2x00queue.c > @@ -671,7 +671,7 @@ int rt2x00queue_write_tx_frame(struct data_queue *queue, > struct sk_buff *skb, > spin_lock(&queue->tx_lock); > > if (unlikely(rt2x00queue_full(queue))) { > - rt2x00_err(queue->rt2x00dev, "Dropping frame due to full tx queue %d\n", > + rt2x00_dbg(queue->rt2x00dev, "Dropping frame due to full tx queue %d\n", > queue->qid); > ret = -ENOBUFS; > goto out; > -- > 2.7.5 > >
