From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from sabertooth01.qualcomm.com ([65.197.215.72]:35229 "EHLO sabertooth01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753082Ab3H1LEG (ORCPT ); Wed, 28 Aug 2013 07:04:06 -0400 From: Kalle Valo To: Michal Kazior CC: , linux-wireless Subject: Re: [PATCH v2 1/4] ath10k: synchronize tx/rx reporting to mac80211 References: <1377066854-13981-1-git-send-email-michal.kazior@tieto.com> <1377507205-5386-1-git-send-email-michal.kazior@tieto.com> <1377507205-5386-2-git-send-email-michal.kazior@tieto.com> <87txiajveo.fsf@kamboji.qca.qualcomm.com> Date: Wed, 28 Aug 2013 14:04:01 +0300 In-Reply-To: (Michal Kazior's message of "Wed, 28 Aug 2013 12:54:46 +0200") Message-ID: <87bo4igjq6.fsf@kamboji.qca.qualcomm.com> (sfid-20130828_130411_666364_2E36813E) MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: linux-wireless-owner@vger.kernel.org List-ID: Michal Kazior writes: > On 28 August 2013 06:23, Kalle Valo wrote: >> Michal Kazior writes: >> >>> According to mac80211 docs tx and rx reporting >>> routines must not run concurrently unless they are >>> _irqsafe variants. >>> >>> This doesn't fix any visible bug but is apparently >>> the right thing to do as per the documentation. >>> >>> Signed-off-by: Michal Kazior >> >> I had a quick chat with Johannes and Sujith about this. The concurrency >> requirements are for the STA PS race in AP mode (see >> ieee80211_handle_filtered_frame()). >> >> I think we should drop this frame, revisit later and properly analyse "drop this frame". Heh, of course I was trying to say "drop this patch" :) >> how to fix the race. But it would be good add this to the todo list: >> >> http://wireless.kernel.org/en/users/Drivers/ath10k#TODO > > I've added this to the TODO list. Thanks! -- Kalle Valo From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from sabertooth02.qualcomm.com ([65.197.215.38]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VEdY4-0008Kx-Co for ath10k@lists.infradead.org; Wed, 28 Aug 2013 11:04:29 +0000 From: Kalle Valo Subject: Re: [PATCH v2 1/4] ath10k: synchronize tx/rx reporting to mac80211 References: <1377066854-13981-1-git-send-email-michal.kazior@tieto.com> <1377507205-5386-1-git-send-email-michal.kazior@tieto.com> <1377507205-5386-2-git-send-email-michal.kazior@tieto.com> <87txiajveo.fsf@kamboji.qca.qualcomm.com> Date: Wed, 28 Aug 2013 14:04:01 +0300 In-Reply-To: (Michal Kazior's message of "Wed, 28 Aug 2013 12:54:46 +0200") Message-ID: <87bo4igjq6.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Michal Kazior Cc: linux-wireless , ath10k@lists.infradead.org Michal Kazior writes: > On 28 August 2013 06:23, Kalle Valo wrote: >> Michal Kazior writes: >> >>> According to mac80211 docs tx and rx reporting >>> routines must not run concurrently unless they are >>> _irqsafe variants. >>> >>> This doesn't fix any visible bug but is apparently >>> the right thing to do as per the documentation. >>> >>> Signed-off-by: Michal Kazior >> >> I had a quick chat with Johannes and Sujith about this. The concurrency >> requirements are for the STA PS race in AP mode (see >> ieee80211_handle_filtered_frame()). >> >> I think we should drop this frame, revisit later and properly analyse "drop this frame". Heh, of course I was trying to say "drop this patch" :) >> how to fix the race. But it would be good add this to the todo list: >> >> http://wireless.kernel.org/en/users/Drivers/ath10k#TODO > > I've added this to the TODO list. Thanks! -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k