On Tuesday 22 September 2009 14:10:34 Jakob Bornecrantz wrote:
> diff --git a/src/gallium/state_trackers/xorg/xorg_composite.c
>  b/src/gallium/state_trackers/xorg/xorg_composite.c index 66ca4cb..ed649a9
>  100644
> --- a/src/gallium/state_trackers/xorg/xorg_composite.c
> +++ b/src/gallium/state_trackers/xorg/xorg_composite.c
> @@ -359,6 +359,9 @@ bind_framebuffer_state(struct exa_context *exa, struct
>  exa_pixmap_priv *pDst) state.zsbuf = 0;
>  
>     cso_set_framebuffer(exa->cso, &state);
> +
> +   /* we do fire and forget for the framebuffer, this is the forget part
>  */ +   pipe_surface_reference(&surface, NULL);
>  }
> 

This doesn't follow what we do in gallium state trackers. It should be done in 
xorg_exa_common_done called from ExaDone. Or in other words when we're really 
done with the surface.

z

------------------------------------------------------------------------------
Come build with us! The BlackBerry® Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9-12, 2009. Register now!
http://p.sf.net/sfu/devconf
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to