From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]:51212 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750898Ab1BIGbj (ORCPT ); Wed, 9 Feb 2011 01:31:39 -0500 Date: Wed, 9 Feb 2011 07:31:37 +0100 From: Stanislaw Gruszka To: wwguy Cc: Intel Linux Wireless , "linux-wireless@vger.kernel.org" , "stable@kernel.org" Subject: Re: [PATCH] iwl3945: increase plcp check threshold Message-ID: <20110209063137.GD2184@redhat.com> References: <20110208090446.GA2490@redhat.com> <1297180040.20613.39.camel@wwguy-ubuntu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1297180040.20613.39.camel@wwguy-ubuntu> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, Feb 08, 2011 at 07:47:20AM -0800, wwguy wrote: > On Tue, 2011-02-08 at 01:04 -0800, Stanislaw Gruszka wrote: > > iwl3945: increase plcp check threshold > > > > Patch fixes: > > https://bugzilla.redhat.com/show_bug.cgi?id=654599 > > > > Many users report very low speed problem on 3945 devices, > > this patch fixes problem, but only for some of them. > > > > For unknown reason, sometimes after hw scanning, device is not able > > to receive frames at high rate. Since plcp health check may request > > hw scan to "reset radio", performance problem start to be observable > > after update kernel to .35, where plcp check was introduced. > > > > Bug reporter confirmed that increase plcp check threshold fixed > > problem for him. > > > > Reported-and-tested-by: SilvioTO > > Cc: stable@kernel.org # 2.6.35+ > > Signed-off-by: Stanislaw Gruszka > > --- > > Patch is oneliner, but does not apply on older kernel versions, > > I will provide -stable patches when fix will land in mainline > > > > diff --git a/drivers/net/wireless/iwlwifi/iwl-3945.c b/drivers/net/wireless/iwlwifi/iwl-3945.c > > index 58213e7..27faa5e 100644 > > --- a/drivers/net/wireless/iwlwifi/iwl-3945.c > > +++ b/drivers/net/wireless/iwlwifi/iwl-3945.c > > @@ -2775,7 +2775,7 @@ static struct iwl_base_params iwl3945_base_params = { > > .use_isr_legacy = true, > > .led_compensation = 64, > > .broken_powersave = true, > > - .plcp_delta_threshold = IWL_MAX_PLCP_ERR_LONG_THRESHOLD_DEF, > > + .plcp_delta_threshold = IWL_MAX_PLCP_ERR_THRESHOLD_MAX, > > .wd_timeout = IWL_DEF_WD_TIMEOUT, > > .max_event_log_size = 512, > > .tx_power_by_driver = true, > > Do you see any help for 3945 with plcp checking at all? Not really, I see only demage :-) Ok, if we argree to remove plcp check from 3945 at all, I will repost with slightly different patch for fix problem on stable. Stanislaw