[Bug 777399] v4l2allocator: memory referenced too late

[GStreamer (GNOME Bugzilla)]

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

Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |NEW

--- Comment #10 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
I believe I need to apologies now. I think I finally understood this case and
why it works with your fix. I tend to forget that all this
allocator/buffer-stuff is trying to be lockless (a bit scary). So indeed, if
one get pre-empted after QBUF, but before the ref, then another thread could
DQUEUE and free immediatly. QDBUF bufs assume the allocater kept a ref (which
may not have been taken yet). Again appology, not an easy one, clearly not what
I expected for this code initially.

p.s. to reproduce easily, add a sleep() after QBUF.

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