[PATCH 1/3] glamor: Just set the logic op to what we want at the start of all rendering.
keithp at keithp.com
Wed Feb 4 20:43:31 PST 2015
Eric Anholt <eric at anholt.net> writes:
> By dropping the unconditional logic op disable at the end of
> rendering, this fixes GL errors being thrown in GLES2 contexts (which
> don't have logic ops). On desktop, this also means a little less
> overhead per draw call from taking one less trip through the
> glEnable/glDisable switch statement of doom in Mesa.
Reviewed-by: Keith Packard <keithp at keithp.com>
Could this be further optimized by not calling glDisable/glEnable in
glamor_set_alu when unnecessary? If those are expensive, keeping track
of the current value would seem like a good optimization.
> diff --git a/hw/kdrive/ephyr/ephyr_glamor_glx.c b/hw/kdrive/ephyr/ephyr_glamor_glx.c
> index 8fe7516..582e3af 100644
> --- a/hw/kdrive/ephyr/ephyr_glamor_glx.c
> +++ b/hw/kdrive/ephyr/ephyr_glamor_glx.c
> @@ -214,6 +214,8 @@ ephyr_glamor_damage_redisplay(struct ephyr_glamor *glamor,
> glBindFramebuffer(GL_FRAMEBUFFER, 0);
> glViewport(0, 0, glamor->width, glamor->height);
> + if (!ephyr_glamor_gles2)
> + glDisable(GL_COLOR_LOGIC_OP);
I'd suggest an unconditional call to glamor_set_alu(screen, GXcopy)
here; that way any future optimizations to avoid calling
glDisable/glEnable would work without having to remember to fix ephyr too.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 810 bytes
Desc: not available
More information about the xorg-devel