From owner-cvs-all Thu Apr 11 0: 2:16 2002 Delivered-To: cvs-all@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id D390737B41A; Thu, 11 Apr 2002 00:02:10 -0700 (PDT) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id RAA01718; Thu, 11 Apr 2002 17:01:48 +1000 Date: Thu, 11 Apr 2002 17:00:49 +1000 (EST) From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: Jake Burkholder Cc: John Baldwin , , , Brian Somers Subject: Re: cvs commit: src/sys/dev/digi digi.c In-Reply-To: <20020410105203.B209@locore.ca> Message-ID: <20020411162116.D3455-100000@gamplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Wed, 10 Apr 2002, Jake Burkholder wrote: > Apparently, On Wed, Apr 10, 2002 at 09:06:18AM -0400, > John Baldwin said words to the effect of; > > I've thought about having some sort of separate API for code that just ways to > > delay. Right now they use tsleep() on a channel that never gets woken up with > > a timeout. If we had a delay(int timo) function then it could use a callout > > when callouts were working and fall back to DELAY() in the cold case. > > No, this is wrong. Anything that uses cold is wrong. The correct fix is to > move the clock initialization earlier in boot to be before the device probe. This can't be done in a machine independent way. Clock interrupts might be shared, and then enabling them first would cause either endless interrupts if another device is driving the interrupt. This happens not to be a problem on i386's because clock interrupts aren't shared. I think this is still a problem for configuration of drivers with shared non-clock interrupts. The handling of this seems to have regressed in -current on i386's. RELENG_4 at least has large comments about it. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message