From mboxrd@z Thu Jan  1 00:00:00 1970
From: Felix Fietkau <nbd@openwrt.org>
Date: Sun, 23 Feb 2014 09:34:28 +0100
Subject: [ath9k-devel] Slow connection when using eduroam (AR9285)
In-Reply-To: <CAJxG8srxXM6yQy9AKmmQx81_OaV5b2oAzQ-rjue6Sa=z2-h9eg@mail.gmail.com>
References: <CAJxG8sqa4sSAO4exZMae3DyyEfGERjftVTZWHyaQwevKs84AmA@mail.gmail.com>	<53038465.1070408@rempel-privat.de>	<CAJxG8srmgxcsfkSZ-f7Zp8SazqJQUWOdBFNxY0yUZn72dzNnZw@mail.gmail.com>	<21252.9623.588940.444889@gargle.gargle.HOWL>	<CAJxG8spLmdaPta7QjajVbO=UrX5wBfDT5nLTpL4KZ2vKjcdnCw@mail.gmail.com>	<CAJxG8sp-wz_5cDU7cfcnRsnoMTPEunmHh4kD9LLJOaABbC45Zw@mail.gmail.com>	<530866F7.3050107@rempel-privat.de>	<CAJxG8sqQHzjN-qVfiGj_bS8e8UBGB=KGtLYb-iWRGSSxdVsiGA@mail.gmail.com>	<5308A99D.5060506@rempel-privat.de>	<CAJxG8sp0nYOCbu2L3xvOb_w3Cw=8H1XisnME2zW6tZ4EBDqy9g@mail.gmail.com>	<5308CA10.5060007@rempel-privat.de>	<CAJxG8spYQAiFWTagc+g9Zsg01P9ftXf_Zd9GA=OS4y0tKvjq3A@mail.gmail.com>	<5308F462.3090509@rempel-privat.de>
	<CAJxG8srxXM6yQy9AKmmQx81_OaV5b2oAzQ-rjue6Sa=z2-h9eg@mail.gmail.com>
Message-ID: <5309B294.7090407@openwrt.org>
List-Id: <ath9k-devel.lists.ath9k.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: ath9k-devel@lists.ath9k.org

On 2014-02-23 04:08, Marco Andr? Dinis wrote:
> (After some hours :D ) I found the first bad commit
> 
> 
>     ? wireless-testing $ git bisect good 
> 
>     723e711356b5a8a95728a890e254e8b0d47b55cf is the first bad commit
>     commit 723e711356b5a8a95728a890e254e8b0d47b55cf
>     Author: Felix Fietkau <nbd at openwrt.org <mailto:nbd@openwrt.org>>
>     Date:   Mon Apr 8 00:04:11 2013 +0200
>         ath9k: fix handling of broken descriptors

Please apply this patch and show me how often the printk triggers.

---
diff --git a/drivers/net/wireless/ath/ath9k/recv.c b/drivers/net/wireless/ath/ath9k/recv.c
index 076dae1..3c86d08 100644
--- a/drivers/net/wireless/ath/ath9k/recv.c
+++ b/drivers/net/wireless/ath/ath9k/recv.c
@@ -794,8 +794,11 @@ static int ath9k_rx_skb_preprocess(struct ath_softc *sc,
 	 * ath_get_next_rx_buf().
 	 */
 	sc->rx.discard_next = rx_stats->rs_more;
-	if (discard_current)
+	if (discard_current) {
+		if (net_ratelimit())
+			ath_printk(KERN_INFO, common, "discard current packet, more: %d\n", rx_stats->rs_more);
 		return -EINVAL;
+	}
 
 	/*
 	 * Discard zero-length packets.