[PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Adam Jackson
I broke this, back in: commit a48dadc98a28c969741979b70b7a639f24f4cbbd Author: Adam Jackson a...@redhat.com Date: Mon Mar 21 11:59:29 2011 -0400 glx: Reimplement context tags In that, I changed the glx client state to not explicitly track the list of current contexts for

Re: [PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: I broke this, back in: commit a48dadc98a28c969741979b70b7a639f24f4cbbd Author: Adam Jackson a...@redhat.com Date: Mon Mar 21 11:59:29 2011 -0400 glx: Reimplement context tags In that, I changed the glx client state to not

Re: [PATCH 2/3] glx: Fix memory leak in context garbage collection

2013-09-30 Thread Adam Jackson
On Mon, 2013-09-30 at 11:17 -0700, Eric Anholt wrote: But I'm lost on this one. loseCurrent is just -core-unbindContext, which is mesa's dri_util.c dereferencing the drawables and calling the driver UnbindContext, which is just _mesa_make_current(NULL, NULL, NULL) regardless of the current