multiqueue: Use running time of gap events for wakeups.
authorJan Schmidt <jan@centricular.com>
Sun, 5 Sep 2021 15:43:57 +0000 (01:43 +1000)
committerJan Schmidt <jan@centricular.com>
Sun, 5 Sep 2021 15:43:57 +0000 (01:43 +1000)
Use gap events to update the next_time of a queue the same
as buffers or segment events. Fixes problems where a group
consisting only of sparse streams primarily driven by
gap events would stall with a full multiqueue because
unlinked streams in the group were not being woken to
push data.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/879>

plugins/elements/gstmultiqueue.c

index bc86999..e6231c3 100644 (file)
@@ -1822,6 +1822,10 @@ apply_gap (GstMultiQueue * mq, GstSingleQueue * sq, GstEvent * event,
       timestamp += duration;
     }
 
+    GST_DEBUG_OBJECT (mq, "queue %d, %s position updated to %" GST_TIME_FORMAT,
+        sq->id, segment == &sq->sink_segment ? "sink" : "src",
+        GST_TIME_ARGS (timestamp));
+
     segment->position = timestamp;
 
     if (segment == &sq->sink_segment)
@@ -1884,6 +1888,14 @@ get_running_time (GstSegment * segment, GstMiniObject * object, gboolean end)
             my_segment_to_running_time ((GstSegment *) new_segment,
             new_segment->start);
       }
+    } else if (GST_EVENT_TYPE (event) == GST_EVENT_GAP) {
+      GstClockTime ts, dur;
+      gst_event_parse_gap (event, &ts, &dur);
+      if (GST_CLOCK_TIME_IS_VALID (ts)) {
+        if (GST_CLOCK_TIME_IS_VALID (dur))
+          ts += dur;
+        time = my_segment_to_running_time (segment, ts);
+      }
     }
   }