[Mesa-dev] [PATCH 09/10] st/vdpau: implement the new DMA-buf based interop

Michel Dänzer michel at daenzer.net
Fri Oct 28 08:44:02 UTC 2016

On 28/10/16 03:58 PM, Nayan Deshmukh wrote:
> On Fri, Oct 28, 2016 at 7:00 AM, Michel Dänzer <michel at daenzer.net
> <mailto:michel at daenzer.net>> wrote:
>     On 28/10/16 03:08 AM, Marek Olšák wrote:
>     > On Thu, Oct 27, 2016 at 5:23 PM, Emil Velikov <emil.l.velikov at gmail.com <mailto:emil.l.velikov at gmail.com>> wrote:
>     >> On 17 October 2016 at 10:37, Marek Olšák <maraeo at gmail.com <mailto:maraeo at gmail.com>> wrote:
>     >>> Reverting the whole commit is too much. You can just remove the PIPE BIND
>     >>> SHARED usage if you need to.
>     >>>
>     >> I'd imagine that one wants to check if radeon(s) behave OK with the
>     >> flag dropped ? Thus it might be better for someone with radeon HW to
>     >> give it a bash.
>     >> Can we have a volunteer please :-)
>     >
>     > r600-radeonsi ignore PIPE_BIND_SHARED on textures.
>     That doesn't mean the state tracker can just drop PIPE_BIND_SHARED.
>     https://patchwork.freedesktop.org/patch/110569/
>     <https://patchwork.freedesktop.org/patch/110569/> seems the best
>     candidate
>     for resolving the nouveau issue, but I suspect as is it might break DRI3
>     PRIME. Nayan, can you check this and fix it up as necessary? Basically,
>     PIPE_BIND_LINEAR is only really necessary for buffers which are sent to
>     the X server for presentation on a different GPU.
> It works fine with DRI3 Prime with my I+A system, but I think it won't
> work with a I+N system as it uses PIPE_BIND_SHARE and PIPE_BIND_LINEAR
> together for one buffer.

The patch above removes PIPE_BIND_LINEAR. Which case are you thinking of?

> Since this buffer is being sent to X we need to have a PIPE_BIND_LINEAR,
> so should I send a patch removing SHARED flag, it is working fine on
> my system without the shared flag. 

PIPE_BIND_LINEAR is only required for buffers which are shared between
different GPUs. Note that in that case it's required with nouveau as well.

Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer

More information about the mesa-dev mailing list