[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
Hans and Laurent, Thanks for the feedback. On Tue, Jun 26, 2012 at 2:40 AM, Hans Verkuil wrote: > On Tue 26 June 2012 11:11:06 Laurent Pinchart wrote: >> Hi Dima and Tomasz, >> >> Sorry for the late reply. >> >> On Tuesday 26 June 2012 10:40:44 Tomasz Stani

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
On Tue, Jun 26, 2012 at 1:40 AM, Tomasz Stanislawski wrote: > Hi Dima Zavin, > Thank you for the patch and for a ping remainder :). > > You are right. The unmap is missing in __vb2_queue_cancel. > I will apply your fix into next version of V4L2 support for dmabuf. > >

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
On Tue, Jun 26, 2012 at 1:40 AM, Tomasz Stanislawski t.stanisl...@samsung.com wrote: Hi Dima Zavin, Thank you for the patch and for a ping remainder :). You are right. The unmap is missing in __vb2_queue_cancel. I will apply your fix into next version of V4L2 support for dmabuf. Please

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-26 Thread Dima Zavin
Hans and Laurent, Thanks for the feedback. On Tue, Jun 26, 2012 at 2:40 AM, Hans Verkuil hverk...@xs4all.nl wrote: On Tue 26 June 2012 11:11:06 Laurent Pinchart wrote: Hi Dima and Tomasz, Sorry for the late reply. On Tuesday 26 June 2012 10:40:44 Tomasz Stanislawski wrote: Hi Dima Zavin

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-25 Thread Dima Zavin
ping? On Tue, Jun 19, 2012 at 11:12 PM, Dima Zavin wrote: > Tomasz, > > I've encountered an issue with this patch when userspace does several > stream_on/stream_off cycles. When the user tries to qbuf a buffer > after doing stream_off, we trigger the "dmabuf already pinned

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-25 Thread Dima Zavin
ping? On Tue, Jun 19, 2012 at 11:12 PM, Dima Zavin dmitr...@google.com wrote: Tomasz, I've encountered an issue with this patch when userspace does several stream_on/stream_off cycles. When the user tries to qbuf a buffer after doing stream_off, we trigger the dmabuf already pinned warning

[PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-20 Thread Dima Zavin
ki Signed-off-by: Dima Zavin --- drivers/media/video/videobuf2-core.c | 22 -- 1 files changed, 20 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/videobuf2-core.c b/drivers/media/video/videobuf2-core.c index b431dc6..e2a8f12 100644 --- a/drivers/media/video

Re: [PATCHv7 03/15] v4l: vb2: add support for shared buffer (dma_buf)

2012-06-20 Thread Dima Zavin
Stanislawski t.stanisl...@samsung.com Signed-off-by: Dima Zavin d...@android.com --- drivers/media/video/videobuf2-core.c | 22 -- 1 files changed, 20 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/videobuf2-core.c b/drivers/media/video/videobuf2-core.c index