Re: Memory leak in svn_ra_get_file() with svn+ssh://?

On Saturday 06 October 2007 David Glasser wrote:
> On 10/6/07, Philipp Marek <philipp@marek.priv.at> wrote:
> > Could you *please* *please* *please* change the read size to something
> > higher - how about 128k instead of the 4k we currently have?
> >
> > Similar for the ra_file layer ... there's 16k, but I think that is much
> > too low for current machines; we loose too much on context switches.
>
> This seems like the sort of thing that is most convincing with a
> benchmark. Care to write up a script we can run against servers of
> various block sizes and see which is most efficient?
How about using the recipe I had for the memory leak, possibly with a bigger
file? Using null bytes instead of random data would make no difference here,
too.

Example:
        svnadmin create test
        svn co file:///.../test wc
        dd if=/dev/null of=wc/1 bs=1024k seek=640
        svn add wc/1
        svn ci wc -m1

And then do
        time svn cat file:///...test/1 > /dev/null
with different block sizes.

(Or better, test with svn+ssh:// too).

Regards,

Phil

-- 
Versioning your /etc, /home or even your whole installation?
             Try fsvs (fsvs.tigris.org)!
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org