Hi,
> The problem is now that the daemon is leaking 256 bytes of memory on
> every PNG production. I analysed the problem with an OS X tool called
> MallocDebug, and it turned out that the leaked memory was allocated in
> pixman. If cairo/pixman is invoked from the main thread only, then I
> observe no leaking. Also a first time invocation from any thread does
> not result in leaking. If cairo/pixman is invoked from a spawned
> thread, then the leak occurs, once the thread is finished.
Thanks for reporting the bug.
> I read something about pixman utilizing TLS (thread local strorage),
> and I suspected that the observed leaking may be somehow related, but
> my config.log tells us:
>
> conftest.c:31: error: thread-local storage not supported for this
> target
That error message is somewhat misleading. It is a GCC error that _it_
doesn't support the use of __thread on Mac OS X.
Pixman instead uses pthreads for thread local storage on Mac OS X, and
the problem you are seeing is related to that. Basically we need to
free the stored data when a thread exists.
The patch included is an attempt at fixing this. If you can test it,
that would be appreciated. There is a tarball here:
http://www.daimi.au.dk/~sandmann/pixman-0.19.1.tar.gz
which has the patch included.
It seems we have the same problem on Windows. Unfortunately, I don't
really have any idea how to fix it there. If anyone can help out, that
would be appreciated.
Thanks,
Soren
>From 2e22e3435e1c1171c7ae840b4bf017b16e1fe52a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B8ren=20Sandmann=20Pedersen?= <[email protected]>
Date: Wed, 30 Jun 2010 02:31:10 -0400
Subject: [PATCH] Fix memory leak in the pthreads thread local storage code
When a thread exits, we leak whatever is stored in thread local
variables, so install a destructor to free it.
---
pixman/pixman-compiler.h | 13 ++++++++++---
1 files changed, 10 insertions(+), 3 deletions(-)
diff --git a/pixman/pixman-compiler.h b/pixman/pixman-compiler.h
index f0f9d91..2b15cc3 100644
--- a/pixman/pixman-compiler.h
+++ b/pixman/pixman-compiler.h
@@ -72,9 +72,9 @@
/* TLS */
#if defined(PIXMAN_NO_TLS)
-# define PIXMAN_DEFINE_THREAD_LOCAL(type, name) \
+# define PIXMAN_DEFINE_THREAD_LOCAL(type, name) \
static type name
-# define PIXMAN_GET_THREAD_LOCAL(name) \
+# define PIXMAN_GET_THREAD_LOCAL(name) \
(&name)
#elif defined(TOOLCHAIN_SUPPORTS__THREAD)
@@ -165,9 +165,16 @@ extern __stdcall int ReleaseMutex (void *);
static pthread_key_t tls_ ## name ## _key; \
\
static void
\
+ tls_ ## name ## _destroy_value (void *value) \
+ { \
+ free (value); \
+ } \
+ \
+ static void
\
tls_ ## name ## _make_key (void) \
{ \
- pthread_key_create (&tls_ ## name ## _key, NULL); \
+ pthread_key_create (&tls_ ## name ## _key, \
+ tls_ ## name ## _destroy_value); \
} \
\
static type * \
--
1.7.0.1
_______________________________________________
Pixman mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pixman
