From owner-cvs-all@FreeBSD.ORG  Sat Jan 31 18:54:47 2004
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 3683616A4CE; Sat, 31 Jan 2004 18:54:47 -0800 (PST)
Received: from mail.chesapeake.net (chesapeake.net [208.142.252.6])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id BEBE743D39; Sat, 31 Jan 2004 18:54:45 -0800 (PST)
	(envelope-from jroberson@chesapeake.net)
Received: from localhost (jroberson@localhost)
	by mail.chesapeake.net (8.11.6/8.11.6) with ESMTP id i112sjG61305;
	Sat, 31 Jan 2004 21:54:45 -0500 (EST)
	(envelope-from jroberson@chesapeake.net)
Date: Sat, 31 Jan 2004 21:54:45 -0500 (EST)
From: Jeff Roberson <jroberson@chesapeake.net>
To: John Baldwin <jhb@FreeBSD.org>
In-Reply-To: <200401282044.i0SKifeF063778@repoman.freebsd.org>
Message-ID: <20040131215357.N36463-100000@mail.chesapeake.net>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/kern kern_synch.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 01 Feb 2004 02:54:47 -0000

On Wed, 28 Jan 2004, John Baldwin wrote:

> jhb         2004/01/28 12:44:41 PST
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/kern             kern_synch.c
>   Log:
>   Move the loadav() callout into its own kthread since it uses allproc_lock
>   which is a sleepable lock and thus is not safe to acquire from a callout
>   routine.

Can you back this out now?  I've commited a slightly better solution that
makes our loadavg function more correct as well as removing the lock
requirement.

Cheers,
Jeff

>
>   Revision  Changes    Path
>   1.240     +31 -12    src/sys/kern/kern_synch.c
>