[PATCH 0/3] Free EGL images in glamor as needed

Keith Packard keithp at keithp.com
Mon Dec 8 13:38:42 PST 2014


Here are the three patches I mentioned which take care of destroying
the EGL image structure when freeing pixmaps and when assigning a new
image to a pixmap which already has one.

The second patch is a 'clean up' which removes a redundant reference
to the screen EGL image -- it's already referenced from the screen
pixmap, so we don't need another pointer. I could have removed this
patch, but I'd have to fix other bits of the sequence to make sure it
still tracked the screen pixmap image.

Keith Packard (3):
  glamor: Always destroy EGL image associated with destroyed pixmap
  glamor: Remove redundant reference to screen pixmap EGL image
  glamor: Free existing EGL image when assigning new one

 glamor/glamor.c               | 26 ++++----------
 glamor/glamor.h               |  3 +-
 glamor/glamor_egl.c           | 81 ++++++++++++++++---------------------------
 glamor/glamor_egl_stubs.c     |  2 +-
 glamor/glamor_priv.h          |  1 -
 hw/xwayland/xwayland-glamor.c |  3 +-
 6 files changed, 40 insertions(+), 76 deletions(-)

-- 
2.1.3



More information about the xorg-devel mailing list