vdpau: flush the context before exporting the surface v2
authorMarek Olšák <marek.olsak@amd.com>
Mon, 13 Jan 2014 13:13:01 +0000 (14:13 +0100)
committerChristian König <christian.koenig@amd.com>
Thu, 6 Feb 2014 19:58:07 +0000 (20:58 +0100)
commit3f98053fc94a964930c73c43154daddfd7824e7c
treec88ea78d36b175b772c47a301b548e4011e9858b
parente2ef93cf94ee553e5de70a7c26fd0724d967d0b2
vdpau: flush the context before exporting the surface v2

Bugzilla (bug needs XBMC changes as well):
https://bugs.freedesktop.org/show_bug.cgi?id=73191

When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which always
flushes the context in the winsys if the buffer being mapped is busy. Since
I added handling of DISCARD_RANGE, DONTBLOCK has had no effect when combined
with DISCARD_RANGE and I think the context isn't flushed anywhere else,
so no commands are submitted to the GPU until the IB is full, which takes
a lot of frames.

Using DISCARD_RANGE is not the only way to trigger this bug. The other way
is to reallocate the vertex buffer before every upload.

BTW, I'm not sure if this is the right place for flushing, but it does fix
the bug.

v2 (chk): move the flush to the right place.

Signed-off-by: Christian König <christian.koenig@amd.com>
Tested-by: StrangeNoises (rachel@strangenoises.org)
src/gallium/state_trackers/vdpau/output.c