Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

Hi Christian, Sorry for the delay, I confirm the flush is required with that config described before. Also in order to only affects that particular I wanted to do this: +if (buf->export_refcount > 0) drv->pipe->flush(drv->pipe, NULL, 0); But this is not the same kind of vlVaBuffer, so not pos

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

Are you sure the flush after calling the compositor is really necessary? That clearly looks odd, but if it works I'm fine with keeping that for now. Regards, Christian. On 15.12.2015 10:06, Julien Isorce wrote: And the attachment :) On 15 December 2015 at 09:06, Julien Isorce

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

Hi Christian, I tried your v2. I had to apply attached change on top of your patch. (the one in buffer.c to avoid crashing, the one postproc.c otherwise same behavior as the v1 of this patch). Note that I export the RGB-like surface (the one that vpp output), not the NV12 one that come from the d

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

And the attachment :) On 15 December 2015 at 09:06, Julien Isorce wrote: > Hi Christian, > > I tried your v2. > > I had to apply attached change on top of your patch. (the one in buffer.c > to avoid crashing, the one postproc.c otherwise same behavior as the v1 of > this patch). Note that I expo

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

Also note that in this pipeline, HW decoding is done with nouveau driver and rendering is done with intel. dmabuf in between. Yeah, I already thought that somebody is using it like this. I'm not sure if this is actually supposed to work because we don't have proper synchronization between kernel

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

Hi Christian, I have tested this patch but then the displayed video is garbage (mostly white and sometimes just garbage). It also stall the nouveau driver which requires to reboot but I guess this is another issue. I tested with: GST_GL_WINDOW=x11 GST_GL_PLATFORM=egl GST_GL_API=gles2 GST_DEBUG=2 L

[Mesa-dev] [PATCH 6/8] st/va: remove fence handling

From: Christian König It's nonsense to drain the pipeline like this. Signed-off-by: Christian König --- src/gallium/state_trackers/va/buffer.c | 5 - src/gallium/state_trackers/va/image.c | 1 - src/gallium/state_trackers/va/postproc.c | 6 -- src/gallium/state_trackers/v