From owner-cvs-src@FreeBSD.ORG  Tue Nov 18 16:15:27 2003
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 8B9F916A4E4; Tue, 18 Nov 2003 16:15:27 -0800 (PST)
Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id C07E343FA3; Tue, 18 Nov 2003 16:15:24 -0800 (PST)
	(envelope-from bde@zeta.org.au)
Received: from gamplex.bde.org (katana.zip.com.au [61.8.7.246])
	by mailman.zeta.org.au (8.9.3p2/8.8.7) with ESMTP id LAA31467;
	Wed, 19 Nov 2003 11:15:21 +1100
Date: Wed, 19 Nov 2003 11:15:20 +1100 (EST)
From: Bruce Evans <bde@zeta.org.au>
X-X-Sender: bde@gamplex.bde.org
To: Mark Murray <markm@FreeBSD.org>
In-Reply-To: <200311181435.hAIEZhtY048135@repoman.freebsd.org>
Message-ID: <20031119111015.E5018@gamplex.bde.org>
References: <200311181435.hAIEZhtY048135@repoman.freebsd.org>
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_intr.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Nov 2003 00:15:27 -0000

On Tue, 18 Nov 2003, Mark Murray wrote:

> markm       2003/11/18 06:35:43 PST
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/kern             kern_intr.c
>   Log:
>   Hackfix to patch around a kernel panic I introduced. Real fix to
>   follow. In the meanwhile, we are not harvesting interrupt entropy.
>
>   Approved by:    re (jhb)

The correct fix is approximately to back out the changes to the
harvester.  It needs to use preallocated storage like it used to since
it is called from nearly the lowest level of interrupt handling when
neither malloc() nor sleep locks may be used.  Using preallocated storage
also simplifies the locking.

Bruce