Re: Scheduling while atomic (2.6.10-rc3-bk13)

[Lee Revell <rlrevell@joe-job.com>]

On Mon, 2004-12-20 at 11:52 -0800, David Brownell wrote:
> On Monday 20 December 2004 10:48 am, Greg KH wrote:
> 
> > 
> > David, it looks like you grab a spinlock, and then call msleep(20);
> > which causes this warning.
> > 
> > Care to fix it?
> 
> How bizarre ... I must have been tested that without spinlock
> debugging, for some reason.  Grr.  I usually leave that on,
> just to prevent stuff like this.
> 
> Here's a quick'n'dirty patch, msleep --> mdelay.  I'd rather
> not mdelay for that long, but this late in 2.6.10 it's safer.
> (And this is also what OHCI does in that same code path.)

Ugh.  20ms is WAY too long to hold a spinlock.  That's guaranteed to
cause audio skips.  Isn't there another way?

If OHCI calls mdelay(20) while holding a spinlock that needs to be
fixed.

Lee