[erlang-patches] Remove mutex lock around sys_gettimeofday() in os:timestamp/0

Tuncer Ayaz tuncer.ayaz@REDACTED
Sat Jan 23 11:32:17 CET 2010


2010/1/23 Tuncer Ayaz <tuncer.ayaz@REDACTED>:
> 2010/1/23 Björn Gustavsson <bgustavsson@REDACTED>:
>> On Sat, Jan 23, 2010 at 12:41 AM, Tuncer Ayaz <tuncer.ayaz@REDACTED> wrote:
>>> Remove mutex lock around sys_gettimeofday() in os:timestamp/0
>>>
>>> In the erlang-questions thread "os:timestamp() uses mutex lock"
>>> Jonas Falkevik questioned the need for a mutex lock in os:timestamp/0.
>>> Kenneth Lundin confirmed that the lock is not needed and asked for
>>> a patch.
>>>
>>> git fetch git://github.com/tuncer/otp.git os_timestamp_no_mutex
>>
>> Thanks!
>>
>> Since you seems to have based your branch on 'pu', I have rebased
>> your branch on ccase/r13b04_dev.
>
> Thanks, I wondered why Kenneth wrote
> "provide it as a patch to the pu branch"
> and due to that confusion branched off pu.
> My last patch was branched off ccase/r13b04_dev :).
>
>> Since I have touched the commit anyway, I also edited the
>> commit message to be more informative regarding the
>> technical details:
>>
>>    In the erlang-questions thread "os:timestamp() uses mutex lock"
>>    Jonas Falkevik questioned the need for a mutex lock in
>>    os:timestamp/0. The mutex lock *is* needed in erlang:now()
>>    to guarantee the uniqueness of the returned time, but serves
>>    no useful purpose in os:timestamp().
>>
>> If you are not happy with this change, please edit the text and
>> submit the branch again.
>
> I can redo the branch with the changed commit message
> if you prefer.

Rebuilt the branch with the changes.


More information about the erlang-patches mailing list