[Bug 727100] v4l2bufferpool: Race condition on qbuf/dqbuf in a multithreaded context

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=727100
  GStreamer | gst-plugins-good | 1.2.3

--- Comment #11 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2014-04-09 16:00:52 UTC ---
(In reply to comment #10)
> Review of attachment 273167 [details]:
> 
> ::: sys/v4l2/gstv4l2bufferpool.c
> @@ +852,3 @@
> +    pool->buffers[index] = NULL;
> +    pool->num_queued--;
> +    GST_OBJECT_UNLOCK (pool);
> 
> Maybe we should process QBUF ioctl error in 'queue_failed' block.

It's a small change, but it also could be nicer. Btw, there is a big
refactoring I'm working on, that add an allocator and all. But when that is
done (really soon now), I'll have a look if there is a lockless way to handle
that.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.