From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from sabertooth01.qualcomm.com ([65.197.215.72]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1YB2mF-0000ng-Aa for ath10k@lists.infradead.org; Tue, 13 Jan 2015 14:49:03 +0000 From: Kalle Valo Subject: Re: [PATCH] ath10k: Make HTT fill size configurable References: <1420432796-10765-1-git-send-email-sujith@msujith.org> <54AD103B.40002@qti.qualcomm.com> Date: Tue, 13 Jan 2015 16:48:29 +0200 In-Reply-To: <54AD103B.40002@qti.qualcomm.com> (Vasanthakumar Thiagarajan's message of "Wed, 7 Jan 2015 16:23:47 +0530") Message-ID: <87vbka4tz6.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: Vasanthakumar Thiagarajan Cc: Sujith Manoharan , linux-wireless , Michal Kazior , "ath10k@lists.infradead.org" Vasanthakumar Thiagarajan writes: >>> This patch adds a module parameter to adjust the fill size >>> based on the platform/usage. >> >> I don't think this should be adjustable. > > Default value of 16 causes rx ring corruption within 1 hour of stress > testing. Increasing this limit to 1000 solves the issue, not sure what > max value in > 16 and <= 1000 works. Does the patch below fix that issue? Or is this issue still open? 5de6dfc82f71 ath10k: Fix potential Rx ring corruption -- Kalle Valo _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from sabertooth01.qualcomm.com ([65.197.215.72]:28450 "EHLO sabertooth01.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751052AbbAMOsm (ORCPT ); Tue, 13 Jan 2015 09:48:42 -0500 From: Kalle Valo To: Vasanthakumar Thiagarajan CC: Michal Kazior , Sujith Manoharan , linux-wireless , "ath10k@lists.infradead.org" Subject: Re: [PATCH] ath10k: Make HTT fill size configurable References: <1420432796-10765-1-git-send-email-sujith@msujith.org> <54AD103B.40002@qti.qualcomm.com> Date: Tue, 13 Jan 2015 16:48:29 +0200 In-Reply-To: <54AD103B.40002@qti.qualcomm.com> (Vasanthakumar Thiagarajan's message of "Wed, 7 Jan 2015 16:23:47 +0530") Message-ID: <87vbka4tz6.fsf@kamboji.qca.qualcomm.com> (sfid-20150113_154846_151517_305FEFF5) MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: linux-wireless-owner@vger.kernel.org List-ID: Vasanthakumar Thiagarajan writes: >>> This patch adds a module parameter to adjust the fill size >>> based on the platform/usage. >> >> I don't think this should be adjustable. > > Default value of 16 causes rx ring corruption within 1 hour of stress > testing. Increasing this limit to 1000 solves the issue, not sure what > max value in > 16 and <= 1000 works. Does the patch below fix that issue? Or is this issue still open? 5de6dfc82f71 ath10k: Fix potential Rx ring corruption -- Kalle Valo