From: Yong Zhang <yong.zhang0@gmail.com>
To: Michal Hocko <mhocko@suse.cz>
Cc: Stanislaw Gruszka <sgruszka@redhat.com>,
LKML <linux-kernel@vger.kernel.org>,
"John W. Linville" <linville@tuxdriver.com>,
linux-wireless@vger.kernel.org
Subject: Re: [3.2-rc3] 100% CPU usage while in del_timer_sync from iwl3945_rs_free_sta
Date: Thu, 1 Dec 2011 20:04:01 +0800 [thread overview]
Message-ID: <20111201120400.GA2112@zhy> (raw)
In-Reply-To: <20111201105538.GA4504@tiehlicka.suse.cz>
On Thu, Dec 01, 2011 at 11:55:38AM +0100, Michal Hocko wrote:
> On Thu 01-12-11 09:58:45, Michal Hocko wrote:
> [...]
> > I will try to turn on timer debugging suggested by Yong Zhang in other
> > email.
>
> OK, so applied the following commits:
> $ git shortlog master..
> Christine Chan (2):
> timer: Use debugobjects to catch deletion of uninitialized timers
> debugobjects: Extend to assert that an object is initialized
>
> Michal Hocko (1):
> Debugging patch
>
> Stephen Boyd (2):
> timer: Setup uninitialized timer with a stub callback
> debugobjects: Be smarter about static objects
>
> Debugging patch is the one from Stanislaw posted earlier in the thread.
> It prevents del_timer_sync and warns on NULL timer in iwl3945_rs_free_sta.
> Also turned on the suggested DEBUG_OBJECTS_TIMERS.
>
> I do not see any timer related warnings. I will try it with the slightly
> modified Debugging patch and just warn without returning before calling
> del_timer_sync.
>
> [ 349.316076] ------------[ cut here ]------------
> [ 349.316097] WARNING: at drivers/net/wireless/iwlegacy/iwl-3945-rs.c:438 iwl3945_rs_free_sta+0x33/0x42 [iwl3945]()
> [ 349.316105] Hardware name: LIFEBOOK S7110
> [ 349.316110] Modules linked in: aes_i586 aes_generic i915 fbcon font bitblit softcursor drm_kms_helper drm fb fbdev i2c_algo_bit cfbcopyarea i2c_core cfbimgblt cfbfillrect fuse tun coretemp hwmon snd_hda_codec_realtek arc4 snd_hda_intel snd_hda_codec snd_pcm_oss snd_mixer_oss iwl3945 snd_pcm iwl_legacy snd_seq_oss snd_seq_midi_event mac80211 snd_seq snd_timer snd_seq_device cfg80211 snd video snd_page_alloc fujitsu_laptop backlight [last unloaded: scsi_wait_scan]
> [ 349.316203] Pid: 12, comm: kworker/u:1 Tainted: G W 3.2.0-rc3-timerstest-00009-g762dd0e #20
^
seems there was warning but we did't catch it, right?
not sure if it's related.
Thanks,
Yong
> [ 349.316209] Call Trace:
> [ 349.316223] [<c01331b4>] warn_slowpath_common+0x68/0x7d
> [ 349.316237] [<f8169454>] ? iwl3945_rs_free_sta+0x33/0x42 [iwl3945]
> [ 349.316247] [<c01331dd>] warn_slowpath_null+0x14/0x18
> [ 349.316260] [<f8169454>] iwl3945_rs_free_sta+0x33/0x42 [iwl3945]
> [ 349.316283] [<f8102f94>] __sta_info_free.isra.24+0x28/0x60 [mac80211]
> [ 349.316304] [<f8103df9>] __sta_info_destroy+0x259/0x283 [mac80211]
> [ 349.316328] [<f8104194>] sta_info_flush+0x43/0x6e [mac80211]
> [ 349.316353] [<f8109453>] ieee80211_set_disassoc+0x1a3/0x1d7 [mac80211]
> [ 349.316379] [<f81095e8>] ieee80211_sta_connection_lost+0x31/0x8a [mac80211]
> [ 349.316390] [<c031208f>] ? dev_printk+0x2b/0x2d
> [ 349.316416] [<f810bde2>] ieee80211_sta_work+0x1b0/0x1c3 [mac80211]
> [ 349.316446] [<f810e38a>] ieee80211_iface_work+0x23e/0x250 [mac80211]
> [ 349.316457] [<c01468af>] process_one_work+0x19b/0x2e6
> [ 349.316483] [<f810e14c>] ? ieee80211_netdev_select_queue+0x14/0x14 [mac80211]
> [ 349.316492] [<c0146ddd>] worker_thread+0x136/0x1ee
> [ 349.316501] [<c0146ca7>] ? manage_workers.isra.23+0x14f/0x14f
> [ 349.316509] [<c014a0f5>] kthread+0x67/0x6c
> [ 349.316517] [<c014a08e>] ? kthread_worker_fn+0x119/0x119
> [ 349.316527] [<c04762f6>] kernel_thread_helper+0x6/0x10
> [ 349.316532] ---[ end trace 5cf96275a244e7ca ]---
> [ 349.316538] ieee80211 phy0: Destroyed STA 00:0b:6b:3c:8c:e4
> [ 349.316556] ieee80211 phy0: device now idle
> [ 349.328105] cfg80211: Calling CRDA to update world regulatory domain
> [ 349.416952] ieee80211 phy0: device no longer idle - scanning
> [ 353.091211] ieee80211 phy0: device now idle
> [ 358.092230] ieee80211 phy0: device no longer idle - scanning
> [ 361.772470] ieee80211 phy0: device now idle
> [ 366.774826] ieee80211 phy0: device no longer idle - scanning
> [ 370.448401] ieee80211 phy0: device now idle
> [ 375.455127] ieee80211 phy0: device no longer idle - scanning
> [ 379.127638] ieee80211 phy0: device now idle
> [ 384.131847] ieee80211 phy0: device no longer idle - scanning
> [ 387.806711] ieee80211 phy0: device now idle
> [ 396.708125] sky2 0000:02:00.0: lan0: Link is up at 1000 Mbps, full duplex, flow control both
> [ 407.120063] lan0: no IPv6 routers present
> [ 434.003896] ieee80211 phy0: device no longer idle - scanning
> [ 438.694843] ieee80211 phy0: device now idle
> [ 441.485602] process `skype' is using obsolete setsockopt SO_BSDCOMPAT
> [ 494.004619] ieee80211 phy0: device no longer idle - scanning
> [ 498.698796] ieee80211 phy0: device now idle
> [ 554.004822] ieee80211 phy0: device no longer idle - scanning
> [ 558.697908] ieee80211 phy0: device now idle
> [ 614.003956] ieee80211 phy0: device no longer idle - scanning
> [ 618.700408] ieee80211 phy0: device now idle
> [ 674.004164] ieee80211 phy0: device no longer idle - scanning
> [ 678.702423] ieee80211 phy0: device now idle
>
> --
> Michal Hocko
> SUSE Labs
> SUSE LINUX s.r.o.
> Lihovarska 1060/12
> 190 00 Praha 9
> Czech Republic
next prev parent reply other threads:[~2011-12-01 12:04 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-29 10:07 [3.2-rc3] 100% CPU usage while in del_timer_sync from iwl3945_rs_free_sta Michal Hocko
2011-11-29 11:39 ` Stanislaw Gruszka
2011-11-30 10:10 ` Michal Hocko
2011-11-30 14:23 ` Stanislaw Gruszka
2011-11-30 14:03 ` Michal Hocko
2011-11-30 15:39 ` Stanislaw Gruszka
2011-12-01 8:58 ` Michal Hocko
2011-12-01 10:55 ` Michal Hocko
2011-12-01 11:59 ` Stanislaw Gruszka
2011-12-01 12:44 ` Michal Hocko
2011-12-01 12:55 ` Stanislaw Gruszka
2011-12-01 13:13 ` Eliad Peller
2011-12-01 13:31 ` Stanislaw Gruszka
2011-12-01 13:37 ` Sedat Dilek
2011-12-01 14:05 ` Michal Hocko
2011-12-01 14:23 ` Eliad Peller
2011-12-01 15:45 ` John W. Linville
2011-12-01 12:04 ` Yong Zhang [this message]
2011-12-01 12:42 ` Michal Hocko
2011-12-01 2:09 ` Yong Zhang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20111201120400.GA2112@zhy \
--to=yong.zhang0@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=mhocko@suse.cz \
--cc=sgruszka@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).