From owner-freebsd-fs Fri Feb 27 11:58:23 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id LAA17469 for freebsd-fs-outgoing; Fri, 27 Feb 1998 11:58:23 -0800 (PST) (envelope-from owner-freebsd-fs@FreeBSD.ORG) Received: from sumatra.americantv.com (sumatra.americantv.com [207.170.17.37]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id LAA17464 for ; Fri, 27 Feb 1998 11:58:18 -0800 (PST) (envelope-from jlemon@americantv.com) Received: from right.PCS (right.PCS [148.105.10.31]) by sumatra.americantv.com (8.8.5/8.8.5) with ESMTP id NAA22981; Fri, 27 Feb 1998 13:58:05 -0600 (CST) Received: (from jlemon@localhost) by right.PCS (8.6.13/8.6.4) id NAA12914; Fri, 27 Feb 1998 13:57:33 -0600 Message-ID: <19980227135733.19894@right.PCS> Date: Fri, 27 Feb 1998 13:57:33 -0600 From: Jonathan Lemon To: Eivind Eklund Cc: fs@FreeBSD.ORG, jlemon@americantv.com Subject: Re: syncer / SMP question References: <19980227164859.25557@follo.net> <19980227102555.05064@right.PCS> <19980227190132.53798@follo.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 0.61.1 In-Reply-To: <19980227190132.53798@follo.net>; from Eivind Eklund on Feb 02, 1998 at 07:01:32PM +0100 Sender: owner-freebsd-fs@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org On Feb 02, 1998 at 07:01:32PM +0100, Eivind Eklund wrote: > Yes, I later noticed. What do you think of the above? I think I'll take the original code. :-) While layering is nice, it imposes a performance hit. Also, I think that the above is overkill, and just serves to obfuscate the code. It took me longer to understand the macros above than it did to read the original implementation. IMHO, all the original code really needs is a nice little comment stating that vfs_busy() releases the lock on the mountlist if it succeeds. I'll also note that it appears possible to pass a NULL third argument to vfs_busy() and thus have it not release the lock on the mountlist. Then simple_unlock would be called explicitly in the sync() function, also making things clearer. I'm unsure if the lock release done by vfs_busy/lockmgr was for 1) convenience, or 2) some form of atomic unlocking. Perhaps someone else could explain? Terry? Poul? -- Jonathan To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-fs" in the body of the message