From owner-freebsd-current@FreeBSD.ORG Tue Oct 12 09:14:45 2004 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2AEF916A4CE; Tue, 12 Oct 2004 09:14:45 +0000 (GMT) Received: from bloodwood.hunterlink.net.au (smtp-local.hunterlink.net.au [203.12.144.17]) by mx1.FreeBSD.org (Postfix) with ESMTP id 887C943D39; Tue, 12 Oct 2004 09:14:43 +0000 (GMT) (envelope-from boris@brooknet.com.au) Received: from ppp2804.dyn.pacific.net.au (ppp2804.dyn.pacific.net.au [61.8.40.4])i9C9BNi3010870; Tue, 12 Oct 2004 19:11:25 +1000 From: Sam Lawrance To: Julian Elischer In-Reply-To: <416AC227.7010901@elischer.org> References: <416AC227.7010901@elischer.org> Content-Type: text/plain Date: Tue, 12 Oct 2004 19:16:59 +1000 Message-Id: <1097572619.822.19.camel@dirk.no.domain> Mime-Version: 1.0 X-Mailer: Evolution 2.0.1FreeBSD GNOME Team Port Content-Transfer-Encoding: 7bit cc: freebsd-current@freebsd.org cc: Robert Watson cc: David Xu Subject: Re: Noticable Delays Since Beta 3 (possible cause) X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 12 Oct 2004 09:14:45 -0000 On Mon, 2004-10-11 at 10:25 -0700, Julian Elischer wrote: > > On Sat, 9 Oct 2004, Sam Lawrance wrote: > > > >>I'm sorry, that patch was missing paths.. this one: > >> > >>http://sam.stral.net/freebsd/wakeupdelay-patch > >> > > It has been less than 2 days since it was committed to -current. Are you thinking of something else? The patch hasn't been committed AFAICS. Here is *another* patch, along the lines of what Matt suggested - the changes are only to sleepq_broadcast. Seems to work. This patch includes comment and doc changes to kern_synch and scheduler.9 indicating that setrunnable() no longer awakens the scheduler. I was not sure whether wchan != &proc0 should be added to the 'if' around the wakeup. I don't think it is needed. http://sam.stral.net/freebsd/wakeupdelay2-patch