From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleksij Rempel Date: Tue, 03 Feb 2015 15:35:25 +0100 Subject: [ath9k-devel] [PATCH] ath9k_htc: add adaptive usb flow control to repair soft lockup with monitor mode In-Reply-To: <87fvanxldu.fsf@kamboji.qca.qualcomm.com> References: <1422957585-19736-1-git-send-email-yuweizheng@139.com> <87fvanxldu.fsf@kamboji.qca.qualcomm.com> Message-ID: <54D0DCAD.8060409@rempel-privat.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ath9k-devel@lists.ath9k.org It seems to work without noticeable regressions. Am 03.02.2015 um 12:52 schrieb Kalle Valo: > yuweizheng at 139.com writes: > >> From: Yuwei Zheng >> >> In the environment with heavy wifi traffic, set the ar9271 into monitor mode, will >> trigger a deadloop panic. >> >> The ath9k_hif_usb_rx_cb function excute on the interrupt context, and ath9k_rx_tasklet excute >> on the soft irq context. In other words, the ath9k_hif_usb_rx_cb have more chance to excute than >> ath9k_rx_tasklet. So in the worst condition, the rx.rxbuf receive list is always full, >> and the do {}while(true) loop will not be break. The kernel get a soft lockup panic. > > The word wrapping is still wrong, please limit the width to 72 chars or > so. But no need to wrap log snippets, they can exceed the limit. > >> Signed-off-by: Yuwei Zheng >> Signed-off-by: Yuwei Zheng > > Why two of these? Please use just one Signed-off-by line. Unless these > are two different persons with the same name :) > -- Regards, Oleksij -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 213 bytes Desc: OpenPGP digital signature Url : http://lists.ath9k.org/pipermail/ath9k-devel/attachments/20150203/698d37d2/attachment.pgp