Re: Some tips on profiling

Hi Stefan,

Stefan Fuhrmann writes:
> You obviously don't have APR threads enabled.
> Thanks for finding this. Fixed in r1003430.

I enabled it, but there's still some issue:
subversion/svnadmin/main.c:1892: undefined reference to `svn_fs_get_cache_config'

> The server caches almost everything. So, the
> first ("cold") run demonstrates the worst-case,
> the second run ("hot") the best case. Real
> world performance depends on server memory
> and load.

Ah. Thanks for the clarification.

> For the merge part: please review the "integrate-membuffer"
> branch (only 3 patches). You may also review and
> merge any of the patches listed in /branches/performance/STATUS.

integrate-cache-membuffer, you mean? Thanks! Your emails contains
references exactly to the patches I'm looking for :)

> For the MD5 stuff, try r986459 (performance branch).
> It should eliminate 1 of the 3 MD5 calculations.

Why doesn't STATUS document this and everything else consistently?

> As for the temp files, there are some improvements
> listed in /branches/performance/STATUS. These would
> also reduce your "system" CPU time.

I had the chance to check them out and apply them just now. They work
as expected. I'll submit some (uneducated) patch reviews to the list
and request a merge. I don't have access to the areas your patches
touch.

-- Ram
Received on 2010-10-01 15:58:44 CEST