glfilter: add parent meta to output buffer for input buffer
authorHaihua Hu <jared.hu@nxp.com>
Tue, 13 Jun 2023 02:30:16 +0000 (10:30 +0800)
committerTim-Philipp Müller <tim@centricular.com>
Tue, 13 Jun 2023 11:34:57 +0000 (12:34 +0100)
glfilter will unref input buffer after _transform() call immidiately,
but gpu may still reading input buffer for rendering because gl
api is executed async. Need hold reference for input buffer by
adding parent meta to output buffer.

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

subprojects/gst-plugins-base/gst-libs/gst/gl/gstglfilter.c

index edb12b2..54bb050 100644 (file)
@@ -1032,6 +1032,14 @@ gst_gl_filter_transform (GstBaseTransform * bt, GstBuffer * inbuf,
   if (out_sync_meta)
     gst_gl_sync_meta_set_sync_point (out_sync_meta, context);
 
+  /* since gl api is async operation, when return from transform()
+   * function, basetransform will unref input buffer immidiately,
+   * but gpu may still reading input buffer for rendering. Add parent
+   * buffer meta to hold one reference of inbuf, this can avoid this
+   * buffer sync problem.
+   */
+  gst_buffer_add_parent_buffer_meta (outbuf, inbuf);
+
   return ret ? GST_FLOW_OK : GST_FLOW_ERROR;
 }