[Prism54-devel] Recent changes to reset functionality
Luis R. Rodriguez
mcgrof@ruslug.rutgers.edu
Mon, 9 Feb 2004 15:15:27 -0500
Oh yea, feel free to take care of this too if you get around to it
before I do 8)
Luis
On Mon, Feb 09, 2004 at 02:38:42PM -0500, Luis R. Rodriguez wrote:
>
> I'll take care of this, thanks.
> Luis
>
> On Sat, Feb 07, 2004 at 06:04:10PM +0100, Jens Maurer wrote:
> >
> > I've had a look at islpci_reset_if() and
> > the function that calls it, islpci_reset().
> >
> > I think the general approach is fine (give the card a little
> > more time if the interrupt doesn't come through the
> > first time), but the implementation has a race condition.
> > There is a time window between leaving islpci_reset_if() the
> > first time and re-entering it later where a reset interrupt
> > would not trigger the wait queue, because no interested
> > parties are registered at this moment.
> >
> > I suggest moving the "wait a little longer" functionality
> > entirely into islpci_reset_if(), and make sure to continue
> > to perform adequate waitqueue maintenance there.
> >
> > Jens Maurer
> >
> > _______________________________________________
> > Prism54-devel mailing list
> > Prism54-devel@prism54.org
> > http://prism54.org/mailman/listinfo/prism54-devel
> _______________________________________________
> Prism54-devel mailing list
> Prism54-devel@prism54.org
> http://prism54.org/mailman/listinfo/prism54-devel