Re: svn commit: r27097 - trunk/subversion/libsvn_fs_fs

On 11/6/07, David Glasser <glasser@davidglasser.net> wrote:
> On 10/11/07, Malcolm Rowe <malcolm-svn-dev@farside.org.uk> wrote:
> > On Wed, Oct 10, 2007 at 04:02:37PM -0700, glasser@tigris.org wrote:
> > > Move the cache of *immutable* DAG noderevs from revision roots to the
> > > svn_fs_t object. This extends the lifetime of cache entries; in my
> > > test, a somewhat large diff operation on a repository in NFS got a 4x
> > > speedup. Transaction DAG noderevs are still in the roots (though
> > > perhaps this should change).
> > >
> >
> > Nice change! Is it worth considering whether this cache can go into the
> > per-filesystem object (fs_fs_shared_data_t) instead of the
> > per-filesystem-handle object (fs_fs_data_t)?
> >
> > Obviously you'd have to add some locking, but it would help to ensure
> > that we can still benefit from the cache when we open a new handle.
>
> That would work, but as you mentioned would require mutex overhead for
> threaded builds; one would need to benchmark multithreaded servers to
> see if it ends up being a win or a loss.

I added a comment noting the possibility in r27638.

--dave

-- 
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org