Re: PCMCIA 2.5.X sleeping from illegal context

From: Paul Fulghum (paulkf@microgate.com)
Date: Tue May 13 2003 - 10:23:05 EST


On Mon, 2003-05-12 at 17:48, Russell King wrote:
> On Mon, May 12, 2003 at 07:36:05AM -0500, Paul Fulghum wrote:
> > On Mon, 2003-05-12 at 16:07, Alan Cox wrote:
> > > On Llu, 2003-05-12 at 22:35, Paul Fulghum wrote:
> > > > The 2.5.X PCMCIA kernel support seems to have a problem
> > > > with drivers/pcmcia/rsrc_mgr.c in function undo_irq().
> > >
> > > Does this still happen with all the patches Russell King posted
> > > that everyone else is ignoring ?
> >
> > I don't know, I've been ignoring them :-)
> >
> > Seriously, are they centralized someplace or
> > should I scan back and try to extract them
> > from the lk archive? Do you know about when
> > they were posted?
>
> Try this one. Its a slightly updated version from the one I sent earlier.

I tested this patch, and it works.
But I was not seeing any problems before the patch,
other than the 'sleeping in illegal context message'

There seems to be agreement that the cause of the
message is that individual PCMCIA drivers need to be
modified to call the release function directly
instead of from a timer context.

I modified my synclink_cs driver to do this and it
works and eliminates the error message.

Which brings up a question:
The include/pcmcia/ds.h file defines the dev_link_t
structure that contains the release timer_list member.

Should I continue to initialize this member in
my driver or will this member be eliminated requiring
that all references to this member be removed from
the individual PCMCIA drivers?

-- 
Paul Fulghum, paulkf@microgate.com
Microgate Corporation, http://www.microgate.com

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



This archive was generated by hypermail 2b29 : Thu May 15 2003 - 22:00:45 EST