From owner-freebsd-arch Sun Mar 2 20:25:50 2003 Delivered-To: freebsd-arch@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id D4E4837B405 for ; Sun, 2 Mar 2003 20:25:48 -0800 (PST) Received: from mail.chesapeake.net (chesapeake.net [205.130.220.14]) by mx1.FreeBSD.org (Postfix) with ESMTP id A83F143F85 for ; Sun, 2 Mar 2003 20:25:47 -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 h234PkN85111; Sun, 2 Mar 2003 23:25:46 -0500 (EST) (envelope-from jroberson@chesapeake.net) Date: Sun, 2 Mar 2003 23:25:46 -0500 (EST) From: Jeff Roberson To: Terry Lambert Cc: arch@FreeBSD.ORG Subject: Re: New getblk parameter. In-Reply-To: <3E62CB0D.92E9FF78@mindspring.com> Message-ID: <20030302232340.R84333-100000@mail.chesapeake.net> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Sun, 2 Mar 2003, Terry Lambert wrote: > Jeff Roberson wrote: > > I'd like to add a new parameter to getblk called 'flags'. The only flag > > I'm currently defining is GB_LOCK_NOWAIT so that it doesn't block trying > > to get the block. This is useful in the vfs_cluster code where we want to > > include a block in a cluster but only if it isn't currently in use. > > FWIW, I like it; the cleanup that results in kern/vfs_cluster.c > looks nice. Me too, thanks. > If you are not going to change all the calls to getblk(), it > should probably be a wrapper function, or, minimally, an inline > and a wrapper function. > > The reasoning is that it chould be called from precompiled > modules, so you want to leave a symbol visible for it, which > defining it to getblkf(..., 0) doesn't do. Precompiled modules are already going to be broken with the new locking semantics. I think requiring them to recompile is OK. I intend to bump the FreeBSD version if this goes in. I'd sort of like to change all the getblk() calls actually. If no one strongly objects to that I'll do it. Cheers, Jeff To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message