svga: re-validate sampler view at draw time if needed
authorCharmaine Lee <charmainel@vmware.com>
Tue, 27 Jun 2017 16:47:01 +0000 (10:47 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 30 Jun 2017 14:24:27 +0000 (08:24 -0600)
commitdbb5d2a790d38d99ae26136014c1c5ebd03c5811
tree8692c510c61ea3bf1013b2890a077e10027ff1f3
parent5355107034908b6d0430f25f0537b06f92fdd320
svga: re-validate sampler view at draw time if needed

This patch validates those sampler views with backing copy
of texture whose original copy has been updated since the
view is last validated.
This is done here at draw time because the texture binding might not
have modified, hence validation is not triggered at state update time,
and yet the texture might have been updated in another context, so
we need to re-validate the sampler view in order to update the backing
copy of the updated texture.

This fixes a rendering flickering issue with Photoshop running in
Linux VM with HWversion 11. The problem is Photoshop renders to texture A
in context X, and then bind texture A to context Y. The first time
when texture A is bound to context Y, cso calls pipe->set_sampler_views().
Validation of sampler views is done, rendering is fine.
But when texture A is rendered to again in context X, and rebound in
context Y, cso skips pipe->set_sampler_views() because texture A is already
bound in context Y. SVGA driver is not given a chance to re-validate
the texture binding, the backing copy of the texture is not updated,
and hence causes black image.

Tested with Photoshop, MTT glretrace, piglit.
Fixes VMware bug 1769103.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
src/gallium/drivers/svga/svga_context.c
src/gallium/drivers/svga/svga_context.h
src/gallium/drivers/svga/svga_draw.c
src/gallium/drivers/svga/svga_state_tss.c