Recursive mutex todo item

Thiago Macieira thiago at kde.org
Fri Oct 27 01:53:15 PDT 2006


Havoc Pennington wrote:
>Hi,
>
>If people could look this over and see if it looks sane, that would be
>good. I haven't tested it.

The patch looks sane to me. I'd just recommend making these two new 
variables volatile:

>diff -u -p -r1.2 dbus-sysdeps-pthread.c
>--- dbus/dbus-sysdeps-pthread.c 27 Oct 2006 02:17:42 -0000      1.2
>+++ dbus/dbus-sysdeps-pthread.c 27 Oct 2006 03:14:48 -0000
>@@ -31,6 +31,8 @@
> 
> typedef struct {
>   pthread_mutex_t lock;
>+  int count;
>+  pthread_t holder;

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/dbus/attachments/20061027/daf7fb52/attachment-0001.pgp


More information about the dbus mailing list