[FFmpeg-cvslog] pthread: update all get_buffer checks for get_buffer2

Hendrik Leppkes git at videolan.org
Tue Mar 19 02:32:42 CET 2013


ffmpeg | branch: master | Hendrik Leppkes <h.leppkes at gmail.com> | Sun Mar 17 10:59:39 2013 +0100| [97f8c6e14753b94c1f6a96fe354a125bbfdea2cb] | committer: Michael Niedermayer

pthread: update all get_buffer checks for get_buffer2

This restores the performance of simple frame-threaded codecs back to
their pre-merge levels.

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=97f8c6e14753b94c1f6a96fe354a125bbfdea2cb
---

 libavcodec/pthread.c |   20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/libavcodec/pthread.c b/libavcodec/pthread.c
index 799644f..b32d707 100644
--- a/libavcodec/pthread.c
+++ b/libavcodec/pthread.c
@@ -376,7 +376,11 @@ static attribute_align_arg void *frame_worker_thread(void *arg)
 
         if (fctx->die) break;
 
-        if (!codec->update_thread_context && (avctx->thread_safe_callbacks || avctx->get_buffer == avcodec_default_get_buffer))
+        if (!codec->update_thread_context && (avctx->thread_safe_callbacks || (
+#if FF_API_GET_BUFFER
+            !avctx->get_buffer &&
+#endif
+            avctx->get_buffer2 == avcodec_default_get_buffer2)))
             ff_thread_finish_setup(avctx);
 
         avcodec_get_frame_defaults(&p->frame);
@@ -933,8 +937,11 @@ int ff_thread_can_start_frame(AVCodecContext *avctx)
 {
     PerThreadContext *p = avctx->thread_opaque;
     if ((avctx->active_thread_type&FF_THREAD_FRAME) && p->state != STATE_SETTING_UP &&
-        (avctx->codec->update_thread_context || (!avctx->thread_safe_callbacks &&
-                avctx->get_buffer != avcodec_default_get_buffer))) {
+        (avctx->codec->update_thread_context || (!avctx->thread_safe_callbacks && (
+#if FF_API_GET_BUFFER
+                avctx->get_buffer ||
+#endif
+                avctx->get_buffer2 != avcodec_default_get_buffer2)))) {
         return 0;
     }
     return 1;
@@ -953,8 +960,11 @@ static int thread_get_buffer_internal(AVCodecContext *avctx, ThreadFrame *f, int
         return ff_get_buffer(avctx, f->f, flags);
 
     if (p->state != STATE_SETTING_UP &&
-        (avctx->codec->update_thread_context || (!avctx->thread_safe_callbacks &&
-                avctx->get_buffer != avcodec_default_get_buffer))) {
+        (avctx->codec->update_thread_context || (!avctx->thread_safe_callbacks && (
+#if FF_API_GET_BUFFER
+                avctx->get_buffer ||
+#endif
+                avctx->get_buffer2 != avcodec_default_get_buffer2)))) {
         av_log(avctx, AV_LOG_ERROR, "get_buffer() cannot be called after ff_thread_finish_setup()\n");
         return -1;
     }



More information about the ffmpeg-cvslog mailing list