From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from charlotte.tuxdriver.com ([70.61.120.58]:38996 "EHLO smtp.tuxdriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752999Ab1AYPPK (ORCPT ); Tue, 25 Jan 2011 10:15:10 -0500 Date: Tue, 25 Jan 2011 10:14:46 -0500 From: "John W. Linville" To: Stanislaw Gruszka Cc: Wey-Yi Guy , Abhijeet Kolekar , linux-wireless@vger.kernel.org, Intel Linux Wireless Subject: Re: [RFC] iwl3945: remove check_plcp_health Message-ID: <20110125151445.GA2927@tuxdriver.com> References: <20110125093507.GA6429@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20110125093507.GA6429@redhat.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, Jan 25, 2011 at 10:35:28AM +0100, Stanislaw Gruszka wrote: > This patch helps some users currently complaining about iwl3945 > performance. > > https://bugzilla.redhat.com/show_bug.cgi?id=654599 > https://bugs.launchpad.net/ubuntu/maverick/+source/linux/+bug/621265/+index?comments=all > > Patch does not helps all users, but they seems to be affected > by different bug or bugs. > > Patch effectively reverts commit a29576a7844326c5223f4d4adbfd3f4d64173d4c > "iwl3945: add plcp error checking". It is minimal fix intended to > -stable posting. I will post cleaning up patch, if fix will be accepted. > > --- > drivers/net/wireless/iwlwifi/iwl-3945.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/wireless/iwlwifi/iwl-3945.c b/drivers/net/wireless/iwlwifi/iwl-3945.c > index 8cacb4e..eb916d7 100644 > --- a/drivers/net/wireless/iwlwifi/iwl-3945.c > +++ b/drivers/net/wireless/iwlwifi/iwl-3945.c > @@ -2832,7 +2832,6 @@ static struct iwl_lib_ops iwl3945_lib = { > .config_ap = iwl3945_config_ap, > .manage_ibss_station = iwl3945_manage_ibss_station, > .recover_from_tx_stall = iwl_bg_monitor_recover, > - .check_plcp_health = iwl3945_good_plcp_health, > > .debugfs_ops = { > .rx_stats_read = iwl3945_ucode_rx_stats_read, Did you experiment with different values for IWL_MAX_PLCP_ERR_LONG_THRESHOLD_DEF for iwl3945? Should we simply revert commit a29576a7844326c5223f4d4adbfd3f4d64173d4c instead? John -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready.