Return-path: Received: from nbd.name ([46.4.11.11]:39611 "EHLO nbd.name" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752384AbaDVJJV (ORCPT ); Tue, 22 Apr 2014 05:09:21 -0400 Message-ID: <535631B7.7050208@openwrt.org> (sfid-20140422_110926_839481_2A1EAC93) Date: Tue, 22 Apr 2014 11:09:11 +0200 From: Felix Fietkau MIME-Version: 1.0 To: Tim Harvey , linux-wireless@vger.kernel.org CC: linville@tuxdriver.com Subject: Re: [PATCH 2/2] ath9k: add a recv budget References: <1398122097-23544-1-git-send-email-tharvey@gateworks.com> <1398122097-23544-2-git-send-email-tharvey@gateworks.com> In-Reply-To: <1398122097-23544-2-git-send-email-tharvey@gateworks.com> Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 2014-04-22 01:14, Tim Harvey wrote: > Implement a recv budget so that in cases of high traffic we still allow other > taskets to get processed. > > Without this, we can encounter a host of issues during high wireless traffic > reception depending on system load including rcu stall's detected (ARM), > soft lockups, failure to service critical tasks such as watchdog resets, > and triggering of the tx stuck tasklet. > > The same thing was proposed previously by Ben: > http://www.spinics.net/lists/linux-wireless/msg112891.html > > The only difference here is that I make sure only processed packets are counted > in the budget by checking at the end of the rx loop. > > Signed-off-by: Tim Harvey For both patches: Acked-by: Felix Fietkau