* Re: minstrel_tx_status mac80211 WARNINGs in vanilla 2.6.34.1 [not found] <20100722123000.GA16657@geggus.net> @ 2010-07-22 14:47 ` John W. Linville 2010-07-22 19:07 ` Felix Fietkau 0 siblings, 1 reply; 3+ messages in thread From: John W. Linville @ 2010-07-22 14:47 UTC (permalink / raw) To: Sven Geggus; +Cc: netdev, linux-wireless, nbd On Thu, Jul 22, 2010 at 02:30:01PM +0200, Sven Geggus wrote: > Hello, > > running vanilla 2.6.34.1 I get the following warnings in kernel log: > > WARNING: at net/mac80211/rc80211_minstrel.c:70 minstrel_tx_status+0x67/0xd1 [mac80211]() > Hardware name: SCENIC E300/E600 > Modules linked in: i915 drm_kms_helper drm video backlight output lp loop > snd_intel8x0 snd_ac97_codec ac97_bus snd_pcm_oss snd_mixer_oss snd_pcm > snd_seq_dummy zd1211rw snd_seq_oss usbhid mac80211 option cfg80211 > snd_seq_midi usbserial snd_rawmidi snd_seq_midi_event snd_seq snd_timer > snd_seq_device snd parport_pc ehci_hcd uhci_hcd soundcore intel_agp parport > usbcore nls_base snd_page_alloc agpgart rng_core floppy sg > Pid: 0, comm: swapper Tainted: G W 2.6.34.1 #3 > Call Trace: > [<c102af25>] warn_slowpath_common+0x60/0x90 > [<c102af62>] warn_slowpath_null+0xd/0x10 > [<f83cbd48>] minstrel_tx_status+0x67/0xd1 [mac80211] > [<f83b6eb1>] ieee80211_tx_status+0x1f6/0x5ac [mac80211] > [<c1261533>] ? skb_dequeue+0x45/0x4c > [<f83b6896>] ieee80211_tasklet_handler+0x61/0xd6 [mac80211] > [<c102ed7d>] tasklet_action+0x62/0x9f > [<c102f331>] __do_softirq+0x77/0xe5 > [<c102f3c5>] do_softirq+0x26/0x2b > [<c102f52f>] irq_exit+0x29/0x66 > [<c1003e90>] do_IRQ+0x85/0x9b > [<c1002d29>] common_interrupt+0x29/0x30 > [<c10083ac>] ? default_idle+0x2d/0x42 > [<c1001a9b>] cpu_idle+0x44/0x71 > [<c12e00de>] rest_init+0x96/0x98 > [<c1498862>] start_kernel+0x2a5/0x2aa > [<c14980b7>] i386_start_kernel+0xb7/0xbf > ---[ end trace f22ceacef336878f ]--- > > Wireless driver is zd1211rw. > > Did not test with older kernel because this device has not been in user on > this machine before. > > WLAN does however seem to work anyway. Well, I just took a quick look -- so, I'm not 100% sure... But, it looks to me like zd1211rw is reporting tx status on a rate that minstrel didn't expect it to use. It seems like the hardware is pre-wired to do retries on sequentially lower rates, which seems a bit incompatible with minstrel's worldview. Felix, can we accomodate this? The "WLAN does however seem to work anyway" seems to suggest things work, so can we at least not yell about it? Thanks, John -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready. ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: minstrel_tx_status mac80211 WARNINGs in vanilla 2.6.34.1 2010-07-22 14:47 ` minstrel_tx_status mac80211 WARNINGs in vanilla 2.6.34.1 John W. Linville @ 2010-07-22 19:07 ` Felix Fietkau 2010-07-22 19:46 ` [PATCH] minstrel: don't complain about feedback for unrequested rates John W. Linville 0 siblings, 1 reply; 3+ messages in thread From: Felix Fietkau @ 2010-07-22 19:07 UTC (permalink / raw) To: John W. Linville; +Cc: Sven Geggus, netdev, linux-wireless On 2010-07-22 4:47 PM, John W. Linville wrote: > On Thu, Jul 22, 2010 at 02:30:01PM +0200, Sven Geggus wrote: >> Hello, >> >> running vanilla 2.6.34.1 I get the following warnings in kernel log: >> >> WARNING: at net/mac80211/rc80211_minstrel.c:70 minstrel_tx_status+0x67/0xd1 [mac80211]() >> Hardware name: SCENIC E300/E600 >> Modules linked in: i915 drm_kms_helper drm video backlight output lp loop >> snd_intel8x0 snd_ac97_codec ac97_bus snd_pcm_oss snd_mixer_oss snd_pcm >> snd_seq_dummy zd1211rw snd_seq_oss usbhid mac80211 option cfg80211 >> snd_seq_midi usbserial snd_rawmidi snd_seq_midi_event snd_seq snd_timer >> snd_seq_device snd parport_pc ehci_hcd uhci_hcd soundcore intel_agp parport >> usbcore nls_base snd_page_alloc agpgart rng_core floppy sg >> Pid: 0, comm: swapper Tainted: G W 2.6.34.1 #3 >> Call Trace: >> [<c102af25>] warn_slowpath_common+0x60/0x90 >> [<c102af62>] warn_slowpath_null+0xd/0x10 >> [<f83cbd48>] minstrel_tx_status+0x67/0xd1 [mac80211] >> [<f83b6eb1>] ieee80211_tx_status+0x1f6/0x5ac [mac80211] >> [<c1261533>] ? skb_dequeue+0x45/0x4c >> [<f83b6896>] ieee80211_tasklet_handler+0x61/0xd6 [mac80211] >> [<c102ed7d>] tasklet_action+0x62/0x9f >> [<c102f331>] __do_softirq+0x77/0xe5 >> [<c102f3c5>] do_softirq+0x26/0x2b >> [<c102f52f>] irq_exit+0x29/0x66 >> [<c1003e90>] do_IRQ+0x85/0x9b >> [<c1002d29>] common_interrupt+0x29/0x30 >> [<c10083ac>] ? default_idle+0x2d/0x42 >> [<c1001a9b>] cpu_idle+0x44/0x71 >> [<c12e00de>] rest_init+0x96/0x98 >> [<c1498862>] start_kernel+0x2a5/0x2aa >> [<c14980b7>] i386_start_kernel+0xb7/0xbf >> ---[ end trace f22ceacef336878f ]--- >> >> Wireless driver is zd1211rw. >> >> Did not test with older kernel because this device has not been in user on >> this machine before. >> >> WLAN does however seem to work anyway. > > Well, I just took a quick look -- so, I'm not 100% sure... > > But, it looks to me like zd1211rw is reporting tx status on a rate > that minstrel didn't expect it to use. It seems like the hardware > is pre-wired to do retries on sequentially lower rates, which seems > a bit incompatible with minstrel's worldview. > > Felix, can we accomodate this? The "WLAN does however seem to work > anyway" seems to suggest things work, so can we at least not yell > about it? I think we should just drop that WARN_ON(). It's not problematic if minstrel gets feedback for rates that it doesn't have in its list, it should just ignore it. - Felix ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] minstrel: don't complain about feedback for unrequested rates 2010-07-22 19:07 ` Felix Fietkau @ 2010-07-22 19:46 ` John W. Linville 0 siblings, 0 replies; 3+ messages in thread From: John W. Linville @ 2010-07-22 19:46 UTC (permalink / raw) To: linux-wireless; +Cc: Felix Fietkau, Sven Geggus, netdev, John W. Linville "It's not problematic if minstrel gets feedback for rates that it doesn't have in its list, it should just ignore it. - Felix" Signed-off-by: John W. Linville <linville@tuxdriver.com> Cc: Felix Fietkau <nbd@openwrt.org> --- net/mac80211/rc80211_minstrel.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/net/mac80211/rc80211_minstrel.c b/net/mac80211/rc80211_minstrel.c index f65ce6d..778c604 100644 --- a/net/mac80211/rc80211_minstrel.c +++ b/net/mac80211/rc80211_minstrel.c @@ -67,7 +67,6 @@ rix_to_ndx(struct minstrel_sta_info *mi, int rix) for (i = rix; i >= 0; i--) if (mi->r[i].rix == rix) break; - WARN_ON(i < 0); return i; } -- 1.7.1.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-07-22 20:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20100722123000.GA16657@geggus.net>
2010-07-22 14:47 ` minstrel_tx_status mac80211 WARNINGs in vanilla 2.6.34.1 John W. Linville
2010-07-22 19:07 ` Felix Fietkau
2010-07-22 19:46 ` [PATCH] minstrel: don't complain about feedback for unrequested rates John W. Linville
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).