> From: kfogel@collab.net [mailto:kfogel@collab.net]
> Sent: Friday, August 22, 2003 8:44 PM
> Hi, Paul, welcome back. As you know, there's one issue left in 0.28:
> issue #1429 ("slow checkouts"). Sander Striker's been working on it,
> and has learned that it's not a matter of one single factor that we
> can just *zap* to get faster checkouts. It's about making several
> improvements.
Indeed it is. libsvn_wc and libsvn_ra_dav are on the list. For the
rest it is about overall little inefficiencies accumulating.
> He's going to file a separate new issue for each improvement, and then
> use Saturday to work on one of them (probably some libsvn_wc
> efficiency improvements).
My profile* just showed some data that made me freak, since it tells
me it has a function, apr_allocator_set_mutex, that's being called
about a half a million times, but there is no reference in the source
anywhere(!). It appears that it happens when I let gprof attribute
costs to the exported api alone (the -a switch).
I'm going to call it a night, sleep on it and give it another go in
the morning.
From what I've seen this week, the xml related code was pretty prominently
present when added up. As a data point: xml_escape_attr alone was taking
almost 2% of the total time.
Anyway, moving from xml to something else isn't something I'll be attacking
in a measly sunday morning...
Another thing is the md5 checksums. We might want to think about a
little less CPU intensive algorithm, since this one is up there aswell.
MD5Transform (which is internal to apr, called by apr_md5_update) is taking
~15% of total checkout time over ra_local. It's at the very top of the
flat profile.
> So watch issue #1429, and the commit logs.
>
> I guess if Sander has any particular instructions about which revision
> to use (or not use) for the release branch, he'll post them here.
Just use rev 6841. Any changes I may be committing this weekend
would have too little world exposure to include them in the release.
Sander
*) checkout over ra_local of the mozilla tree. The subversion tree
is too small to see differences between times for a lot of calls,
hence the large tree.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Aug 24 00:36:57 2003