Re: [BUG] Oopsen with pcmcia aironet wireless (2.5.47)

Jean Tourrilhes (jt@bougret.hpl.hp.com)
Tue, 12 Nov 2002 10:02:34 -0800


On Mon, Nov 11, 2002 at 08:13:40PM -0800, Andrew Morton wrote:
> Andrew McGregor wrote:
> >
> > I see lots of oopsen when I insert an Aironet PC4800 802.11 card.
>
> They are debug warnings, not oopses.
>
> > ...
> > Nov 12 15:40:39 localhost kernel: end_request: I/O error, dev hdb, sector 0
> > Nov 12 15:40:39 localhost last message repeated 3 times
> > Nov 12 15:40:39 localhost kernel: end_request: I/O error, dev hdc, sector 0
> > Nov 12 15:40:39 localhost last message repeated 2 times
>
> hm. IDE sick?
>
> > Nov 12 15:40:39 localhost kernel: Debug: sleeping function called from
> > illegal context at include/asm/semaphore.h:145
> > Nov 12 15:40:39 localhost kernel: Call Trace:
> > Nov 12 15:40:39 localhost kernel: [<e1a59215>] PC4500_readrid+0x55/0x160
>
> airo_get_stats is called under the read_lock(&dev_base_lock); which was
> taken in dev_get_info. So it may not call sleeping functions (ie:
> down_interruptible()).
>
> It would appear that no netdevice->get_stats() method is allowed to sleep,
> which seems pretty dumb, IMVHO.

Forwarded to airo maintainer.

Javier :
Most drivers collect stats based on Tx/Rx packets (they just
increment counters). Look at other driver for details. If you really
want to get stats out of the firmware, you may use the techinique I
used in orinoco for wireless stats, which is to use previous stats and
only call async the stat function.
Have fun...

Jean
-
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/