[Mesa-dev] [PATCH] Revert "st/vdpau: use linear layout for output surfaces"

[Alex Deucher]

On Fri, Nov 4, 2016 at 12:43 PM, Marek Olšák <maraeo at gmail.com> wrote:
> On Fri, Nov 4, 2016 at 5:23 PM, Alex Deucher <alexdeucher at gmail.com> wrote:
>> On Fri, Nov 4, 2016 at 10:57 AM, Marek Olšák <maraeo at gmail.com> wrote:
>>> On Nov 4, 2016 3:29 PM, "Emil Velikov" <emil.l.velikov at gmail.com> wrote:
>>>>
>>>> On 31 October 2016 at 15:12, Marek Olšák <maraeo at gmail.com> wrote:
>>>> > First, DRI_PRIME should work OK on radeon & amdgpu. I've been testing
>>>> > it with 2 GPUs in 2 PCIe slots. (only OpenGL though)
>>>> >
>>>> > I think the thread got derailed at the beginning and nobody asked the
>>>> > important question:
>>>> >
>>>> > Does radeonsi need PIPE_BIND_VIDEO_DECODE_OUTPUT that it would
>>>> > translate into linear?
>>>> >
>>>> Agreed things got derailed a bit.
>>>>
>>>> On the patch itself, can we get an ack/r-b, please?
>>>> It'll be nice to have nouveau working again, until a
>>>> better/alternative solution is found.
>>>
>>> I'm okay with the patch. 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.
>>
>> Linear definitely performs better.  Some early UVD blocks didn't even
>> support tiling IIRC.
>
> I've looked at the code and it looks like VdpOutputSurface is only
> used for final 3D passes and not as a decoding output, while
> VdpVideoSurface is used as a decoding output. Is that correct?

I think so, but I'm not that familiar with the video code.

Alex