Re: CFR: ACPI Dock driver

From: Søren Schmidt <sos_at_FreeBSD.org>
Date: Fri, 14 Apr 2006 18:24:10 +0200
On Lør, 2006-04-15 at 01:20 +0900, Mitsuru IWASAKI wrote:
> Hi,
> 
> > --- ata-all.c   31 Mar 2006 08:09:04 -0000      1.271
> > +++ ata-all.c   14 Apr 2006 15:39:20 -0000
> > _at__at_ -277,8 +277,8 _at__at_
> >      if (!dev || !(ch = device_get_softc(dev)))
> >         return ENXIO;
> > 
> > -    /* wait for the channel to be IDLE before entering suspend mode */
> > -    while (1) {
> > +    /* wait for the channel to be IDLE or detached before suspending */
> > +    while (ch->r_irq) {
> >         mtx_lock(&ch->state_mtx);
> >         if (ch->state == ATA_IDLE) {
> >             ch->state = ATA_ACTIVE;
> > 
> > That I'll gladly commit for you instead...
> 
> Yes, This have also the same effect.  I prefer this.

Good, this one also doesn't relay on a mutx that has been destroyed :)

I'll commit asap...

-Søren
Received on Fri Apr 14 2006 - 14:24:12 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:54 UTC