Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 1 Dec 1999 13:28:12 -0700
From:      Nate Williams <nate@mt.sri.com>
To:        Mike Smith <msmith@FreeBSD.ORG>
Cc:        Warner Losh <imp@village.org>, mobile@FreeBSD.ORG
Subject:   Re: cvs commit: src/sys/pccard pccard.c src/sys/isa sio.c src/sys/dev/ed if_ed_pccard.c src/sys/dev/ep if_ep_pccard.c 
Message-ID:  <199912012028.NAA01870@mt.sri.com>
In-Reply-To: <199912012010.MAA00653@mass.cdrom.com>
References:  <199912011600.JAA02186@harmony.village.org> <199912012010.MAA00653@mass.cdrom.com>

next in thread | previous in thread | raw e-mail | index | archive | help
> > Spin limiters in the ISRs of drivers that support detach.  Doing
> > thinks like sio does with sc->gone (w/o the softc) and other
> > techniques are going to be what is required to fix this.
> 
> Again, soft state is not adequate.  sc->gone can only be set if something 
> preempts the spinning interrupt handler.

Correct.

> In the old old code this only 
> worked because the pcic interrupt handler wasn't masked, so the detach 
> routine could preempt the device interrupt handler and set the gone flag.

Do you have a better solution than this?  This was the 'most' correct
solution I could come up with.  (I'd love to hear about a better
solution than this).

The only other solution I could envision was one involving re-writing
all of the drivers to modify them so they would have timeouts reading
hardware (difficult and time consuming), but this wouldn't work in
newbus due to softc going away. :(




Nate


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-mobile" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199912012028.NAA01870>