intel: Don't flush the old context in intelMakeCurrent
authorNeil Roberts <neil@linux.intel.com>
Wed, 4 Feb 2015 15:20:12 +0000 (10:20 -0500)
committerAdam Jackson <ajax@redhat.com>
Mon, 6 Nov 2017 21:08:58 +0000 (16:08 -0500)
commit8c0729fd99c8548ceaf40e273b809ce0a69d8a44
treeb3fad704126a6d599d1c2c2cfa3d5e49fbe8740b
parent9ef7158a0985d789958eb43cf08476d50af00f2a
intel: Don't flush the old context in intelMakeCurrent

It shouldn't be necessary to flush the context within the driver
implementation because the old context is explicitly flushed in
_mesa_make_current which is called a little further on. It is useful to
only have a single place that flushes when switching contexts to make it
easier to later implement the GL_KHR_context_flush_control extension.

The flush in intelMakeCurrent was added in commit 5505865 to implement
the GLX semantics that the context should be flushed when it is
released.  When the commit was made there was no flush in
_mesa_make_current because it was only added later in 93102b4c. I think
that later commit effectively makes the first commit redundant.

Reviewed-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Neil Roberts <neil@linux.intel.com>
src/mesa/drivers/dri/i915/intel_context.c
src/mesa/drivers/dri/i965/brw_context.c