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

[Marek Olšák]

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?

Marek


On Mon, Oct 31, 2016 at 3:10 PM, Christian König
<deathsimple at vodafone.de> wrote:
> Good, thanks for testing.
>
> Dave, are you ok with pushing the patch now or do we need more testing from
> my side?
>
> Thanks,
> Christian.
>
>
> Am 31.10.2016 um 14:55 schrieb Nayan Deshmukh:
>
> Hi Guys,
>
> This not in master yet, I have tested it on my system(I+A) and it works
> fine.
>
> Regards,
> Nayan
>
> On Sat, Oct 1, 2016 at 1:57 PM, Christian König <deathsimple at vodafone.de>
> wrote:
>>
>> Am 01.10.2016 um 00:06 schrieb Marek Olšák:
>>>
>>> On Fri, Sep 30, 2016 at 11:23 PM, Dave Airlie <airlied at gmail.com> wrote:
>>>>
>>>> On 1 October 2016 at 06:11, Marek Olšák <maraeo at gmail.com> wrote:
>>>>>
>>>>> On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin <imirkin at alum.mit.edu>
>>>>> wrote:
>>>>>>
>>>>>> So is this getting pushed, or should I push my "don't build vdpau for
>>>>>> nouveau" patch?
>>>>>
>>>>> Please wait.
>>>>>
>>>>> Christian, your commit says it fixes a radeonsi issue, but Ilia says
>>>>> it breaks nouveau.
>>>>>
>>>>> Can we do something to accommodate both drivers? Should we add a CAP
>>>>> for whether to use a linear layout for output surfaces?
>>>>
>>>> I think we need a flag that isn't SHARED/LINEAR that is more
>>>> SHARED_OTHER_GPU.
>>>
>>> That would be useless. Most OpenGL textures can be shared with other
>>> GPUs and processes via the EGL dmabuf extensions. Relying on
>>> PIPE_BIND_SHARED is a bug by itself and no driver should do that.
>>
>>
>> Yeah, completely agree.
>>
>> To answer the original question the comment in the patch was about an
>> issue with sharing tiling information, but I think that is long fixed by
>> now.
>>
>> Anyway somebody needs to test this, but I don't have time right now.
>>
>> Christian.
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>