Return-path: Received: from ist.d-labs.de ([213.239.218.44]:46589 "EHLO mx01.d-labs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753181Ab0JFRcs (ORCPT ); Wed, 6 Oct 2010 13:32:48 -0400 Date: Wed, 6 Oct 2010 19:32:45 +0200 From: Florian Mickler To: "Guy, Wey-Yi" Cc: Stanislaw Gruszka , "linville@tuxdriver.com" , "stable@kernel.org" , "linux-wireless@vger.kernel.org" , "Chatre, Reinette" , "ilw@linux.intel.com" , "Berg, Johannes" , "Cahill, Ben M" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH wireless-2.6 or stable] iwlwifi: return error when fail to start scanning Message-ID: <20101006193245.698957cc@schatten.dmk.lab> In-Reply-To: <1286381578.12594.18.camel@wwguy-ubuntu> References: <20101005085717.GA18012@redhat.com> <1286317292-10679-1-git-send-email-florian@mickler.org> <20101006090228.GA2523@redhat.com> <20101006160434.GB24581@redhat.com> <1286381578.12594.18.camel@wwguy-ubuntu> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 06 Oct 2010 09:12:58 -0700 "Guy, Wey-Yi" wrote: > Hi Gruszka, > > On Wed, 2010-10-06 at 09:04 -0700, Stanislaw Gruszka wrote: > > This is stable friendly backport of wireless-testing commit > > 3eecce527c7434dfd16517ce08b49632c8a26717 "iwlwifi: unify scan start > > checks". Wireless-testing version include also a lot of cleanups. > > > > On error case in {iwl3945,iwlagn}_request_scan we queue abort_scan work, > > which in a fact do nothing, so we never complete the scan. Thats gives > > "WARNING: at net/wireless/core.c:614 wdev_cleanup_work" and stopped > > network connection until reload iwlwifi modules. Return of -EIO, and > > stopping queuing any work is a fix. > > > > Note there are still many cases when we can get: > > > > WARNING: at net/wireless/core.c:614 wdev_cleanup_work > > WARNING: at net/mac80211/scan.c:266 ieee80211_scan_completed > > WARNING: at net/mac80211/scan.c:269 ieee80211_scan_complete > > > > which are not fixed. Unfortunately does not exist single, small fix > > for that problems, and we probably will see some more bug reports > > with scan warnings. As solution we rewrite iwl-scan.c code to avoid > > all possible race conditions. That quite big patch set is queued > > for 2.6.37 . > > > > Signed-off-by: Stanislaw Gruszka > > --- > > Patch is intended for wireless-2.6, or in case when it does not > > make 2.6.36 release, for -stable 2.6.36.y > > > > Is this the replacement for Mickler's patch? > > Thanks > Wey > This is on top of current mainline, as far as I see. (And it replaces the patch I sent few hours ago) Regards, Flo