On Fri, 27 Jul 2012 17:06:21 +0800, zhigang.gong at 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, -Chris -- Chris Wilson, Intel Open Source Technology Centre