From: Subash Patel <subashrp@gmail.com>
To: Tomasz Stanislawski <t.stanislaws@samsung.com>
Cc: linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org,
airlied@redhat.com, m.szyprowski@samsung.com,
kyungmin.park@samsung.com, laurent.pinchart@ideasonboard.com,
sumit.semwal@ti.com, daeinki@gmail.com, daniel.vetter@ffwll.ch,
robdclark@gmail.com, pawel@osciak.com,
linaro-mm-sig@lists.linaro.org, hverkuil@xs4all.nl,
remi@remlab.net, mchehab@redhat.com, linux-doc@vger.kernel.org,
g.liakhovetski@gmx.de,
Andrzej Pietrasiewicz <andrzej.p@samsung.com>,
Kamil Debski <k.debski@samsung.com>
Subject: Re: [PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode
Date: Tue, 08 May 2012 11:11:08 +0530 [thread overview]
Message-ID: <4FA8B1F4.2000200@gmail.com> (raw)
In-Reply-To: <4FA7E623.2060500@samsung.com>
Hi Thomasz,
I have extended the MFC-FIMC testcase posted by Kamil sometime ago to
sanity test the UMM patches. This test is multi-threaded(further
explanation for developers who may not have seen it yet), where thread
one parses the encoded stream and feeds into the codec IP driver(aka
MFC driver). Second thread will dequeue the buffer from MFC driver
(DMA_BUF export) and queues it into a CSC IP(aka FIMC) driver(DMA_BUF
import). Third thread dequeues the frame from FIMC driver and either
pushes it into LCD driver for display or dumps to a flat file for analysis.
MFC driver exports the fd's and FIMC driver imports and attaches it.
During FIMC QBUF (thats when the attach and map happens), it is observed
that in the function vb2_dc_map_dmabuf() call to
vb2_dc_get_contiguous_size() fails. This is because contig_size <
buf->size. contig_size is calculated from the SGT which we would have
constructed in the function vb2_dc_pages_to_sgt().
Let me know if you need more details.
Regards,
Subash
On 05/07/2012 08:41 PM, Tomasz Stanislawski wrote:
> Hi Subash,
> Could you provide a detailed description of a test case
> that causes a failure of vb2_dc_pages_to_sgt?
>
> Regards,
> Tomasz Stanislawski
next prev parent reply other threads:[~2012-05-08 5:41 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-20 14:45 [PATCHv5 00/13] Integration of videobuf2 with dmabuf Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 01/13] v4l: Add DMABUF as a memory type Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 02/13] Documentation: media: description of DMABUF importing in V4L2 Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 03/13] v4l: vb2: add support for shared buffer (dma_buf) Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 04/13] v4l: vb: remove warnings about MEMORY_DMABUF Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 05/13] v4l: vb2-dma-contig: Shorten vb2_dma_contig prefix to vb2_dc Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 06/13] v4l: vb2-dma-contig: Remove unneeded allocation context structure Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 07/13] v4l: vb2-dma-contig: Reorder functions Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode Tomasz Stanislawski
2012-04-21 17:24 ` Laurent Pinchart
2012-05-07 14:38 ` Subash Patel
2012-05-07 15:11 ` Tomasz Stanislawski
2012-05-08 5:41 ` Subash Patel [this message]
2012-05-08 9:14 ` Laurent Pinchart
2012-05-08 11:15 ` Subash Patel
2012-05-09 6:46 ` Subash Patel
2012-05-09 9:58 ` Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 09/13] v4l: vb2: add prepare/finish callbacks to allocators Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 10/13] v4l: vb2-dma-contig: add prepare/finish to dma-contig allocator Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 11/13] v4l: vb2-dma-contig: add support for dma_buf importing Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 12/13] v4l: s5p-tv: mixer: support for dmabuf importing Tomasz Stanislawski
2012-04-20 14:45 ` [PATCHv5 13/13] v4l: s5p-fimc: " Tomasz Stanislawski
2012-04-21 15:59 ` [PATCHv5 00/13] Integration of videobuf2 with dmabuf Rob Landley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FA8B1F4.2000200@gmail.com \
--to=subashrp@gmail.com \
--cc=airlied@redhat.com \
--cc=andrzej.p@samsung.com \
--cc=daeinki@gmail.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=g.liakhovetski@gmx.de \
--cc=hverkuil@xs4all.nl \
--cc=k.debski@samsung.com \
--cc=kyungmin.park@samsung.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linaro-mm-sig@lists.linaro.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=mchehab@redhat.com \
--cc=pawel@osciak.com \
--cc=remi@remlab.net \
--cc=robdclark@gmail.com \
--cc=sumit.semwal@ti.com \
--cc=t.stanislaws@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).