cairo: add 1.4.4 + Chris Wilsons workqueue patch

3 files changed, 2155 insertions, 0 deletions

diff --git a/packages/cairo/cairo-1.4.4/.mtn2git_empty b/packages/cairo/cairo-1.4.4/.mtn2git_empty
new file mode 100644
index 0000000000..e69de29bb2
--- /dev/null
+++ b/packages/cairo/cairo-1.4.4/.mtn2git_empty
diff --git a/packages/cairo/cairo-1.4.4/cairo-workqueue.patch b/packages/cairo/cairo-1.4.4/cairo-workqueue.patch
new file mode 100644
index 0000000000..4522e99e68
--- /dev/null
+++ b/packages/cairo/cairo-1.4.4/cairo-workqueue.patch
@@ -0,0 +1,2148 @@
+From: Chris Wilson <chris@chris-wilson.co.uk>
+Date: Wed, 18 Apr 2007 17:34:44 +0000 (+0100)
+Subject: Cache freed GCs
+X-Git-Url: http://gitweb.freedesktop.org/?p=users/ickle/cairo;a=commitdiff;h=3126c3948691cf7a6c0fc0f9cc6a7eca43c3553b
+
+Cache freed GCs
+
+Maintain a cache of freed GCs, one for each used depth.
+---
+
+--- a/src/Makefile.am
++++ b/src/Makefile.am
+@@ -46,7 +46,11 @@ endif
+ 
+ if CAIRO_HAS_XLIB_SURFACE
+ libcairo_xlib_headers = cairo-xlib.h cairo-xlib-xrender.h
+-libcairo_xlib_sources = cairo-xlib-surface.c cairo-xlib-screen.c cairo-xlib-private.h cairo-xlib-test.h
++libcairo_xlib_sources = cairo-xlib-surface.c \
++			cairo-xlib-display.c \
++			cairo-xlib-screen.c \
++			cairo-xlib-private.h \
++			cairo-xlib-test.h
+ backend_pkgconfigs += cairo-xlib.pc
+ endif
+ 
+--- a/src/cairo-clip.c
++++ b/src/cairo-clip.c
+@@ -404,7 +404,8 @@ _cairo_clip_intersect_mask (cairo_clip_t
+ 						   CAIRO_CONTENT_ALPHA,
+ 						   surface_rect.width,
+ 						   surface_rect.height,
+-						   CAIRO_COLOR_WHITE);
++						   CAIRO_COLOR_WHITE,
++						   NULL);
+     if (surface->status)
+ 	return CAIRO_STATUS_NO_MEMORY;
+ 
+--- a/src/cairo-debug.c
++++ b/src/cairo-debug.c
+@@ -59,10 +59,6 @@
+ void
+ cairo_debug_reset_static_data (void)
+ {
+-#if CAIRO_HAS_XLIB_SURFACE
+-    _cairo_xlib_screen_reset_static_data ();
+-#endif
+-
+     _cairo_font_reset_static_data ();
+ 
+ #if CAIRO_HAS_FT_FONT
+--- a/src/cairo-directfb-surface.c
++++ b/src/cairo-directfb-surface.c
+@@ -1511,6 +1511,16 @@ _cairo_directfb_surface_show_glyphs ( vo
+ #endif /* DFB_SHOW_GLYPHS */
+ 
+ 
++static cairo_bool_t
++_cairo_directfb_surface_is_compatible (void *surface_a,
++	                               void *surface_b)
++{
++    cairo_directfb_surface_t *a = (cairo_directfb_surface_t *) surface_a;
++    cairo_directfb_surface_t *b = (cairo_directfb_surface_t *) surface_b;
++
++    return a->dfb == b->dfb;
++}
++
+ static cairo_surface_backend_t cairo_directfb_surface_backend = {
+          CAIRO_SURFACE_TYPE_DIRECTFB, /*type*/
+         _cairo_directfb_surface_create_similar,/*create_similar*/
+@@ -1560,7 +1570,8 @@ static cairo_surface_backend_t cairo_dir
+ #else
+         NULL, /* show_glyphs */
+ #endif
+-        NULL /* snapshot */
++        NULL, /* snapshot */
++	_cairo_directfb_is_compatible
+ };
+ 
+ 
+--- a/src/cairo-glitz-surface.c
++++ b/src/cairo-glitz-surface.c
+@@ -1002,7 +1002,8 @@ _cairo_glitz_surface_fill_rectangles (vo
+ 	    _cairo_surface_create_similar_solid (&dst->base,
+ 						 CAIRO_CONTENT_COLOR_ALPHA,
+ 						 1, 1,
+-						 (cairo_color_t *) color);
++						 (cairo_color_t *) color,
++						 NULL);
+ 	if (src->base.status)
+ 	    return CAIRO_STATUS_NO_MEMORY;
+ 
+@@ -2171,6 +2172,19 @@ _cairo_glitz_surface_flush (void *abstra
+     return CAIRO_STATUS_SUCCESS;
+ }
+ 
++static cairo_bool_t
++_cairo_glitz_surface_is_compatible (void *surface_a,
++	                            void *surface_b)
++{
++    cairo_glitz_surface_t *a = (cairo_glitz_surface_t *) surface_a;
++    cairo_glitz_surface_t *b = (cairo_glitz_surface_t *) surface_b;
++
++    glitz_drawable_t *drawable_a = glitz_surface_get_drawable (a->surface);
++    glitz_drawable_t *drawable_b = glitz_surface_get_drawable (b->surface);
++
++    return drawable_a == drawable_b;
++}
++
+ static const cairo_surface_backend_t cairo_glitz_surface_backend = {
+     CAIRO_SURFACE_TYPE_GLITZ,
+     _cairo_glitz_surface_create_similar,
+@@ -2193,7 +2207,16 @@ static const cairo_surface_backend_t cai
+     _cairo_glitz_surface_flush,
+     NULL, /* mark_dirty_rectangle */
+     _cairo_glitz_surface_scaled_font_fini,
+-    _cairo_glitz_surface_scaled_glyph_fini
++    _cairo_glitz_surface_scaled_glyph_fini,
++
++    NULL, /* paint */
++    NULL, /* mask */
++    NULL, /* stroke */
++    NULL, /* fill */
++    NULL, /* show_glyphs */
++
++    NULL, /* snapshot */
++    _cairo_glitz_surface_is_compatible
+ };
+ 
+ static const cairo_surface_backend_t *
+--- a/src/cairo-mutex-list-private.h
++++ b/src/cairo-mutex-list-private.h
+@@ -34,7 +34,8 @@
+ #ifndef CAIRO_MUTEX_LIST_PRIVATE_H
+ #define CAIRO_MUTEX_LIST_PRIVATE_H
+ 
+-CAIRO_MUTEX_DECLARE (_cairo_pattern_solid_cache_lock);
++CAIRO_MUTEX_DECLARE (_cairo_pattern_solid_pattern_cache_lock);
++CAIRO_MUTEX_DECLARE (_cairo_pattern_solid_surface_cache_lock);
+ 
+ CAIRO_MUTEX_DECLARE (_cairo_font_face_mutex);
+ CAIRO_MUTEX_DECLARE (_cairo_scaled_font_map_mutex);
+@@ -44,7 +45,7 @@ CAIRO_MUTEX_DECLARE (_cairo_ft_unscaled_
+ #endif
+ 
+ #if CAIRO_HAS_XLIB_SURFACE
+-CAIRO_MUTEX_DECLARE (_cairo_xlib_screen_mutex);
++CAIRO_MUTEX_DECLARE (_cairo_xlib_display_mutex);
+ #endif
+ 
+ #endif
+--- a/src/cairo-pattern.c
++++ b/src/cairo-pattern.c
+@@ -282,7 +282,7 @@ _cairo_pattern_create_solid (const cairo
+ {
+     cairo_solid_pattern_t *pattern = NULL;
+ 
+-    CAIRO_MUTEX_LOCK (_cairo_pattern_solid_cache_lock);
++    CAIRO_MUTEX_LOCK (_cairo_pattern_solid_pattern_cache_lock);
+ 
+     if (solid_pattern_cache.size) {
+ 	int i = --solid_pattern_cache.size %
+@@ -291,7 +291,7 @@ _cairo_pattern_create_solid (const cairo
+ 	solid_pattern_cache.patterns[i] = NULL;
+     }
+ 
+-    CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_cache_lock);
++    CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_pattern_cache_lock);
+ 
+     if (pattern == NULL) {
+ 	/* None cached, need to create a new pattern. */
+@@ -306,12 +306,12 @@ _cairo_pattern_create_solid (const cairo
+     return &pattern->base;
+ }
+ 
+-void
+-_cairo_pattern_reset_static_data (void)
++static void
++_cairo_pattern_reset_solid_pattern_cache (void)
+ {
+     int i;
+ 
+-    CAIRO_MUTEX_LOCK (_cairo_pattern_solid_cache_lock);
++    CAIRO_MUTEX_LOCK (_cairo_pattern_solid_pattern_cache_lock);
+ 
+     for (i = 0; i < MIN (ARRAY_LENGTH (solid_pattern_cache.patterns), solid_pattern_cache.size); i++) {
+ 	free (solid_pattern_cache.patterns[i]);
+@@ -319,7 +319,7 @@ _cairo_pattern_reset_static_data (void)
+     }
+     solid_pattern_cache.size = 0;
+ 
+-    CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_cache_lock);
++    CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_pattern_cache_lock);
+ }
+ 
+ static const cairo_pattern_t *
+@@ -630,7 +630,7 @@ cairo_pattern_destroy (cairo_pattern_t *
+     if (pattern->type == CAIRO_PATTERN_TYPE_SOLID) {
+ 	int i;
+ 
+-	CAIRO_MUTEX_LOCK (_cairo_pattern_solid_cache_lock);
++	CAIRO_MUTEX_LOCK (_cairo_pattern_solid_pattern_cache_lock);
+ 
+ 	i = solid_pattern_cache.size++ %
+ 	    ARRAY_LENGTH (solid_pattern_cache.patterns);
+@@ -640,7 +640,7 @@ cairo_pattern_destroy (cairo_pattern_t *
+ 
+ 	solid_pattern_cache.patterns[i] = (cairo_solid_pattern_t *) pattern;
+ 
+-	CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_cache_lock);
++	CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_pattern_cache_lock);
+     } else {
+ 	free (pattern);
+     }
+@@ -1235,6 +1235,17 @@ _cairo_pattern_acquire_surface_for_gradi
+     return status;
+ }
+ 
++/* We maintain a small cache here, because we don't want to constantly
++ * recreate surfaces for simple solid colors. */
++#define MAX_SURFACE_CACHE_SIZE 16
++static struct {
++    struct {
++	cairo_color_t    color;
++	cairo_surface_t *surface;
++    } cache[MAX_SURFACE_CACHE_SIZE];
++    int size;
++} solid_surface_cache;
++
+ static cairo_int_status_t
+ _cairo_pattern_acquire_surface_for_solid (cairo_solid_pattern_t	     *pattern,
+ 					  cairo_surface_t	     *dst,
+@@ -1245,12 +1256,54 @@ _cairo_pattern_acquire_surface_for_solid
+ 					  cairo_surface_t	     **out,
+ 					  cairo_surface_attributes_t *attribs)
+ {
+-    *out = _cairo_surface_create_similar_solid (dst,
+-						CAIRO_CONTENT_COLOR_ALPHA,
+-						1, 1,
+-						&pattern->color);
+-    if ((*out)->status)
+-	return CAIRO_STATUS_NO_MEMORY;
++    static int i;
++
++    cairo_surface_t *surface;
++    cairo_status_t   status;
++
++    CAIRO_MUTEX_LOCK (_cairo_pattern_solid_surface_cache_lock);
++
++    /* Check cache first */
++    if (i < solid_surface_cache.size &&
++	    _cairo_color_equal (&solid_surface_cache.cache[i].color,
++		                &pattern->color) &&
++	    _cairo_surface_is_compatible (solid_surface_cache.cache[i].surface, dst))
++	    goto DONE;
++
++    for (i = 0 ; i < solid_surface_cache.size; i++) {
++	if (_cairo_color_equal (&solid_surface_cache.cache[i].color,
++		                &pattern->color) &&
++	    _cairo_surface_is_compatible (solid_surface_cache.cache[i].surface, dst))
++	    goto DONE;
++    }
++
++    /* Not cached, need to create new */
++    surface = _cairo_surface_create_similar_solid (dst,
++						   CAIRO_CONTENT_COLOR_ALPHA,
++						   1, 1,
++						   &pattern->color,
++						   pattern);
++    assert (_cairo_surface_is_compatible (surface, dst));
++    if (surface->status) {
++	status = surface->status;
++	goto UNLOCK;
++    }
++
++    /* Cache new */
++    if (solid_surface_cache.size < MAX_SURFACE_CACHE_SIZE) {
++	solid_surface_cache.size++;
++    } else {
++	i = rand () % MAX_SURFACE_CACHE_SIZE;
++
++	/* Evict old */
++	cairo_surface_destroy (solid_surface_cache.cache[i].surface);
++    }
++
++    solid_surface_cache.cache[i].color = pattern->color;
++    solid_surface_cache.cache[i].surface = surface;
++
++DONE:
++    *out = cairo_surface_reference (solid_surface_cache.cache[i].surface);
+ 
+     attribs->x_offset = attribs->y_offset = 0;
+     cairo_matrix_init_identity (&attribs->matrix);
+@@ -1258,7 +1311,26 @@ _cairo_pattern_acquire_surface_for_solid
+     attribs->filter = CAIRO_FILTER_NEAREST;
+     attribs->acquired = FALSE;
+ 
+-    return CAIRO_STATUS_SUCCESS;
++    status = CAIRO_STATUS_SUCCESS;
++
++UNLOCK:
++    CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_surface_cache_lock);
++
++    return status;
++}
++
++static void
++_cairo_pattern_reset_solid_surface_cache (void)
++{
++    int i;
++
++    CAIRO_MUTEX_LOCK (_cairo_pattern_solid_surface_cache_lock);
++
++    for (i = 0; i < solid_surface_cache.size; i++)
++	cairo_surface_destroy (solid_surface_cache.cache[i].surface);
++    solid_surface_cache.size = 0;
++
++    CAIRO_MUTEX_UNLOCK (_cairo_pattern_solid_surface_cache_lock);
+ }
+ 
+ /**
+@@ -2049,3 +2121,10 @@ cairo_pattern_get_radial_circles (cairo_
+ 
+     return CAIRO_STATUS_SUCCESS;
+ }
++
++void
++_cairo_pattern_reset_static_data (void)
++{
++    _cairo_pattern_reset_solid_pattern_cache ();
++    _cairo_pattern_reset_solid_surface_cache ();
++}
+--- a/src/cairo-surface.c
++++ b/src/cairo-surface.c
+@@ -288,7 +288,8 @@ cairo_surface_create_similar (cairo_surf
+ 
+     return _cairo_surface_create_similar_solid (other, content,
+ 						width, height,
+-						CAIRO_COLOR_TRANSPARENT);
++						CAIRO_COLOR_TRANSPARENT,
++						NULL);
+ }
+ slim_hidden_def (cairo_surface_create_similar);
+ 
+@@ -297,7 +298,8 @@ _cairo_surface_create_similar_solid (cai
+ 				     cairo_content_t	  content,
+ 				     int		  width,
+ 				     int		  height,
+-				     const cairo_color_t *color)
++				     const cairo_color_t *color,
++				     cairo_pattern_t     *pattern)
+ {
+     cairo_status_t status;
+     cairo_surface_t *surface;
+@@ -310,19 +312,23 @@ _cairo_surface_create_similar_solid (cai
+ 	return (cairo_surface_t*) &_cairo_surface_nil;
+     }
+ 
+-    source = _cairo_pattern_create_solid (color);
+-    if (source->status) {
+-	cairo_surface_destroy (surface);
+-	_cairo_error (CAIRO_STATUS_NO_MEMORY);
+-	return (cairo_surface_t*) &_cairo_surface_nil;
+-    }
++    if (pattern == NULL) {
++	source = _cairo_pattern_create_solid (color);
++	if (source->status) {
++	    cairo_surface_destroy (surface);
++	    _cairo_error (CAIRO_STATUS_NO_MEMORY);
++	    return (cairo_surface_t*) &_cairo_surface_nil;
++	}
++    } else
++	source = pattern;
+ 
+     status = _cairo_surface_paint (surface,
+ 				   color == CAIRO_COLOR_TRANSPARENT ?
+ 				   CAIRO_OPERATOR_CLEAR :
+ 				   CAIRO_OPERATOR_SOURCE, source);
+ 
+-    cairo_pattern_destroy (source);
++    if (source != pattern)
++	cairo_pattern_destroy (source);
+ 
+     if (status) {
+ 	cairo_surface_destroy (surface);
+@@ -1056,6 +1062,32 @@ _cairo_surface_snapshot (cairo_surface_t
+     return _cairo_surface_fallback_snapshot (surface);
+ }
+ 
++/**
++ * _cairo_surface_is_compatible
++ * @surface_a: a #cairo_surface_t
++ * @surface_b: a #cairo_surface_t
++ *
++ * Find out whether the given surfaces share the same backend,
++ * and if so, whether they can be considered compatible.
++ *
++ * The definition of "compatible" depends on the backend. In the
++ * xlib case, it means the surface share the same display.
++ *
++ * Return value: TRUE if the surfaces are compatible.
++ **/
++cairo_bool_t
++_cairo_surface_is_compatible (cairo_surface_t *surface_a,
++	                      cairo_surface_t *surface_b)
++{
++    if (surface_a->backend != surface_b->backend)
++	return FALSE;
++
++    if (surface_a->backend->is_compatible != NULL)
++	return surface_a->backend->is_compatible (surface_a, surface_b);
++
++    return TRUE;
++}
++
+ cairo_status_t
+ _cairo_surface_composite (cairo_operator_t	op,
+ 			  cairo_pattern_t	*src,
+--- a/src/cairo-win32-surface.c
++++ b/src/cairo-win32-surface.c
+@@ -1854,6 +1854,16 @@ cairo_win32_surface_get_image (cairo_sur
+     return ((cairo_win32_surface_t*)surface)->image;
+ }
+ 
++static cairo_bool_t
++_cairo_win32_surface_is_compatible (void *surface_a,
++	                            void *surface_b)
++{
++    cairo_win32_surface_t *a = (cairo_win32_surface_t *) surface_a;
++    cairo_win32_surface_t *b = (cairo_win32_surface_t *) surface_b;
++
++    return a->dc == b->dc;
++}
++
+ static const cairo_surface_backend_t cairo_win32_surface_backend = {
+     CAIRO_SURFACE_TYPE_WIN32,
+     _cairo_win32_surface_create_similar,
+@@ -1884,7 +1894,8 @@ static const cairo_surface_backend_t cai
+     NULL, /* fill */
+     _cairo_win32_surface_show_glyphs,
+ 
+-    NULL  /* snapshot */
++    NULL,  /* snapshot */
++    _cairo_win32_surface_is_compatible
+ };
+ 
+ /* Notes:
+--- a/src/cairo-xcb-surface.c
++++ b/src/cairo-xcb-surface.c
+@@ -1577,6 +1577,16 @@ _cairo_xcb_surface_show_glyphs (void    
+ 				 int		      num_glyphs,
+ 				 cairo_scaled_font_t *scaled_font);
+ 
++static cairo_bool_t
++_cairo_xcb_surface_is_compatible (void *surface_a,
++	                          void *surface_b)
++{
++    cairo_xcb_surface_t *a = (cairo_xcb_surface_t *) surface_a;
++    cairo_xcb_surface_t *b = (cairo_xcb_surface_t *) surface_b;
++
++    return _cairo_xcb_surface_same_screen (dst, src);
++}
++
+ /* XXX: move this to the bottom of the file, XCB and Xlib */
+ 
+ static const cairo_surface_backend_t cairo_xcb_surface_backend = {
+@@ -1608,7 +1618,8 @@ static const cairo_surface_backend_t cai
+     NULL, /* stroke */
+     NULL, /* fill */
+     _cairo_xcb_surface_show_glyphs,
+-    NULL  /* snapshot */
++    NULL,  /* snapshot */
++    _cairo_xcb_surface_is_compatible
+ };
+ 
+ /**
+--- /dev/null
++++ b/src/cairo-xlib-display.c
+@@ -0,0 +1,490 @@
++/* Cairo - a vector graphics library with display and print output
++ *
++ * Copyright © 2007 Chris Wilson
++ *
++ * This library is free software; you can redistribute it and/or
++ * modify it either under the terms of the GNU Lesser General Public
++ * License version 2.1 as published by the Free Software Foundation
++ * (the "LGPL") or, at your option, under the terms of the Mozilla
++ * Public License Version 1.1 (the "MPL"). If you do not alter this
++ * notice, a recipient may use your version of this file under either
++ * the MPL or the LGPL.
++ *
++ * You should have received a copy of the LGPL along with this library
++ * in the file COPYING-LGPL-2.1; if not, write to the Free Software
++ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
++ * You should have received a copy of the MPL along with this library
++ * in the file COPYING-MPL-1.1
++ *
++ * The contents of this file are subject to the Mozilla Public License
++ * Version 1.1 (the "License"); you may not use this file except in
++ * compliance with the License. You may obtain a copy of the License at
++ * http://www.mozilla.org/MPL/
++ *
++ * This software is distributed on an "AS IS" basis, WITHOUT WARRANTY
++ * OF ANY KIND, either express or implied. See the LGPL or the MPL for
++ * the specific language governing rights and limitations.
++ *
++ * The Original Code is the cairo graphics library.
++ *
++ * The Initial Developer of the Original Code is Chris Wilson.
++ *
++ */
++
++#include "cairoint.h"
++
++#include "cairo-xlib-private.h"
++
++#include <fontconfig/fontconfig.h>
++
++#include <X11/Xlibint.h>	/* For XESetCloseDisplay */
++#include <X11/extensions/Xrender.h>
++
++typedef int (*cairo_xlib_error_func_t) (Display     *display,
++					XErrorEvent *event);
++
++struct _cairo_xlib_job {
++    cairo_xlib_job_t *next;
++    enum {
++	RESOURCE,
++	WORK
++    } type;
++    union {
++	struct {
++	    cairo_xlib_notify_resource_func notify;
++	    XID xid;
++	} resource;
++	struct {
++	    cairo_xlib_notify_func notify;
++	    void *data;
++	    void (*destroy) (void *);
++	} work;
++    } func;
++};
++
++static cairo_xlib_display_t *_cairo_xlib_display_list = NULL;
++
++static void
++_cairo_xlib_call_close_display_hooks (cairo_xlib_display_t *info)
++{
++    cairo_xlib_screen_info_t	    *screen;
++    cairo_xlib_hook_t		    *hooks;
++
++    /* call all registered shutdown routines */
++    CAIRO_MUTEX_LOCK (info->mutex);
++
++    for (screen = info->screens; screen != NULL; screen = screen->next)
++	_cairo_xlib_screen_info_close_display (screen);
++
++    hooks = info->close_display_hooks;
++    while (hooks != NULL) {
++	info->close_display_hooks = NULL;
++	CAIRO_MUTEX_UNLOCK (info->mutex);
++
++	do {
++	    cairo_xlib_hook_t *hook = hooks;
++	    hooks = hook->next;
++
++	    hook->func (info->display, hook->data);
++
++	    free (hook);
++	} while (hooks != NULL);
++
++	CAIRO_MUTEX_LOCK (info->mutex);
++	hooks = info->close_display_hooks;
++    }
++    info->closed = TRUE;
++
++    CAIRO_MUTEX_UNLOCK (info->mutex);
++}
++
++cairo_xlib_display_t *
++_cairo_xlib_display_reference (cairo_xlib_display_t *info)
++{
++    if (info == NULL)
++	return NULL;
++
++    /* use our mutex until we get a real atomic inc */
++    CAIRO_MUTEX_LOCK (info->mutex);
++
++    assert (info->ref_count > 0);
++    info->ref_count++;
++
++    CAIRO_MUTEX_UNLOCK (info->mutex);
++
++    return info;
++}
++
++void
++_cairo_xlib_display_destroy (cairo_xlib_display_t *info)
++{
++    if (info == NULL)
++	return;
++
++    CAIRO_MUTEX_LOCK (info->mutex);
++    assert (info->ref_count > 0);
++    if (--info->ref_count == 0) {
++	assert (info->closed == TRUE);
++	assert (info->screens == NULL);
++
++	/* destroy all outstanding notifies */
++	while (info->workqueue != NULL) {
++	    cairo_xlib_job_t *job = info->workqueue;
++	    info->workqueue = job->next;
++
++	    if (job->type == WORK && job->func.work.destroy != NULL)
++		job->func.work.destroy (job->func.work.data);
++
++	    _cairo_freelist_free (&info->wq_freelist, job);
++	}
++	_cairo_freelist_fini (&info->wq_freelist);
++
++	CAIRO_MUTEX_UNLOCK (info->mutex);
++
++	free (info);
++    } else
++	CAIRO_MUTEX_UNLOCK (info->mutex);
++}
++
++static int
++_noop_error_handler (Display     *display,
++		     XErrorEvent *event)
++{
++    return False;		/* return value is ignored */
++}
++static int
++_cairo_xlib_close_display (Display *dpy, XExtCodes *codes)
++{
++    cairo_xlib_display_t *info, **prev, *next;
++
++    /*
++     * Unhook from the global list
++     */
++    CAIRO_MUTEX_LOCK (_cairo_xlib_display_mutex);
++    prev = &_cairo_xlib_display_list;
++    for (info = _cairo_xlib_display_list; info; info = next) {
++	next = info->next;
++	if (info->display == dpy) {
++	    cairo_xlib_error_func_t old_handler;
++
++	    /* drop the list mutex whilst triggering the hooks */
++	    CAIRO_MUTEX_UNLOCK (_cairo_xlib_display_mutex);
++
++	    /* protect the notifies from triggering XErrors */
++	    old_handler = XSetErrorHandler (_noop_error_handler);
++
++	    _cairo_xlib_display_notify (info);
++	    _cairo_xlib_call_close_display_hooks (info);
++
++	    /* catch any that arrived before marking the display as closed */
++	    _cairo_xlib_display_notify (info);
++
++	    XSync (dpy, False);
++	    XSetErrorHandler (old_handler);
++
++	    CAIRO_MUTEX_LOCK (_cairo_xlib_display_mutex);
++	    _cairo_xlib_display_destroy (info);
++	    *prev = next;
++	    break;
++	} else
++	    prev = &info->next;
++    }
++    CAIRO_MUTEX_UNLOCK (_cairo_xlib_display_mutex);
++
++    /* Return value in accordance with requirements of
++     * XESetCloseDisplay */
++    return 0;
++}
++
++cairo_xlib_display_t *
++_cairo_xlib_display_get (Display *dpy)
++{
++    cairo_xlib_display_t *info;
++    cairo_xlib_display_t **prev;
++    XExtCodes *codes;
++
++    /* There is an apparent deadlock between this mutex and the
++     * mutex for the display, but it's actually safe. For the
++     * app to call XCloseDisplay() while any other thread is
++     * inside this function would be an error in the logic
++     * app, and the CloseDisplay hook is the only other place we
++     * acquire this mutex.
++     */
++    CAIRO_MUTEX_LOCK (_cairo_xlib_display_mutex);
++
++    for (prev = &_cairo_xlib_display_list; (info = *prev); prev = &(*prev)->next)
++    {
++	if (info->display == dpy) {
++	    /*
++	     * MRU the list
++	     */
++	    if (prev != &_cairo_xlib_display_list) {
++		*prev = info->next;
++		info->next = _cairo_xlib_display_list;
++		_cairo_xlib_display_list = info;
++	    }
++	    break;
++	}
++    }
++
++    if (info != NULL) {
++	info = _cairo_xlib_display_reference (info);
++	goto UNLOCK;
++    }
++
++    info = malloc (sizeof (cairo_xlib_display_t));
++    if (info == NULL)
++	goto UNLOCK;
++
++    codes = XAddExtension (dpy);
++    if (codes == NULL) {
++	free (info);
++	info = NULL;
++	goto UNLOCK;
++    }
++
++    XESetCloseDisplay (dpy, codes->extension, _cairo_xlib_close_display);
++
++    _cairo_freelist_init (&info->wq_freelist, sizeof (cairo_xlib_job_t));
++    info->ref_count = 2; /* add one for the CloseDisplay */
++    CAIRO_MUTEX_INIT (&info->mutex);
++    info->display = dpy;
++    info->screens = NULL;
++    info->workqueue = NULL;
++    info->close_display_hooks = NULL;
++    info->closed = FALSE;
++
++    info->next = _cairo_xlib_display_list;
++    _cairo_xlib_display_list = info;
++
++UNLOCK:
++    CAIRO_MUTEX_UNLOCK (_cairo_xlib_display_mutex);
++    return info;
++}
++
++cairo_bool_t
++_cairo_xlib_add_close_display_hook (Display *dpy, void (*func) (Display *, void *), void *data, const void *key)
++{
++    cairo_xlib_display_t *info;
++    cairo_xlib_hook_t *hook;
++    cairo_bool_t ret = FALSE;
++
++    info = _cairo_xlib_display_get (dpy);
++    if (info == NULL)
++	return FALSE;
++
++    hook = malloc (sizeof (cairo_xlib_hook_t));
++    if (hook != NULL) {
++	hook->func = func;
++	hook->data = data;
++	hook->key = key;
++
++	CAIRO_MUTEX_LOCK (info->mutex);
++	if (info->closed == FALSE) {
++	    hook->next = info->close_display_hooks;
++	    info->close_display_hooks = hook;
++	    ret = TRUE;
++	}
++	CAIRO_MUTEX_UNLOCK (info->mutex);
++    }
++
++    _cairo_xlib_display_destroy (info);
++
++    return ret;
++}
++
++void
++_cairo_xlib_remove_close_display_hooks (Display *dpy, const void *key)
++{
++    cairo_xlib_display_t *info;
++    cairo_xlib_hook_t *hook, *next, **prev;
++
++    info = _cairo_xlib_display_get (dpy);
++    if (info == NULL)
++	return;
++
++    CAIRO_MUTEX_LOCK (info->mutex);
++    prev = &info->close_display_hooks;
++    for (hook = info->close_display_hooks; hook != NULL; hook = next) {
++	next = hook->next;
++	if (hook->key == key) {
++	    *prev = hook->next;
++	    free (hook);
++	} else
++	    prev = &hook->next;
++    }
++    *prev = NULL;
++    CAIRO_MUTEX_UNLOCK (info->mutex);
++
++    _cairo_xlib_display_destroy (info);
++}
++
++void
++_cairo_xlib_display_cancel_resource (cairo_xlib_display_t *display,
++				     XID xid)
++{
++    cairo_xlib_job_t *job, *next, **prev;
++
++    CAIRO_MUTEX_LOCK (display->mutex);
++    prev = &display->workqueue;
++    for (job = display->workqueue; job != NULL; job = next) {
++	next = job->next;
++	if (job->type == RESOURCE && job->func.resource.xid == xid) {
++	    _cairo_freelist_free (&display->wq_freelist, job);
++
++	    *prev = next;
++	    break;
++	}
++	prev = &job->next;
++    }
++    CAIRO_MUTEX_UNLOCK (display->mutex);
++}
++
++static cairo_xlib_job_t *
++_cairo_xlib_display_find_resource (cairo_xlib_display_t *display, XID xid)
++{
++    cairo_xlib_job_t *job;
++
++    CAIRO_MUTEX_LOCK (display->mutex);
++    for (job = display->workqueue; job != NULL; job = job->next) {
++	if (job->type == RESOURCE && job->func.resource.xid == xid)
++	    break;
++    }
++    CAIRO_MUTEX_UNLOCK (display->mutex);
++
++    return job;
++}
++
++cairo_status_t
++_cairo_xlib_display_queue_resource (cairo_xlib_display_t *display,
++	                            cairo_xlib_notify_resource_func notify,
++				    XID xid)
++{
++    cairo_xlib_job_t *job;
++    cairo_status_t status = CAIRO_STATUS_SUCCESS;
++
++    assert (_cairo_xlib_display_find_resource (display, xid) == NULL);
++
++    job = _cairo_freelist_alloc (&display->wq_freelist);
++    if (job == NULL)
++	return CAIRO_STATUS_NO_MEMORY;
++
++    job->type = RESOURCE;
++    job->func.resource.xid = xid;
++    job->func.resource.notify = notify;
++
++    CAIRO_MUTEX_LOCK (display->mutex);
++    if (display->closed == FALSE) {
++	job->next = display->workqueue;
++	display->workqueue = job;
++    } else {
++	_cairo_freelist_free (&display->wq_freelist, job);
++	job = NULL;
++	status = CAIRO_STATUS_NO_MEMORY;
++    }
++    CAIRO_MUTEX_UNLOCK (display->mutex);
++
++    return status;
++}
++
++void
++_cairo_xlib_display_cancel_work (cairo_xlib_display_t *display,
++				 void *data)
++{
++    cairo_xlib_job_t *job, *next, **prev;
++
++    CAIRO_MUTEX_LOCK (display->mutex);
++    prev = &display->workqueue;
++    for (job = display->workqueue; job != NULL; job = next) {
++	next = job->next;
++	if (job->type == WORK && job->func.work.data == data) {
++	    if (job->func.work.destroy != NULL)
++		job->func.work.destroy (job->func.work.data);
++	    _cairo_freelist_free (&display->wq_freelist, job);
++
++	    *prev = next;
++	    break;
++	}
++	prev = &job->next;
++    }
++    CAIRO_MUTEX_UNLOCK (display->mutex);
++}
++
++cairo_status_t
++_cairo_xlib_display_queue_work (cairo_xlib_display_t *display,
++	                        cairo_xlib_notify_func notify,
++				void *data,
++				void (*destroy) (void *))
++{
++    cairo_xlib_job_t *job;
++    cairo_status_t status = CAIRO_STATUS_SUCCESS;
++
++    job = _cairo_freelist_alloc (&display->wq_freelist);
++    if (job == NULL)
++	return CAIRO_STATUS_NO_MEMORY;
++
++    job->type = WORK;
++    job->func.work.data    = data;
++    job->func.work.notify  = notify;
++    job->func.work.destroy = destroy;
++
++    CAIRO_MUTEX_LOCK (display->mutex);
++    if (display->closed == FALSE) {
++	job->next = display->workqueue;
++	display->workqueue = job;
++    } else {
++	_cairo_freelist_free (&display->wq_freelist, job);
++	job = NULL;
++	status = CAIRO_STATUS_NO_MEMORY;
++    }
++    CAIRO_MUTEX_UNLOCK (display->mutex);
++
++    return status;
++}
++
++void
++_cairo_xlib_display_notify (cairo_xlib_display_t *display)
++{
++    cairo_xlib_job_t *jobs, *job;
++
++    CAIRO_MUTEX_LOCK (display->mutex);
++    jobs = display->workqueue;
++    while (jobs != NULL) {
++	display->workqueue = NULL;
++	CAIRO_MUTEX_UNLOCK (display->mutex);
++
++	/* reverse the list to obtain FIFO order */
++	job = NULL;
++	do {
++	    cairo_xlib_job_t *next = jobs->next;
++	    jobs->next = job;
++	    job = jobs;
++	    jobs = next;
++	} while (jobs != NULL);
++	jobs = job;
++
++	do {
++	    job = jobs;
++	    jobs = job->next;
++
++	    switch (job->type){
++	    case WORK:
++		job->func.work.notify (display->display, job->func.work.data);
++		if (job->func.work.destroy != NULL)
++		    job->func.work.destroy (job->func.work.data);
++		break;
++
++	    case RESOURCE:
++		job->func.resource.notify (display->display,
++			                   job->func.resource.xid);
++		break;
++	    }
++
++	    _cairo_freelist_free (&display->wq_freelist, job);
++	} while (jobs != NULL);
++
++	CAIRO_MUTEX_LOCK (display->mutex);
++	jobs = display->workqueue;
++    }
++    CAIRO_MUTEX_UNLOCK (display->mutex);
++}
+--- a/src/cairo-xlib-private.h
++++ b/src/cairo-xlib-private.h
+@@ -35,30 +35,82 @@
+ 
+ #include "cairoint.h"
+ #include "cairo-xlib.h"
++#include "cairo-freelist-private.h"
+ 
++typedef struct _cairo_xlib_display cairo_xlib_display_t;
+ typedef struct _cairo_xlib_screen_info cairo_xlib_screen_info_t;
+ typedef struct _cairo_xlib_hook cairo_xlib_hook_t;
++typedef struct _cairo_xlib_job cairo_xlib_job_t;
++typedef void (*cairo_xlib_notify_func) (Display *, void *);
++typedef void (*cairo_xlib_notify_resource_func) (Display *, XID);
+ 
+ struct _cairo_xlib_hook {
+     cairo_xlib_hook_t *next;
+     void (*func) (Display *display, void *data);
+     void *data;
+-    void *key;
++    const void *key;
++};
++
++struct _cairo_xlib_display {
++    cairo_xlib_display_t *next;
++    unsigned int ref_count;
++    cairo_mutex_t mutex;
++
++    Display *display;
++    cairo_xlib_screen_info_t *screens;
++
++    cairo_xlib_job_t *workqueue;
++    cairo_freelist_t wq_freelist;
++
++    cairo_xlib_hook_t *close_display_hooks;
++    unsigned int closed :1;
+ };
+ 
+ struct _cairo_xlib_screen_info {
+     cairo_xlib_screen_info_t *next;
+     unsigned int ref_count;
+ 
+-    Display *display;
++    cairo_xlib_display_t *display;
+     Screen *screen;
+     cairo_bool_t has_render;
+ 
+     cairo_font_options_t font_options;
+ 
+-    cairo_xlib_hook_t *close_display_hooks;
++    GC gc[6];
+ };
+ 
++cairo_private cairo_xlib_display_t *
++_cairo_xlib_display_get (Display *display);
++
++cairo_private cairo_xlib_display_t *
++_cairo_xlib_display_reference (cairo_xlib_display_t *info);
++cairo_private void
++_cairo_xlib_display_destroy (cairo_xlib_display_t *info);
++
++cairo_private cairo_bool_t
++_cairo_xlib_add_close_display_hook (Display *display, void (*func) (Display *, void *), void *data, const void *key);
++cairo_private void
++_cairo_xlib_remove_close_display_hooks (Display *display, const void *key);
++
++cairo_private cairo_status_t
++_cairo_xlib_display_queue_work (cairo_xlib_display_t *display,
++	                        cairo_xlib_notify_func notify,
++				void *data,
++				void (*destroy)(void *));
++cairo_private cairo_status_t
++_cairo_xlib_display_queue_resource (cairo_xlib_display_t *display,
++	                           cairo_xlib_notify_resource_func notify,
++				   XID resource);
++cairo_private void
++_cairo_xlib_display_cancel_work (cairo_xlib_display_t *display,
++				 void *data);
++cairo_private void
++_cairo_xlib_display_cancel_resource (cairo_xlib_display_t *display,
++				     XID resource);
++
++cairo_private void
++_cairo_xlib_display_notify (cairo_xlib_display_t *display);
++
+ cairo_private cairo_xlib_screen_info_t *
+ _cairo_xlib_screen_info_get (Display *display, Screen *screen);
+ 
+@@ -67,10 +119,14 @@ _cairo_xlib_screen_info_reference (cairo
+ cairo_private void
+ _cairo_xlib_screen_info_destroy (cairo_xlib_screen_info_t *info);
+ 
+-cairo_private cairo_bool_t
+-_cairo_xlib_add_close_display_hook (Display *display, void (*func) (Display *, void *), void *data, void *key);
+ cairo_private void
+-_cairo_xlib_remove_close_display_hook (Display *display, void *key);
++_cairo_xlib_screen_info_close_display (cairo_xlib_screen_info_t *info);
++
++
++cairo_private GC
++_cairo_xlib_screen_get_gc (cairo_xlib_screen_info_t *info, int depth);
++cairo_private void
++_cairo_xlib_screen_put_gc (cairo_xlib_screen_info_t *info, int depth, GC gc);
+ 
+ #if CAIRO_HAS_XLIB_XRENDER_SURFACE
+ 
+--- a/src/cairo-xlib-screen.c
++++ b/src/cairo-xlib-screen.c
+@@ -58,7 +58,6 @@
+ 
+ #include <fontconfig/fontconfig.h>
+ 
+-#include <X11/Xlibint.h>	/* For XESetCloseDisplay */
+ #include <X11/extensions/Xrender.h>
+ 
+ static int
+@@ -133,7 +132,7 @@ get_integer_default (Display    *dpy,
+ #endif
+ 
+ static void
+-_cairo_xlib_init_screen_font_options (cairo_xlib_screen_info_t *info)
++_cairo_xlib_init_screen_font_options (Display *dpy, cairo_xlib_screen_info_t *info)
+ {
+     cairo_bool_t xft_hinting;
+     cairo_bool_t xft_antialias;
+@@ -143,23 +142,23 @@ _cairo_xlib_init_screen_font_options (ca
+     cairo_subpixel_order_t subpixel_order;
+     cairo_hint_style_t hint_style;
+ 
+-    if (!get_boolean_default (info->display, "antialias", &xft_antialias))
++    if (!get_boolean_default (dpy, "antialias", &xft_antialias))
+ 	xft_antialias = TRUE;
+ 
+-    if (!get_boolean_default (info->display, "hinting