[Bug 752962] v4l2videodec: Add resolution change support

[GStreamer (GNOME Bugzilla)]

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

--- Comment #22 from Nicolas Dufresne (stormer) <nicolas.dufresne at collabora.co.uk> ---
(In reply to kevin from comment #21)
> How many threads will be used for CAPTURE port? Below process will block:
> 
> 1. get recycled video buffer from video decoder base pool.
> 2. dequeue video buffer from V4L2 driver.
> 3. dequeue event from V4L2 driver.
> 
> Do those process need separate thread to avoid any dependence?

Of course not, since polling return you a flag telling you which next operation
won't block. This allow you to effectively poll for everything. Then of course
you have the usual streaming thread, which may block (this is normal in
GStreamer to block the streaming thread). This new thread might be optional in
fact, I need to work on the design (code is not a design fwiw) but I believed
it could replace (and centralize the code) for the internal thread required in
encoder/decoder and eventually converters.

My main point, and why I'm not opting for your solution is that it makes no
sense to call gst_buffer_pool_acquire() to discover that there is EVENT
pending. It's a pain to handle, custom return value for acquire() should go
away.

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