[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1591919 - in /subversion/trunk/subversion: include/ include/private/ libsvn_fs/ libsvn_fs_base/bdb/ libsvn_fs_fs/ libsvn_fs_x/ libsvn_ra_svn/ libsvn_subr/ svnserve/ tests/ tests/libsvn_fs_fs/

From: Stefan Sperling <stsp_at_elego.de>
Date: Mon, 5 May 2014 14:06:03 +0200

On Fri, May 02, 2014 at 02:07:40PM -0000, stefan2_at_apache.org wrote:
> Author: stefan2
> Date: Fri May 2 14:07:40 2014
> New Revision: 1591919
>
> URL: http://svn.apache.org/r1591919
> Log:
> APR mutexes don't support recursive locking on all platforms.
> As a result, trying to take out the same lock twice in the
> same thread will cause a lock up under e.g. Linux. This patch
> adds an option to svn_mutex__t that detects recursive locking
> attempts in most cases and returns a proper error.

> * subversion/tests/libsvn_fs_fs/fs-fs-pack-test.c
> (never_reached,
> lock_again): Callback functions required by the new test.
> (recursive_locking): New test expecting an SVN_ERR_RECURSIVE_LOCK.
> (test_funcs): Register the new test.

Hi Stefan,

This new test fails on the bb-openbsd bot.
http://ci.apache.org/builders/bb-openbsd/builds/12/steps/Test/logs/faillog-ra_local-fsfs

Can you please look into this?

Note that the bot is deliberately compiling APR and everything else
without support for threads. Perhaps the test or other code doesn't
take this possibility into account?

Let me know if you need help.

Thanks!
Received on 2014-05-05 14:06:50 CEST

This is an archived mail posted to the Subversion Dev mailing list.