On 04.02.2011 09:08, Neil Bird wrote:
> Around about 03/02/11 17:01, Stefan Sperling typed ...
>>> * subversion/libsvn_subr/io.c
>>> svn_io_open_unique_file3: don't call svn_io_open_uniquely_named(),
>>> but instead use a copy of that routine
>> Which routine, precisely?
>
> svn_io_open_uniquely_named(); sorry, I thought that was clear enough.
I had a similar patch in place on the performance branch.
Due to changes in the calling functions, the extra randomization
has now become unnecessary.
>
>> Have you taken a look at the implementation of
>> svn_io_open_unique_file3()
>> in Subversion's trunk code?
>
> Yes, and in fact that was what I initially tried, but (with
> admittedly little effort) it wouldn't compile for several reasons that
> seemed to be down to other API changes that weren't in 1.6.x (apr ones
> IIRC).
>
> I'll give it another bash, though , if you think it's worth it.
>
Definitely. It contains quite a number of file access
optimizations that should become best visible on
"high overhead" FS like NTFS.
-- Stefan^2.
Received on 2011-02-04 11:53:37 CET