Re: packing race condition

Hyrum K. Wright wrote:
> Daniel Shahaf wrote:
>>>>> Added the use of the write lock in r35031.
>> ...
>>> Do you need acquire_fs_mutex()?
>> Ping? From reading fs-loader.c it seems to me that acquire_fs_mutex()
>> is only needed when common_pool is used. But svn_fs_pack() doesn't use
>> common_pool. Hence my question.
>>
>> (i.e., I'm not saying it's wrong the way it is; I'm only saying that I
>> don't see why *not* acquiring that mutex would be wrong.)
>
> I'm not sure. I simply emulated what I saw for upgrade. It sounds like you
> know more about this than I do. :)

Uhh... don't model after 'upgrade'. That was written by some FSFS-ignorant
dude out in Harrisburg, NC...

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1007706