Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 13 Feb 2007 17:59:40 +0100
From:      Harald Schmalzbauer <harry@schmalzbauer.de>
To:        cvs-src@freebsd.org
Cc:        Craig Rodrigues <rodrigc@freebsd.org>, src-committers@freebsd.org, Kevin Oberman <oberman@es.net>
Subject:   Re: cvs commit: src/sys/fs/unionfs union.h union_subr.c union_vfsops.c union_vnops.c src/sbin/mount_unionfs Makefile mount_unionfs.8 mount_unionfs.c
Message-ID:  <200702131759.41110.harry@schmalzbauer.de>
In-Reply-To: <20070213152252.0E7CB4507B@ptavv.es.net>
References:  <20070213152252.0E7CB4507B@ptavv.es.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Am Dienstag, 13. Februar 2007 16:22 schrieb Kevin Oberman:
> > From: Craig Rodrigues <rodrigc@FreeBSD.org>
> > Date: Tue, 13 Feb 2007 05:56:43 +0000 (UTC)
> > Sender: owner-cvs-all@freebsd.org
> >
> > rodrigc     2007-02-13 05:56:43 UTC
> >
> >   FreeBSD src repository
> >
> >   Modified files:        (Branch: RELENG_6)
> >     sys/fs/unionfs       union.h union_subr.c union_vfsops.c
> >                          union_vnops.c
> >     sbin/mount_unionfs   Makefile mount_unionfs.8 mount_unionfs.c
> >   Log:
> >   MFC: New unionfs implementation from Daichi GOTO and Masanori OZAWA,
> >        which fixes many locking and crashing problems in the previous
> >        implementation.  kib@ helped a lot by eliminating a source
> >        of deadlocks encountered with unionfs with these commits:
> >          rev. 1.50 of src/sys/fs/deadfs/dead_vnops.c
> >          rev. 1.97 of src/sys/kern/vfs_lookup.c
> >   Submitted by:   daichi, Masanori OZAWA <ozawa ongs co jp>
> >
> >   Revision   Changes      Path
> >   1.12.8.1   +1 -1        src/sbin/mount_unionfs/Makefile
> >   1.20.2.1   +198 -25     src/sbin/mount_unionfs/mount_unionfs.8
> >   1.24.2.1   +108 -74     src/sbin/mount_unionfs/mount_unionfs.c
> >   1.31.2.1   +88 -94      src/sys/fs/unionfs/union.h
> >   1.86.2.1   +937 -1080   src/sys/fs/unionfs/union_subr.c
> >   1.76.2.2   +398 -334    src/sys/fs/unionfs/union_vfsops.c
> >   1.132.2.1  +1879 -1607  src/sys/fs/unionfs/union_vnops.c
>
> Thanks for MFCing this work and to the authors who put all the work into
> actually making unionfs work as it should. As a regular user of unionfs
> (at this moment /etc is a unionfs on the system from which I am sending
> this), it's great that I can worry less about blowing off my foot if I
> am not VERY careful what operations I do with the unionfs filesystem.

Full ACK. Thanks a lot, especially to the coders! First thing after cvsup was 
to patch unionfs, now not neccessarry anymore :)

-Harry



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200702131759.41110.harry>