egl: Remove bogus invalidate code.
authorEric Anholt <eric@anholt.net>
Tue, 25 Sep 2012 21:45:16 +0000 (14:45 -0700)
committerEric Anholt <eric@anholt.net>
Tue, 9 Oct 2012 21:32:04 +0000 (14:32 -0700)
The invalidate event support is a careful dance between driver and loader,
where both have to say they can handle it, and then the loader reports
invalidate events for the driver so the driver can do the optimization.

The EGL code doesn't report __DRIuseInvalidateExtension to the driver, so it
has no responsibility to call the driver's invalidate function, and the driver
is doing the glViewport hack because it assume.  This is not
the only time invalidate would need to be called (we need it *any* time an
invalidate event comes down the pipe, but we don't watch for them), so just
stop calling the driver's function.

Acked-by: Chad Versace <chad.versace@linux.intel.com>
src/egl/drivers/dri2/platform_x11.c

index 4757ccfca4f05f21fc184977e9a8d751e1869932..936e3806521ad8e65b5b39f58a9ca7a51c01259b 100644 (file)
@@ -742,11 +742,6 @@ dri2_swap_buffers_msc(_EGLDriver *drv, _EGLDisplay *disp, _EGLSurface *draw,
       free(reply);
    }
 
-   /* If the server doesn't send invalidate events */
-   if (dri2_dpy->invalidate_available && dri2_dpy->flush &&
-       dri2_dpy->flush->base.version >= 3 && dri2_dpy->flush->invalidate)
-      (*dri2_dpy->flush->invalidate)(dri2_surf->dri_drawable);
-
    return swap_count;
 }