On Fri, Oct 28, 2016 at 2:14 PM, Michel Dänzer <mic...@daenzer.net> wrote:
> On 28/10/16 03:58 PM, Nayan Deshmukh wrote: > > On Fri, Oct 28, 2016 at 7:00 AM, Michel Dänzer <mic...@daenzer.net > > <mailto:mic...@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.veli...@gmail.com <mailto:emil.l.veli...@gmail.com>> wrote: > > >> On 17 October 2016 at 10:37, Marek Olšák <mar...@gmail.com > <mailto:mar...@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? > We also use PIPE_BIND_LINEAR and PIPE_BIND_SHARED when allocating a buffer in vl_winsys_dri3.c, so do we also need to change that? > > > > 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 >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
