After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 758424 - v4l2videodec: Keep the input buffers, they are needed to copy metadata
v4l2videodec: Keep the input buffers, they are needed to copy metadata
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 1.8.2
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-11-20 20:01 UTC by Nicolas Dufresne (ndufresne)
Modified: 2016-06-07 21:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
v4l2videodec: Keep part of the input buffer (2.34 KB, patch)
2016-06-07 21:03 UTC, Nicolas Dufresne (ndufresne)
committed Details | Review

Description Nicolas Dufresne (ndufresne) 2015-11-20 20:01:52 UTC
In the v4l2videodec element, we drop the input buffer, to avoid wasting memory. Though, this prevent the metadata from being copied later.

videodecoder gstvideodecoder.c:3074:gst_video_decoder_finish_frame:<v4l2video0dec0> Can't copy metadata because input frame disappeared

Extra careful note, right now we don't share the input buffer pool (it's probably not a good idea anyway), though if we did, in this case, we'd have to make sure that we have enough input buffers (basically, the the capture minimum becomes the encoded buffer minimum).
Comment 1 Nicolas Dufresne (ndufresne) 2016-06-07 21:03:44 UTC
Created attachment 329344 [details] [review]
v4l2videodec: Keep part of the input buffer

Instead of completely getting rid of the input buffer, copy
the metadata, the flags and the timestamp into an empty buffer.
This way the decoder base class can copy that information again
to the output buffer.
Comment 2 Nicolas Dufresne (ndufresne) 2016-06-07 21:04:19 UTC
Attachment 329344 [details] pushed as 462800e - v4l2videodec: Keep part of the input buffer
Comment 3 Nicolas Dufresne (ndufresne) 2016-06-07 21:19:59 UTC
Branch: 1.8
Commit: 8e155edca7cc47b9ae8b786040b92a86aefd8ff1