[Bug 765435] plugins: rework dmabuf import

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=765435

--- Comment #8 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
(In reply to Nicolas Dufresne (stormer) from comment #7)
> If be pleased to help unhack that code path. Since buffer pool have an
> activated function, there is no need to do anything before that moment. So
> vaapi element could offer multiple pool. That is one step.

Are there elements in upstream already offering multiple pools?

My current approach is offering one pools but juggling with the allocators.

> 
> The next step would be in v4l2src. Currently the default is auto. But only
> mmap and read/write is implemented, we ignore downstream pool (unless
> downstream does not support video meta). What we could do is delay the
> io-mode decision to decide_allocation. From there, we would need to test
> each allocator to see if the memory is DMA buf, and can be imported. If not,
> we'd go for mmap (dmabuf export whenever available), otherwise read/write
> for legacy drivers that don't do anything else. What do you think?

Sounds like a good approach.

Another thing to consider is bug 765600 (Add gst_memory_try_map or new
GstMapFlags to silent GST_ERROR on some situation)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.