From owner-freebsd-current Mon Jan 6 2:13:14 2003 Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8AE3637B401; Mon, 6 Jan 2003 02:13:13 -0800 (PST) Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by mx1.FreeBSD.org (Postfix) with ESMTP id 1613A43EC2; Mon, 6 Jan 2003 02:13:12 -0800 (PST) (envelope-from bde@zeta.org.au) Received: from katana.zip.com.au (katana.zip.com.au [61.8.7.246]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id VAA24071; Mon, 6 Jan 2003 21:12:32 +1100 Date: Mon, 6 Jan 2003 21:12:56 +1100 (EST) From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: Nate Lawson Cc: current@freebsd.org, Subject: Re: specfs lock plumbing broken In-Reply-To: Message-ID: <20030106204958.R2459-100000@gamplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Sun, 5 Jan 2003, Nate Lawson wrote: > On Mon, 6 Jan 2003, Bruce Evans wrote: > > - spec_print() is of low quality: it doesn't print the device name or number. > > - devfs_print() would be reachable but doesn't exist, so vprint() prints > > even lower quality output for devfs since there nothing prints an inode > > number either. > > I was the one who left vprint in a not-so-desirable state. I plan to fix > it very soon if you can tell me what info should be printed at what > layer. For instance, several fs's print the device but this is probably > unnecessary since specfs could do this. Care to elaborate? You didn't break this :-). Printing \n\t before VOP_PRINT() in vprint() works poorly for printing the output in log files. It would be better to have everything on one line for grepping on the string in the vprint() call. ufs only prints the device of the file system. Most file systems need to do that for themself since even having a device for the file system is fs-dependent. OTOH, v_rdev is in the vnode and there are fs-independent ways to get its name[s], so it can be printed directly by vprint() However, I prefer to let lower layers handle it. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message