[Bug 784599] kmssink: support videooverlay interface
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Wed Jul 26 18:09:48 UTC 2017
https://bugzilla.gnome.org/show_bug.cgi?id=784599
--- Comment #24 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
Review of attachment 356413:
--> (https://bugzilla.gnome.org/review?bug=784599&attachment=356413)
Looks good indeed, I only have one question below about locking. Is the other
patch obsolete now ?
::: sys/kms/gstkmssink.c
@@ +1293,3 @@
+
+ if (self->can_scale) {
+ gst_kms_sink_show_frame (GST_VIDEO_SINK (self), NULL);
Are you missing some locking here ? Normally show_frame() is called with stream
lock held, but now it's called from application thread, no locks. Have you
verified/protected everything inside that function ?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list