Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 1 Mar 2006 23:20:41 -0800 (PST)
From:      Jeff Roberson <jroberson@chesapeake.net>
To:        Jeff Roberson <jeff@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/sys mount.h vnode.h src/sys/kern vfs_subr.c src/sys/ufs/ffs ffs_alloc.c ffs_extern.h ffs_snapshot.c         ffs_softdep.c softdep.h src/sys/ufs/ufs ufsmount.h
Message-ID:  <20060301231945.R557@10.0.0.1>
In-Reply-To: <200603020550.k225oN2E007232@repoman.freebsd.org>
References:  <200603020550.k225oN2E007232@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Er, sorry folks, there is a small bug in this that will take me a few 
minutes to get into the tree.  Must've snuck in after the last review 
changes which weren't given to kris.

Welcome to current. ;-)

On Thu, 2 Mar 2006, Jeff Roberson wrote:

> jeff        2006-03-02 05:50:23 UTC
>
>  FreeBSD src repository
>
>  Modified files:
>    sys/sys              mount.h vnode.h
>    sys/kern             vfs_subr.c
>    sys/ufs/ffs          ffs_alloc.c ffs_extern.h ffs_snapshot.c
>                         ffs_softdep.c softdep.h
>    sys/ufs/ufs          ufsmount.h
>  Log:
>   - Move softdep from using a global worklist to per-mount worklists.  This
>     has many positive effects including improved smp locking, reducing
>     interdependencies between mounts that can lead to deadlocks, etc.
>   - Add the softdep worklist and various counters to the ufsmnt structure.
>   - Add a mount pointer to the workitem and remove mount pointers from the
>     various structures derived from the workitem as they are now redundant.
>   - Remove the poor-man's semaphore protecting softdep_process_worklist and
>     softdep_flushworklist.  Several threads may now process the list
>     simultaneously.
>   - Add softdep_waitidle() to block the thread until all pending
>     dependencies being operated on by other threads have been flushed.
>   - Use softdep_waitidle() in unmount and snapshots to block either
>     operation until the fs is stable.
>   - Remove softdep worklist processing from the syncer and move it into the
>     softdep_flush() thread.  This thread processes all softdep mounts
>     once each second and when it is called via the new softdep_speedup()
>     when there is a resource shortage.  This removes the softdep hook
>     from the kernel and various hacks in header files to support it.
>
>  Reviewed by/Discussed with:     tegge, truckman, mckusick
>  Tested by:      kris
>
>  Revision  Changes    Path
>  1.661     +0 -10     src/sys/kern/vfs_subr.c
>  1.206     +0 -4      src/sys/sys/mount.h
>  1.314     +0 -1      src/sys/sys/vnode.h
>  1.139     +3 -3      src/sys/ufs/ffs/ffs_alloc.c
>  1.70      +4 -5      src/sys/ufs/ffs/ffs_extern.h
>  1.113     +7 -0      src/sys/ufs/ffs/ffs_snapshot.c
>  1.189     +313 -254  src/sys/ufs/ffs/ffs_softdep.c
>  1.19      +1 -6      src/sys/ufs/ffs/softdep.h
>  1.35      +8 -0      src/sys/ufs/ufs/ufsmount.h
>



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