[Mesa-dev] [PATCH 2/3] dri3: Flush and wait for X rendering upon glXWaitX()

[Adam Jackson]

On Sat, 2016-10-22 at 11:00 +0100, Chris Wilson wrote:

> @@ -113,6 +113,11 @@ loader_dri3_drawable_fini(struct loader_dri3_drawable *draw)
>           dri3_free_render_buffer(draw, draw->buffers[i]);
>     }
>  
> +   if (draw->sync_fence) {
> +      xcb_sync_destroy_fence(draw->conn, draw->sync_fence);
> +      xshmfence_unmap_shm(draw->shm_fence);
> +   }

You're leaking the fence fd. Probably should just close it immediately
after mapping it so you don't occupy an fd per drawable.

- ajax