Revert "st/vdpau: use linear layout for output surfaces"
authorDave Airlie <airlied@redhat.com>
Thu, 15 Sep 2016 03:58:33 +0000 (13:58 +1000)
committerEmil Velikov <emil.l.velikov@gmail.com>
Fri, 4 Nov 2016 15:04:21 +0000 (15:04 +0000)
commitd0d5f7600c2e8ab8d0c153787185f7a534753edd
treeca235068b353e94ea91df97bf7078af30bb234eb
parent00baaa4752ab7e721218a2840cf0952d8c7c6eca
Revert "st/vdpau: use linear layout for output surfaces"

This reverts commit d180de35320eafa3df3d76f0e82b332656530126.

This is a radeon specific hack that causes problems on nouveau
when combined with the SHARED flag later. If radeonsi needs a fix
for this, please fix it in the driver.

[chk]
Using linear surfaces for this makes sense because tilling isn't
beneficial and the surfaces can potentially be shared with other GPUs
using the VDPAU OpenGL interop.

[airlied]
I think we need a flag that isn't SHARED/LINEAR that is more
SHARED_OTHER_GPU.

[mareko]
Does radeonsi need PIPE_BIND_VIDEO_DECODE_OUTPUT that it would translate
into linear ?

[mareko]
My only concern is decoding performance. If the decoder works in 64x1
blocks, tiling will hurt. That's the theory. I don't know how the
decoder works.

Cc: 12.0 13.0 <mesa-stable@lists.freedesktop.org>
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Tested-by: Ilia Mirkin <imirkin@alum.mit.edu>
Tested-by: Nayan Deshmukh <nayan26deshmukh@gmail.com> (I+A)
src/gallium/state_trackers/vdpau/output.c