[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1588651 - in /subversion/trunk/subversion/libsvn_fs_fs: caching.c fs.c fs_fs.h

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Sat, 19 Apr 2014 19:12:16 +0400

On 19 April 2014 16:44, <stefan2_at_apache.org> wrote:
> Author: stefan2
> Date: Sat Apr 19 12:44:07 2014
> New Revision: 1588651
>
> URL: http://svn.apache.org/r1588651
> Log:
> Reduce the size of an FSFS instance by using temporary pools
> for temp. data during fs_open().
>
> * subversion/libsvn_fs_fs/fs_fs.h
> (svn_fs_fs__initialize_caches): Document that the POOL shall be
> used for temporaries only.
>
> * subversion/libsvn_fs_fs/caching.c
> (svn_fs_fs__initialize_caches): Fix the only place where we used
> POOL instead of FS->POOL for
> something with svn_fs_t lifetime.
>
> * subversion/libsvn_fs_fs/fs.c
> (fs_serialized_init): Document POOL usage that we will now rely on.
> (fs_open): Use a SUBPOOL for anything temporary during FS init.
>
Just creating subpool for temporary allocation violates pool design
imho. Proper fix would be add scratch pool parameter to
svn_fs_open()/svn_fs_create().

-- 
Ivan Zhakov
CTO | VisualSVN | http://www.visualsvn.com
Received on 2014-04-19 17:13:09 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.