Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754769Ab3CRTPn (ORCPT ); Mon, 18 Mar 2013 15:15:43 -0400 Received: from charlotte.tuxdriver.com ([70.61.120.58]:33344 "EHLO smtp.tuxdriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751383Ab3CRTPk (ORCPT ); Mon, 18 Mar 2013 15:15:40 -0400 Date: Mon, 18 Mar 2013 15:13:41 -0400 From: "John W. Linville" To: Parag Warudkar Cc: "Luis R. Rodriguez" , Jouni Malinen , Vasanthakumar Thiagarajan , linux-wireless@vger.kernel.org, ath9k-devel@venema.h4ckr.net, netdev@vger.kernel.org, LKML , senthilb@qca.qualcomm.com Subject: Re: [PATCH] ath9k : Fix ieee80211 work while going to suspend Message-ID: <20130318191340.GA28919@tuxdriver.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7188 Lines: 161 Any comments from the ath9k folks? On Sat, Mar 16, 2013 at 12:38:14PM -0400, Parag Warudkar wrote: > During suspend below warning is seen when ath9k is active. Attached > patch fixes the warning for me. Tested to work across few > suspend-resume cycles. > > > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642939] WARNING: at > net/mac80211/util.c:599 ieee80211_can_queue_work.isra.7+0x32/0x40 > [mac80211]() > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642940] Hardware name: iMac12,1 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642941] queueing ieee80211 > work while going to suspend > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642972] Modules linked in: > joydev hid_logitech_dj zfs(POF) bridge stp llc zunicode(POF) zavl(POF) > zcommon(POF) znvpair(POF) spl(OF) zlib_deflate be2iscsi > iscsi_boot_sysfs bnx2i cnic uio cxgb4i cxgb4 cxgb3i cxgb3 mdio > libcxgbi ib_iser rdma_cm ib_addr iw_cm ib_cm ib_sa ib_mad ib_core > iscsi_tcp libiscsi_tcp libiscsi scsi_transport_iscsi rfcomm bnep > nls_utf8 hfsplus btusb uvcvideo videobuf2_vmalloc videobuf2_memops > videobuf2_core videodev bluetooth media coretemp snd_hda_codec_hdmi > snd_hda_codec_cirrus snd_hda_intel snd_hda_codec snd_hwdep snd_seq > snd_seq_device snd_pcm snd_page_alloc snd_timer snd soundcore arc4 > ath9k ath9k_common microcode ath9k_hw ath mac80211 iTCO_wdt > iTCO_vendor_support cfg80211 lpc_ich mei mfd_core rfkill i2c_i801 > applesmc apple_bl input_polldev vhost_net tun macvtap macvlan > kvm_intel kvm binfmt_misc uinput usb_storage crc32c_intel radeon ttm > i915 ghash_clmulni_intel firewire_ohci firewire_core i2c_algo_bit > drm_kms_helper crc_itu_t drm tg3 ptp pps_core i2c_core video sunrpc > [last unloaded: iptable_mangle] > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642985] Pid: 0, comm: > swapper/0 Tainted: PF O 3.8.2-206.fc18.x86_64 #1 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642986] Call Trace: > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.642992] > [] warn_slowpath_common+0x7f/0xc0 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643009] > [] ? ath_start_rx_poll+0x70/0x70 [ath9k] > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643011] > [] warn_slowpath_fmt+0x46/0x50 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643018] > [] ieee80211_can_queue_work.isra.7+0x32/0x40 > [mac80211] > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643024] > [] ieee80211_queue_work+0x2e/0x50 [mac80211] > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643027] > [] ath_rx_poll+0x18/0x20 [ath9k] > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643029] > [] call_timer_fn+0x3a/0x120 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643032] > [] ? ath_start_rx_poll+0x70/0x70 [ath9k] > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643034] > [] ? cpufreq_p4_target+0x120/0x120 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643035] > [] run_timer_softirq+0x1fe/0x2b0 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643037] > [] ? cpufreq_p4_target+0x120/0x120 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643038] > [] __do_softirq+0xd0/0x210 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643040] > [] ? native_sched_clock+0x13/0x80 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643041] > [] ? cpufreq_p4_target+0x120/0x120 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643043] > [] call_softirq+0x1c/0x30 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643045] > [] do_softirq+0x75/0xb0 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643046] > [] irq_exit+0xb5/0xc0 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643048] > [] smp_apic_timer_interrupt+0x6e/0x99 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643049] > [] apic_timer_interrupt+0x6d/0x80 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643052] > [] ? __hrtimer_start_range_ns+0x1be/0x400 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643053] > [] ? cpuidle_wrap_enter+0x50/0xa0 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643054] > [] ? cpuidle_wrap_enter+0x49/0xa0 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643056] > [] cpuidle_enter_tk+0x10/0x20 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643057] > [] cpuidle_idle_call+0xa9/0x260 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643058] > [] cpu_idle+0xaf/0x120 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643061] > [] rest_init+0x72/0x80 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643063] > [] start_kernel+0x3d1/0x3de > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643065] > [] ? repair_env_string+0x5e/0x5e > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643066] > [] x86_64_start_reservations+0x131/0x135 > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643068] > [] x86_64_start_kernel+0x100/0x10f > Mar 16 09:39:17 Parags-iMac kernel: [ 3993.643068] ---[ end trace > 5595a7f5dd9a2949 ]--- > > Signed-off-by: Parag Warudkar > > diff --git a/drivers/net/wireless/ath/ath9k/ath9k.h > b/drivers/net/wireless/ath/ath9k/ath9k.h > index a56b241..b3088a1 100644 > --- a/drivers/net/wireless/ath/ath9k/ath9k.h > +++ b/drivers/net/wireless/ath/ath9k/ath9k.h > @@ -764,6 +764,7 @@ struct ath_softc { > atomic_t wow_got_bmiss_intr; > atomic_t wow_sleep_proc_intr; /* in the middle of WoW sleep ? */ > u32 wow_intr_before_sleep; > + bool suspending; > #endif > }; > > diff --git a/drivers/net/wireless/ath/ath9k/link.c > b/drivers/net/wireless/ath/ath9k/link.c > index ade3afb..fa77e19 100644 > --- a/drivers/net/wireless/ath/ath9k/link.c > +++ b/drivers/net/wireless/ath/ath9k/link.c > @@ -158,7 +158,8 @@ void ath_start_rx_poll(struct ath_softc *sc, u8 nbeacon) > { > if (!AR_SREV_9300(sc->sc_ah)) > return; > - > + if (sc->suspending) > + return; > if (!test_bit(SC_OP_PRIM_STA_VIF, &sc->sc_flags)) > return; > > diff --git a/drivers/net/wireless/ath/ath9k/main.c > b/drivers/net/wireless/ath/ath9k/main.c > index 6e66f9c..0cf88b7 100644 > --- a/drivers/net/wireless/ath/ath9k/main.c > +++ b/drivers/net/wireless/ath/ath9k/main.c > @@ -2150,7 +2150,7 @@ static int ath9k_suspend(struct ieee80211_hw *hw, > int ret = 0; > > mutex_lock(&sc->mutex); > - > + sc->suspending = 1; > ath_cancel_work(sc); > ath_stop_ani(sc); > del_timer_sync(&sc->rx_poll_timer); > @@ -2288,6 +2288,7 @@ static int ath9k_resume(struct ieee80211_hw *hw) > ath9k_start_btcoex(sc); > > ath9k_ps_restore(sc); > + sc->suspending = 0; > mutex_unlock(&sc->mutex); > > return 0; -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/