diff --git a/sys/msdk/gstmsdkcontextutil.c b/sys/msdk/gstmsdkcontextutil.c index 0e25080efb7f0f5cd853e405314d8b7566fe5e97..6f5c7e41d74a8231e106642bcbe61b8a4f4fec10 100644 --- a/sys/msdk/gstmsdkcontextutil.c +++ b/sys/msdk/gstmsdkcontextutil.c @@ -161,6 +161,7 @@ gst_msdk_context_find (GstElement * element, GstMsdkContext ** context_ptr) return TRUE; } + /* This may indirectly set *context_ptr, see function body */ _gst_context_query (element, GST_MSDK_CONTEXT_TYPE_NAME); if (*context_ptr) diff --git a/sys/msdk/gstmsdkdec.c b/sys/msdk/gstmsdkdec.c index 923b5c0ca13c6e1973332db3c63c5ae855e1f001..97da6cd6bf9f9471748ac34626c8cca22f65b139 100644 --- a/sys/msdk/gstmsdkdec.c +++ b/sys/msdk/gstmsdkdec.c @@ -684,6 +684,12 @@ gst_msdkdec_context_prepare (GstMsdkDec * thiz) if (!gst_msdk_context_find (GST_ELEMENT_CAST (thiz), &thiz->context)) return FALSE; + if (thiz->context == thiz->old_context) { + GST_INFO_OBJECT (thiz, "Found old context %" GST_PTR_FORMAT + ", reusing as-is", thiz->context); + return TRUE; + } + /* TODO: Currently d3d allocator is not implemented. * So decoder uses system memory by default on Windows. */ @@ -740,6 +746,11 @@ gst_msdkdec_start (GstVideoDecoder * decoder) thiz->context); } + /* Save the current context in a separate field so that we know whether it + * has changed between calls to _start() */ + gst_object_replace ((GstObject **) & thiz->old_context, + (GstObject *) thiz->context); + gst_msdk_context_add_shared_async_depth (thiz->context, thiz->async_depth); return TRUE; diff --git a/sys/msdk/gstmsdkdec.h b/sys/msdk/gstmsdkdec.h index 8bb746b858fc933e1791454b1ecb509f022f0044..b15c855c04b80dcfa1d43a7493c5fcc7cefeb01b 100644 --- a/sys/msdk/gstmsdkdec.h +++ b/sys/msdk/gstmsdkdec.h @@ -90,6 +90,7 @@ struct _GstMsdkDec /* MFX context */ GstMsdkContext *context; + GstMsdkContext *old_context; mfxVideoParam param; GArray *tasks; guint next_task; diff --git a/sys/msdk/gstmsdkenc.c b/sys/msdk/gstmsdkenc.c index 967e7e9d7cd1a8f9eb0412effa99bc3f233e119c..17732643702b812dc90ba314f96e36ecdd6a81cf 100644 --- a/sys/msdk/gstmsdkenc.c +++ b/sys/msdk/gstmsdkenc.c @@ -1605,6 +1605,12 @@ gst_msdkenc_context_prepare (GstMsdkEnc * thiz) if (!gst_msdk_context_find (GST_ELEMENT_CAST (thiz), &thiz->context)) return FALSE; + if (thiz->context == thiz->old_context) { + GST_INFO_OBJECT (thiz, "Found old context %" GST_PTR_FORMAT + ", reusing as-is", thiz->context); + return TRUE; + } + GST_INFO_OBJECT (thiz, "Found context %" GST_PTR_FORMAT " from neighbour", thiz->context); @@ -1656,6 +1662,11 @@ gst_msdkenc_start (GstVideoEncoder * encoder) thiz->context); } + /* Save the current context in a separate field so that we know whether it + * has changed between calls to _start() */ + gst_object_replace ((GstObject **) & thiz->old_context, + (GstObject *) thiz->context); + gst_msdk_context_add_shared_async_depth (thiz->context, thiz->async_depth); /* Set the minimum pts to some huge value (1000 hours). This keeps @@ -1779,6 +1790,7 @@ gst_msdkenc_finalize (GObject * object) gst_clear_object (&thiz->msdk_pool); gst_clear_object (&thiz->msdk_converted_pool); + gst_clear_object (&thiz->old_context); G_OBJECT_CLASS (parent_class)->finalize (object); } diff --git a/sys/msdk/gstmsdkenc.h b/sys/msdk/gstmsdkenc.h index 98958744970f22f4f2957344e5a67127e7fe49ee..2703fb188a24b1fc5ff6096a3214c4edc1f5799f 100644 --- a/sys/msdk/gstmsdkenc.h +++ b/sys/msdk/gstmsdkenc.h @@ -99,6 +99,7 @@ struct _GstMsdkEnc /* MFX context */ GstMsdkContext *context; + GstMsdkContext *old_context; mfxVideoParam param; guint num_surfaces; guint num_tasks; diff --git a/sys/msdk/gstmsdkvpp.c b/sys/msdk/gstmsdkvpp.c index 05747d9b858d071d87e2a9f136b97c9f3b795c47..dce4934dfae1a804792fd7fc800c397bd018d8dd 100644 --- a/sys/msdk/gstmsdkvpp.c +++ b/sys/msdk/gstmsdkvpp.c @@ -197,6 +197,12 @@ gst_msdkvpp_context_prepare (GstMsdkVPP * thiz) if (!gst_msdk_context_find (GST_ELEMENT_CAST (thiz), &thiz->context)) return FALSE; + if (thiz->context == thiz->old_context) { + GST_INFO_OBJECT (thiz, "Found old context %" GST_PTR_FORMAT + ", reusing as-is", thiz->context); + return TRUE; + } + GST_INFO_OBJECT (thiz, "Found context %" GST_PTR_FORMAT " from neighbour", thiz->context); @@ -247,6 +253,11 @@ ensure_context (GstBaseTransform * trans) thiz->context); } + /* Save the current context in a separate field so that we know whether it + * has changed between calls to _start() */ + gst_object_replace ((GstObject **) & thiz->old_context, + (GstObject *) thiz->context); + gst_msdk_context_add_shared_async_depth (thiz->context, thiz->async_depth); return TRUE; @@ -1476,6 +1487,10 @@ gst_msdkvpp_get_property (GObject * object, guint prop_id, static void gst_msdkvpp_finalize (GObject * object) { + GstMsdkVPP *thiz = GST_MSDKVPP (object); + + gst_clear_object (&thiz->old_context); + G_OBJECT_CLASS (parent_class)->finalize (object); } diff --git a/sys/msdk/gstmsdkvpp.h b/sys/msdk/gstmsdkvpp.h index c644084d3d84725db54104d8d10fe83e94a71f93..83ff2b7f5c4330ec1d4b9c2813aaac2f364dec8e 100644 --- a/sys/msdk/gstmsdkvpp.h +++ b/sys/msdk/gstmsdkvpp.h @@ -89,6 +89,7 @@ struct _GstMsdkVPP /* MFX context */ GstMsdkContext *context; + GstMsdkContext *old_context; mfxVideoParam param; guint in_num_surfaces; guint out_num_surfaces;