023-rt2x00-do-not-print-error-when-queue-is-full.patch 1.8 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243
  1. From patchwork Tue Mar 12 09:51:43 2019
  2. Content-Type: text/plain; charset="utf-8"
  3. MIME-Version: 1.0
  4. Content-Transfer-Encoding: 7bit
  5. X-Patchwork-Submitter: Stanislaw Gruszka <[email protected]>
  6. X-Patchwork-Id: 10848963
  7. X-Patchwork-Delegate: [email protected]
  8. From: Stanislaw Gruszka <[email protected]>
  9. To: [email protected]
  10. Cc: =?utf-8?q?Tomislav_Po=C5=BEega?= <[email protected]>,
  11. Daniel Golle <[email protected]>, Felix Fietkau <[email protected]>,
  12. Mathias Kresin <[email protected]>
  13. Subject: [PATCH v3 4/4] rt2x00: do not print error when queue is full
  14. Date: Tue, 12 Mar 2019 10:51:43 +0100
  15. Message-Id: <[email protected]>
  16. In-Reply-To: <[email protected]>
  17. References: <[email protected]>
  18. For unknown reasons printk() on some context can cause CPU hung on
  19. embedded MT7620 AP/router MIPS platforms. What can result on wifi
  20. disconnects.
  21. This patch move queue full messages to debug level what is consistent
  22. with other mac80211 drivers which drop packet silently if tx queue is
  23. full. This make MT7620 OpenWRT routers more stable, what was reported
  24. by various users.
  25. Signed-off-by: Stanislaw Gruszka <[email protected]>
  26. ---
  27. drivers/net/wireless/ralink/rt2x00/rt2x00queue.c | 2 +-
  28. 1 file changed, 1 insertion(+), 1 deletion(-)
  29. --- a/drivers/net/wireless/ralink/rt2x00/rt2x00queue.c
  30. +++ b/drivers/net/wireless/ralink/rt2x00/rt2x00queue.c
  31. @@ -671,7 +671,7 @@ int rt2x00queue_write_tx_frame(struct da
  32. spin_lock(&queue->tx_lock);
  33. if (unlikely(rt2x00queue_full(queue))) {
  34. - rt2x00_err(queue->rt2x00dev, "Dropping frame due to full tx queue %d\n",
  35. + rt2x00_dbg(queue->rt2x00dev, "Dropping frame due to full tx queue %d\n",
  36. queue->qid);
  37. ret = -ENOBUFS;
  38. goto out;