From owner-cvs-all Tue Dec 12 12:25: 1 2000 From owner-cvs-all@FreeBSD.ORG Tue Dec 12 12:24:56 2000 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from pike.osd.bsdi.com (pike.osd.bsdi.com [204.216.28.222]) by hub.freebsd.org (Postfix) with ESMTP id 7AE6F37B402; Tue, 12 Dec 2000 12:24:56 -0800 (PST) Received: from laptop.baldwin.cx (john@jhb-laptop.osd.bsdi.com [204.216.28.241]) by pike.osd.bsdi.com (8.11.1/8.9.3) with ESMTP id eBCKOjE95645; Tue, 12 Dec 2000 12:24:46 -0800 (PST) (envelope-from jhb@FreeBSD.org) Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: <20001213071758.R69646@gsmx07.alcatel.com.au> Date: Tue, 12 Dec 2000 12:24:50 -0800 (PST) From: John Baldwin To: Peter Jeremy Subject: Re: cvs commit: src/sys/sys kernel.h eventhandler.h src/sys/kern Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On 12-Dec-00 Peter Jeremy wrote: > On 2000-Dec-11 20:01:35 -0800, John Baldwin wrote: >>jhb 2000/12/11 20:01:35 PST >> >> Modified files: >> sys/sys kernel.h eventhandler.h >> sys/kern subr_eventhandler.c >> Log: >> - Convert the per-eventhandler list mutex to a lockmgr lock so that it can >> be safely held across an eventhandler function call. >> - Fix an instance of the head of an eventhandler list being read without >> the lock being held. >> - Break down and use a SYSINIT at the new SI_SUB_EVENTHANDLER to initialize >> the eventhandler global mutex and the eventhandler list of lists rather >> than using a non-MP safe initialization during the first call to >> eventhandler_register(). >> - Add in a KASSERT() to eventhandler_register() to ensure that we don't try >> to register an eventhandler before things have been initialized. >> >> Revision Changes Path >> 1.75 +2 -1 src/sys/sys/kernel.h >> 1.15 +9 -11 src/sys/sys/eventhandler.h > > I think change to eventhandler.h has broken the agp module build: I wasn't sure if this breakage had been seen by anyone else. The fix is to have agp include sys/proc.h. I'll commit in a second.. -- John Baldwin -- http://www.FreeBSD.org/~jhb/ PGP Key: http://www.baldwin.cx/~john/pgpkey.asc "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message