On 20/07/16 19:22, Javier Martinez Canillas wrote: > Currently the dma-buf is unmapped when the buffer is dequeued by userspace > but it's not used anymore after the driver finished processing the buffer. > > So instead of doing the dma-buf unmapping in __vb2_dqbuf(), it can be made > in vb2_buffer_done() after the driver notified that buf processing is done. > > Decoupling the buffer dequeue from the dma-buf unmapping has also the side > effect of making possible to add dma-buf fence support in the future since > the buffer could be dequeued even before the driver has finished using it. > > Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> > > --- > Hello, > > I've tested this patch doing DMA buffer sharing between a > vivid input and output device with both v4l2-ctl and gst: > > $ v4l2-ctl -d0 -e1 --stream-dmabuf --stream-out-mmap > $ v4l2-ctl -d0 -e1 --stream-mmap --stream-out-dmabuf > $ gst-launch-1.0 v4l2src device=/dev/video0 io-mode=dmabuf ! v4l2sink device=/dev/video1 io-mode=dmabuf-import > > And I didn't find any issues but more testing will be appreciated. > > Best regards, > Javier > Hello all, Tested this using the same GStreamer pipeline as Javier mentions above. It works nicely. Thanks, Luis Tested-by: Luis de Bethencourt <luisbg@xxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html