Re: [Intel-gfx] [PATCH] uxa_glamor_dri: Use exchange buffer in glamor fixup.

2012-07-27 Thread Chris Wilson
On Fri, 27 Jul 2012 18:12:26 +0800, Zhigang Gong wrote: > On Fri, Jul 27, 2012 at 10:10:32AM +0100, Chris Wilson wrote: > > On Fri, 27 Jul 2012 17:06:21 +0800, zhigang.g...@linux.intel.com wrote: > > > - if (attachment == DRI2BufferFrontLeft) > > > + if (attachment == DRI2BufferFr

Re: [Intel-gfx] [PATCH] uxa_glamor_dri: Use exchange buffer in glamor fixup.

2012-07-27 Thread Zhigang Gong
On Fri, Jul 27, 2012 at 10:10:32AM +0100, Chris Wilson wrote: > On Fri, 27 Jul 2012 17:06:21 +0800, zhigang.g...@linux.intel.com wrote: > > - if (attachment == DRI2BufferFrontLeft) > > + if (attachment == DRI2BufferFrontLeft && is_glamor_pixmap) > is_glamor_pixmap is only true f

Re: [Intel-gfx] [PATCH] uxa_glamor_dri: Use exchange buffer in glamor fixup.

2012-07-27 Thread Chris Wilson
On Fri, 27 Jul 2012 17:06:21 +0800, zhigang.g...@linux.intel.com wrote: > - if (attachment == DRI2BufferFrontLeft) > + if (attachment == DRI2BufferFrontLeft && is_glamor_pixmap) is_glamor_pixmap is only true for FrontLeft, mind resending with this simplification? Thanks, -C

[Intel-gfx] [PATCH] uxa_glamor_dri: Use exchange buffer in glamor fixup.

2012-07-27 Thread zhigang . gong
From: Zhigang Gong The previous implementation is to create a new textured pixmap based on the newly created pixmap's buffer object. This is not efficient, as we already created it when we call CreatePixmap. We can just exchange the underlying texture/image buffers by calling intel_glamor_exchan