Re: [PATCH 2/3] glamor: turn on glamor.

Zhigang Gong
2011-11-11T10:52:14Z

> -----Original Message-----
> From: Chris Wilson [mailto:chris@chris-wilson.co.uk]
> Sent: Friday, November 11, 2011 5:12 PM
> To: Zhigang Gong; intel-gfx@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@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
case,
> and others, make intel_glamor_block_handler() be a no-op if GLAMOR is
> not enabled.
Agreed, will fix it next version.

> -Chris
>
> --
> Chris Wilson, Intel Open Source Technology Centre