From owner-cvs-all Tue Jan 23 17: 7:46 2001 Delivered-To: cvs-all@freebsd.org Received: from pike.osd.bsdi.com (unknown [204.216.28.222]) by hub.freebsd.org (Postfix) with ESMTP id B171B37B6B6; Tue, 23 Jan 2001 17:07:22 -0800 (PST) Received: from foo.osd.bsdi.com (root@foo.osd.bsdi.com [204.216.28.137]) by pike.osd.bsdi.com (8.11.1/8.9.3) with ESMTP id f0O17Hx73578; Tue, 23 Jan 2001 17:07:17 -0800 (PST) (envelope-from jhb@foo.osd.bsdi.com) Received: (from jhb@localhost) by foo.osd.bsdi.com (8.11.1/8.11.1) id f0O16q031271; Tue, 23 Jan 2001 17:06:52 -0800 (PST) (envelope-from jhb) 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: <025601c085a1$eb367f20$1f90c918@jehovah> Date: Tue, 23 Jan 2001 17:06:52 -0800 (PST) Organization: BSD, Inc. From: John Baldwin To: Bosko Milekic Subject: Re: cvs commit: src/sys/kern kern_malloc.c Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On 24-Jan-01 Bosko Milekic wrote: > > John Baldwin: > >> jhb 2001/01/23 16:36:04 PST >> >> Modified files: >> sys/kern kern_malloc.c >> Log: >> Don't grab Giant when calling kmem_alloc/kmem_free as this is just >> encouraging other people to follow the same practice. If this is >> going >> to be done, then it should be done inside of those two functions >> instead. >> >> Revision Changes Path >> 1.79 +1 -5 src/sys/kern/kern_malloc.c > > We should do this and add the proper (temporary) > assertions/invariants/witness checks in there, as soon as possible, > then. > Several portions of code using the kmem_*() code are doing this right > now, as > you probably now, and we're just (needlessly) repeating the > assertions. > > I don't mind doing it later this week, when I'm done with what I'm > presently > working with. I have the assertions done, but kmem_alloc() is called really early, so Giant has to be obtained really early. I've got this tested on the x86, but no the alpha. I was going to hold off comitting it until I had tested it on the alpha first. :) > -Bosko. -- 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