[Intel-gfx] [PATCH 2/3] glamor: turn on glamor.
zhigang.gong at linux.intel.com
Fri Nov 11 02:52:11 PST 2011
> -----Original Message-----
> From: Chris Wilson [mailto:chris at chris-wilson.co.uk]
> Sent: Friday, November 11, 2011 5:12 PM
> To: Zhigang Gong; intel-gfx at lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH 2/3] glamor: turn on glamor.
> On Fri, 11 Nov 2011 16:31:20 +0800, Zhigang Gong
> <zhigang.gong at linux.intel.com> wrote:
> > @@ -965,6 +969,9 @@ void
> intel_uxa_block_handler(intel_screen_private *intel)
> > * framebuffer until significantly later.
> > */
> > intel_flush_rendering(intel);
> > +#ifdef GLAMOR
> > + intel_glamor_block_handler(intel);
> > +#endif
> > }
> I suspect this is the wrong way around as we are not flushing the render
> cache of glamor's rendering to the scanout until the next block handler.
I don't understand here. Would you please explain more detail? Thanks.
> In general, try to keep the #ifdef out of the body of the code. In this
> and others, make intel_glamor_block_handler() be a no-op if GLAMOR is
> not enabled.
Agreed, will fix it next version.
> Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx