Daniel Shahaf wrote on Wed, 14 Apr 2010 at 23:52 +0300:
> Philip Martin wrote on Wed, 14 Apr 2010 at 12:17 +0100:
> > -svn_error_t *
> > -svn_fs_fs__open_rep_cache(svn_fs_t *fs,
> > - apr_pool_t *pool)
> > {
> > - /* Be idempotent. */
> > + /* Be idempotent. ### This is not thread safe. */
> > if (ffd->rep_cache_db)
> > return SVN_NO_ERROR;
> >
>
> I suppose we could fix the thread-unsafety while we're here. Something like
> this?
>
> [[[
> In FSFS, open the FS object's rep-cache atomically. Extend fs_fs_data_t
> and svn_atomic__init_once() to support this.
Committed in r934599 and r934603, and nominated for backport along with
the #3506 revisions introducing the bug.
Daniel
(probably need some coffee)
