Date: Wed, 18 Jun 2003 15:22:26 +0400 From: Dmitry Sivachenko <demon@freebsd.org> To: "Tim J. Robbins" <tjr@freebsd.org> Cc: cvs-src@freebsd.org Subject: Re: cvs commit: src/sys/fs/nullfs null.h null_subr.c null_vnops.c Message-ID: <20030618112226.GA42606@fling-wing.demos.su> In-Reply-To: <20030618200325.A3179@dilbert.robbins.dropbear.id.au> References: <200306170852.h5H8qjgg087299@repoman.freebsd.org> <20030617133130.GF38547@garage.freebsd.pl> <20030618200325.A3179@dilbert.robbins.dropbear.id.au>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jun 18, 2003 at 08:03:25PM +1000, Tim J. Robbins wrote: > On Tue, Jun 17, 2003 at 03:31:30PM +0200, Pawel Jakub Dawidek wrote: > > > Great work! > > > > You susspect there are more problems with nullfs? > > This file system looks like a very simple thing, maybe it's implementation > > is too complicate? > > > > I'm not sure, but if we forgot about mount flags, etc. (something like > > hardlink to directory) we only have to do one thing: return correct vnode on: > > > > # cd /mnt/null/.. > > > > Every other operation inside nullfs should be done with functions from > > original file system. > > > > Maybe I'm talking stupid things here, but those two file systems are really > > helpfull (I'm talking also about unionfs) and it will be great if there > > will be no BUGS section in manuals for those file systems. > > The main problems with nullfs seem to be locking and trying to create clones > of the lower vnode (wrt. the VM system and special files). Once kern/51583 BTW, what is the reason for creating these clone vnodes? Why we can't simply return the original vnode? > is fixed and I've stress-tested nullfs a bit more, I'll probably be confident > enough in it to remove the BUGS section. I can't really comment on unionfs... > I plan to test it out soon and see whether any of the recent nullfs bugfixes > could apply to it, esp. the deadlock one. >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030618112226.GA42606>