Date: Sat, 17 Jan 2004 00:44:36 +0100 From: des@des.no (Dag-Erling =?iso-8859-1?q?Sm=F8rgrav?=) To: Robert Watson <rwatson@FreeBSD.org> Cc: cvs-src@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_descrip.c Message-ID: <xzp4quvqx0b.fsf@dwp.des.no> In-Reply-To: <Pine.NEB.3.96L.1040116182253.81408I-100000@fledge.watson.org> (Robert Watson's message of "Fri, 16 Jan 2004 18:24:24 -0500 (EST)") References: <Pine.NEB.3.96L.1040116182253.81408I-100000@fledge.watson.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Robert Watson <rwatson@FreeBSD.org> writes: > It sounds like this is an API problem, and is probably what we > should fix. I've found WITNESS an invaluable debugging tool for > locking, and when programming on systems without it, it's a very > painful experience (i.e., debugging lock orders on Darwin). You've > also pointed out that the extra locking work being done is actually > unnecessary, so maybe we just need an _unlocked() version of the > API, or changes elsewhere? The only case in which the locking is unnecessary is when initializing a new struct filedesc. In all other cases it is required. Creating an unlocked version of the API would require duplicating a lot of code, for very little gain. DES --=20 Dag-Erling Sm=F8rgrav - des@des.no
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?xzp4quvqx0b.fsf>