diff options
author | Nicolas Dufresne <nicolas.dufresne@collabora.com> | 2018-09-07 22:57:30 -0400 |
---|---|---|
committer | Nicolas Dufresne <nicolas.dufresne@collabora.com> | 2018-09-10 17:09:11 -0400 |
commit | a84badc5c4fd7d02937cea8eed23c80579a04c9f (patch) | |
tree | ff021ecebd2d6b617c898cf79b6de153f8732609 | |
parent | d80504afcc3902dc7f7f5be18af908c45a8cb094 (diff) | |
download | gst-omx-a84badc5c4fd7d02937cea8eed23c80579a04c9f.tar.gz |
omxvideoenc: Remove spurious locking
The method we call in the context of pushing a buffer are all thread
safe. Holding a lock would prevent input buffers from being queued while
pushing.
https://bugzilla.gnome.org/show_bug.cgi?id=715192
-rw-r--r-- | omx/gstomxvideoenc.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/omx/gstomxvideoenc.c b/omx/gstomxvideoenc.c index f7295dd..abac46d 100644 --- a/omx/gstomxvideoenc.c +++ b/omx/gstomxvideoenc.c @@ -1534,7 +1534,6 @@ gst_omx_video_enc_loop (GstOMXVideoEnc * self) gst_omx_buffer_flags_to_string (buf->omx_buf->nFlags), (guint64) GST_OMX_GET_TICKS (buf->omx_buf->nTimeStamp)); - GST_VIDEO_ENCODER_STREAM_LOCK (self); frame = gst_omx_video_find_nearest_frame (buf, gst_video_encoder_get_frames (GST_VIDEO_ENCODER (self))); @@ -1547,15 +1546,15 @@ gst_omx_video_enc_loop (GstOMXVideoEnc * self) if (err != OMX_ErrorNone) goto release_error; + GST_VIDEO_ENCODER_STREAM_LOCK (self); self->downstream_flow_ret = flow_ret; + GST_VIDEO_ENCODER_STREAM_UNLOCK (self); GST_DEBUG_OBJECT (self, "Read frame from component"); if (flow_ret != GST_FLOW_OK) goto flow_error; - GST_VIDEO_ENCODER_STREAM_UNLOCK (self); - return; component_error: @@ -1592,13 +1591,12 @@ eos: GST_VIDEO_ENCODER_STREAM_LOCK (self); self->downstream_flow_ret = flow_ret; + GST_VIDEO_ENCODER_STREAM_UNLOCK (self); /* Here we fallback and pause the task for the EOS case */ if (flow_ret != GST_FLOW_OK) goto flow_error; - GST_VIDEO_ENCODER_STREAM_UNLOCK (self); - return; } flow_error: @@ -1618,7 +1616,6 @@ flow_error: GST_DEBUG_OBJECT (self, "Flushing -- stopping task"); } gst_omx_video_enc_pause_loop (self, flow_ret); - GST_VIDEO_ENCODER_STREAM_UNLOCK (self); return; } reconfigure_error: @@ -1643,7 +1640,6 @@ release_error: gst_omx_error_to_string (err), err)); gst_pad_push_event (GST_VIDEO_ENCODER_SRC_PAD (self), gst_event_new_eos ()); gst_omx_video_enc_pause_loop (self, GST_FLOW_ERROR); - GST_VIDEO_ENCODER_STREAM_UNLOCK (self); return; } } |