[PATCH 4/4] glamor: Make sure glamor_egl_close_screen wraps glamor_close_screen

[Keith Packard]

Michel Dänzer <michel at daenzer.net> writes:

> From: Michel Dänzer <michel.daenzer at amd.com>
>
> The other way around fails to destroy the screen pixmap EGL image:
>
> ==1782== 80 (32 direct, 48 indirect) bytes in 1 blocks are definitely lost in loss record 981 of 2,171
> ==1782==    at 0x4C28C20: malloc (vg_replace_malloc.c:296)
> ==1782==    by 0xF9D4BD2: dri2_create_image_from_dri (egl_dri2.c:1264)
> ==1782==    by 0xF9D4BD2: dri2_create_image_dma_buf (egl_dri2.c:1764)
> ==1782==    by 0xF9D4BD2: dri2_create_image_khr (egl_dri2.c:1798)
> ==1782==    by 0xF9C7937: eglCreateImageKHR (eglapi.c:1494)
> ==1782==    by 0x85D5655: _glamor_egl_create_image (glamor_egl.c:134)
> ==1782==    by 0x85D5655: glamor_egl_create_textured_pixmap (glamor_egl.c:302)
> ==1782==    by 0x85D579B: glamor_egl_create_textured_screen (glamor_egl.c:225)
> ==1782==    by 0xC1BE05D: radeon_glamor_create_screen_resources (radeon_glamor.c:67)
> ==1782==    by 0xC1B6153: RADEONCreateScreenResources_KMS (radeon_kms.c:258)
> ==1782==    by 0x4B2105: xf86CrtcCreateScreenResources (xf86Crtc.c:709)
> ==1782==    by 0x43C823: dix_main (main.c:223)
> ==1782==    by 0x6CFAB44: (below main) (libc-start.c:287)
>
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>

For the series --

Reviewed-by: Keith Packard <keithp at keithp.com>

Merged.
   913416b..0f5fdaf  master -> master

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20141211/73f1f36f/attachment.sig>