Philip Martin <philip.mar...@wandisco.com> writes:
> Stefan Sperling <s...@elego.de> writes:
>
>> On Mon, Feb 07, 2011 at 11:51:11AM +0000, Philip Martin wrote:
>>> stef...@apache.org writes:
>>>
>>> > Author: stefan2
>>> > Date: Sun Feb 6 15:51:15 2011
>>> > New Revision: 1067687
>>> >
>>> > URL: http://svn.apache.org/viewvc?rev=1067687&view=rev
>>> > Log:
>>> > Merged latest caching bug fixes from performance branch:
>>> > revisions 1029232, 1032333, 1033040, 1033057 and 1033294
>>> > (support for "no threads", typos, error leaks)
>>>
>>> This commit* appears to cause a huge increase in Apache memory use when
>>> running the regression tests, enough to make the machine unuseable and
>>> prevent the tests running.
>>>
>>> [*]Strictly it's r1067712 since r1067687 doesn't build, but I think
>>> r1067687 is the one that causes the problem.
>>
>> My buildbot seems to be affected by this, too.
>>>From the web gui I can tell that it's currently running neon tests.
>> And it's responding to pings... but I cannot ssh into it.
>> ssh can establish a TCP connection but then it hangs.
>
> It appears to be this part that causes the problem:
HA! No, I was dealing with reversed patches. I think this is the
correct fix:
Index: subversion/libsvn_fs_util/caching.c
===================================================================
--- subversion/libsvn_fs_util/caching.c (revision 1067926)
+++ subversion/libsvn_fs_util/caching.c (working copy)
@@ -86,7 +86,7 @@
pool = svn_pool_create_ex(NULL, allocator);
svn_error_clear(svn_cache__membuffer_cache_create(
- &old_cache,
+ &new_cache,
(apr_size_t)cache_size,
(apr_size_t)(cache_size / 16),
! svn_fs_get_cache_config()->single_threaded,
--
Philip
