On 11/07/2014 09:50 AM, Hans Verkuil wrote: > Changes since v4: > - Rebased to latest media_tree master > > Changes since v3: > > - Dropped patch 02/10: succeeded by patch 10/15 in this series > - Added patches 11-15 to correctly handle syncing/mapping dmabuf > buffers for CPU access. This was never done correctly before. > Many thanks to Pawel Osciak for helping me with this during the > media mini-summit last week. > > The patch series adds an allocation context to dma-sg and uses that to move > dma_(un)map_sg into the vb2 framework, which is where it belongs. > > Some drivers needs to fixup the buffers before giving it back to userspace > (or before handing it over to the kernel). Document that this can be done > in buf_prepare and buf_finish. > > The last 5 patches make this more strict by requiring all cpu access to > be bracketed by calls to vb2_plane_begin/end_cpu_access() which replaces > the old vb2_plane_vaddr() call. > > Note: two drivers still use the vb2_plane_addr() call: coda and > exynos4-is/fimc-capture.c. For both drivers I will need some help since > I am not sure where to put the begin/end calls. Patch 14 removes > the vb2_plane_vaddr call, so obviously those two drivers won't compile > after that. > > DMABUF export support is added to dma-sg and vmalloc, so now all memory > models support DMABUF importing and exporting. > > I am inclined to make a pull request for patches 1-10 if there are no > new comments. The issues that patches 11-15 address are separate from > the patches 1-10 and this is only an issue when using dmabuf with > drivers that need cpu access. To be specific: consider patches 1-10 as being patches ready to merge, while patches 11-15 are still in the RFC stage. Regards, Hans > > Reviews are very welcome. > > Regards, > > Hans > > -- > 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 > -- 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
