[PATCH] Initialize dev privates before using any
Vignatti Tiago (Nokia-MS/Helsinki)
tiago.vignatti at nokia.com
Mon Jul 5 05:30:44 PDT 2010
On Thu, Jul 01, 2010 at 03:55:01PM +0200, ext Keith Packard wrote:
> On Thu, 1 Jul 2010 16:42:38 +0300, Tiago Vignatti <tiago.vignatti at nokia.com> wrote:
>
> > 1 PIXMAPs still allocated at reset
> > PIXMAP: 1 objects of 40 bytes = 40 total bytes 0 private allocs
>
> > TOTAL: 1 objects, 40 bytes, 0 allocs
> >
> > I don't think this is expected, right? If you need some help to track down
> > some possible leaks here just tell me an easy way to do so.
>
> Yes, you're leaking a pixmap, most likely a screen pixmap as your driver
> isn't freeing it during server reset. I know that at least the intel
> driver still leaks this pixmap. I thought I had posted a patch to fix
> that, if not, here it is:
In my desktop right now I just have two drivers using XAA to test. Both are
leaking this pixmap.
I don't have time now to keep digging on it but seems that the fix should be
inside XAA, isn't?
> From 4f3397774a48a214755740f846e1eb8995a744bb Mon Sep 17 00:00:00 2001
> From: Keith Packard <keithp at keithp.com>
> Date: Fri, 30 Apr 2010 13:33:14 -0700
> Subject: [PATCH] Destroy screen pixmap on screen close.
>
> This avoids a memory leak on server reset.
>
> Signed-off-by: Keith Packard <keithp at keithp.com>
It's not applied yet on intel driver. So:
Acked-by: Tiago Vignatti <tiago.vignatti at nokia.com>
Cheers,
Tiago
More information about the xorg-devel
mailing list