Merge commit '50612484e058e8b241f0528584d64d9d2ccebf12'
authorMichael Niedermayer <michaelni@gmx.at>
Wed, 17 Jul 2013 08:33:02 +0000 (10:33 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Wed, 17 Jul 2013 08:40:15 +0000 (10:40 +0200)
* commit '50612484e058e8b241f0528584d64d9d2ccebf12':
  pthread: Rename thread_init to avoid symbol collision

Conflicts:
libavcodec/pthread.c
libavfilter/pthread.c

See: 674d8a9629609a17fd6916164556cb012d6e6a1c
Merged-by: Michael Niedermayer <michaelni@gmx.at>
1  2 
libavcodec/pthread.c
libavfilter/pthread.c

diff --combined libavcodec/pthread.c
@@@ -6,20 -6,20 +6,20 @@@
   * to Michael Niedermayer <michaelni@gmx.at> for writing initial
   * implementation.
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -42,8 -42,6 +42,8 @@@
  #include <pthread.h>
  #elif HAVE_W32THREADS
  #include "compat/w32pthreads.h"
 +#elif HAVE_OS2THREADS
 +#include "compat/os2threads.h"
  #endif
  
  typedef int (action_func)(AVCodecContext *c, void *arg);
@@@ -63,7 -61,6 +63,7 @@@ typedef struct ThreadContext 
      pthread_cond_t current_job_cond;
      pthread_mutex_t current_job_lock;
      int current_job;
 +    unsigned int current_execute;
      int done;
  } ThreadContext;
  
@@@ -99,10 -96,6 +99,10 @@@ typedef struct PerThreadContext 
                                       * Set when the codec calls get_buffer().
                                       * State is returned to STATE_SETTING_UP afterwards.
                                       */
 +        STATE_GET_FORMAT,           /**<
 +                                     * Set when the codec calls get_format().
 +                                     * State is returned to STATE_SETTING_UP afterwards.
 +                                     */
          STATE_SETUP_FINISHED        ///< Set after the codec has called ff_thread_finish_setup().
      } state;
  
  
      AVFrame *requested_frame;       ///< AVFrame the codec passed to get_buffer()
      int      requested_flags;       ///< flags passed to get_buffer() for requested_frame
 +
 +    const enum AVPixelFormat *available_formats; ///< Format array for get_format()
 +    enum AVPixelFormat result_format;            ///< get_format() result
  } PerThreadContext;
  
  /**
@@@ -151,7 -141,6 +151,7 @@@ static void* attribute_align_arg worker
      AVCodecContext *avctx = v;
      ThreadContext *c = avctx->thread_opaque;
      int our_job = c->job_count;
 +    int last_execute = 0;
      int thread_count = avctx->thread_count;
      int self_id;
  
              if (c->current_job == thread_count + c->job_count)
                  pthread_cond_signal(&c->last_job_cond);
  
 -            pthread_cond_wait(&c->current_job_cond, &c->current_job_lock);
 +            while (last_execute == c->current_execute && !c->done)
 +                pthread_cond_wait(&c->current_job_cond, &c->current_job_lock);
 +            last_execute = c->current_execute;
              our_job = self_id;
  
              if (c->done) {
  
  static av_always_inline void avcodec_thread_park_workers(ThreadContext *c, int thread_count)
  {
 -    pthread_cond_wait(&c->last_job_cond, &c->current_job_lock);
 +    while (c->current_job != thread_count + c->job_count)
 +        pthread_cond_wait(&c->last_job_cond, &c->current_job_lock);
      pthread_mutex_unlock(&c->current_job_lock);
  }
  
@@@ -234,7 -220,6 +234,7 @@@ static int avcodec_thread_execute(AVCod
          c->rets = &dummy_ret;
          c->rets_count = 1;
      }
 +    c->current_execute++;
      pthread_cond_broadcast(&c->current_job_cond);
  
      avcodec_thread_park_workers(c, avctx->thread_count);
@@@ -249,7 -234,7 +249,7 @@@ static int avcodec_thread_execute2(AVCo
      return avcodec_thread_execute(avctx, NULL, arg, ret, job_count, 0);
  }
  
- static int avcodec_thread_init(AVCodecContext *avctx)
+ static int thread_init_internal(AVCodecContext *avctx)
  {
      int i;
      ThreadContext *c;
  
      if (!thread_count) {
          int nb_cpus = av_cpu_count();
 -        av_log(avctx, AV_LOG_DEBUG, "detected %d logical cores\n", nb_cpus);
 +        if  (avctx->height)
 +            nb_cpus = FFMIN(nb_cpus, (avctx->height+15)/16);
          // use number of cores + 1 as thread count if there is more than one
          if (nb_cpus > 1)
              thread_count = avctx->thread_count = FFMIN(nb_cpus + 1, MAX_AUTO_THREADS);
      return 0;
  }
  
 +#define THREAD_SAFE_CALLBACKS(avctx) \
 +((avctx)->thread_safe_callbacks || (!(avctx)->get_buffer && (avctx)->get_buffer2 == avcodec_default_get_buffer2))
 +
  /**
   * Codec worker thread.
   *
@@@ -323,16 -304,20 +323,16 @@@ static attribute_align_arg void *frame_
      AVCodecContext *avctx = p->avctx;
      const AVCodec *codec = avctx->codec;
  
 +    pthread_mutex_lock(&p->mutex);
      while (1) {
 -        if (p->state == STATE_INPUT_READY && !fctx->die) {
 -            pthread_mutex_lock(&p->mutex);
              while (p->state == STATE_INPUT_READY && !fctx->die)
                  pthread_cond_wait(&p->input_cond, &p->mutex);
 -            pthread_mutex_unlock(&p->mutex);
 -        }
  
          if (fctx->die) break;
  
 -        if (!codec->update_thread_context && avctx->thread_safe_callbacks)
 +        if (!codec->update_thread_context && THREAD_SAFE_CALLBACKS(avctx))
              ff_thread_finish_setup(avctx);
  
 -        pthread_mutex_lock(&p->mutex);
          avcodec_get_frame_defaults(&p->frame);
          p->got_frame = 0;
          p->result = codec->decode(avctx, &p->frame, &p->got_frame, &p->avpkt);
  
          if (p->state == STATE_SETTING_UP) ff_thread_finish_setup(avctx);
  
 +        pthread_mutex_lock(&p->progress_mutex);
 +#if 0 //BUFREF-FIXME
 +        for (i = 0; i < MAX_BUFFERS; i++)
 +            if (p->progress_used[i] && (p->got_frame || p->result<0 || avctx->codec_id != AV_CODEC_ID_H264)) {
 +                p->progress[i][0] = INT_MAX;
 +                p->progress[i][1] = INT_MAX;
 +            }
 +#endif
          p->state = STATE_INPUT_READY;
  
 -        pthread_mutex_lock(&p->progress_mutex);
 +        pthread_cond_broadcast(&p->progress_cond);
          pthread_cond_signal(&p->output_cond);
          pthread_mutex_unlock(&p->progress_mutex);
 -
 -        pthread_mutex_unlock(&p->mutex);
      }
 +    pthread_mutex_unlock(&p->mutex);
  
      return NULL;
  }
@@@ -403,15 -381,9 +403,15 @@@ static int update_context_from_thread(A
  
          dst->hwaccel = src->hwaccel;
          dst->hwaccel_context = src->hwaccel_context;
 +
 +        dst->channels       = src->channels;
 +        dst->sample_rate    = src->sample_rate;
 +        dst->sample_fmt     = src->sample_fmt;
 +        dst->channel_layout = src->channel_layout;
      }
  
      if (for_user) {
 +        dst->delay       = src->thread_count - 1;
          dst->coded_frame = src->coded_frame;
      } else {
          if (dst->codec->update_thread_context)
@@@ -451,7 -423,6 +451,7 @@@ static int update_context_from_user(AVC
  
      dst->frame_number     = src->frame_number;
      dst->reordered_opaque = src->reordered_opaque;
 +    dst->thread_safe_callbacks = src->thread_safe_callbacks;
  
      if (src->slice_count && src->slice_offset) {
          if (dst->slice_count < src->slice_count) {
@@@ -482,8 -453,7 +482,8 @@@ static void release_delayed_buffers(Per
          pthread_mutex_lock(&fctx->buffer_mutex);
  
          // fix extended data in case the caller screwed it up
 -        av_assert0(p->avctx->codec_type == AVMEDIA_TYPE_VIDEO);
 +        av_assert0(p->avctx->codec_type == AVMEDIA_TYPE_VIDEO ||
 +                   p->avctx->codec_type == AVMEDIA_TYPE_AUDIO);
          f = &p->released_buffers[--p->num_released_buffers];
          f->extended_data = f->data;
          av_frame_unref(f);
@@@ -542,29 -512,17 +542,29 @@@ static int submit_packet(PerThreadConte
       */
  
      if (!p->avctx->thread_safe_callbacks && (
 +         p->avctx->get_format != avcodec_default_get_format ||
  #if FF_API_GET_BUFFER
           p->avctx->get_buffer ||
  #endif
           p->avctx->get_buffer2 != avcodec_default_get_buffer2)) {
          while (p->state != STATE_SETUP_FINISHED && p->state != STATE_INPUT_READY) {
 +            int call_done = 1;
              pthread_mutex_lock(&p->progress_mutex);
              while (p->state == STATE_SETTING_UP)
                  pthread_cond_wait(&p->progress_cond, &p->progress_mutex);
  
 -            if (p->state == STATE_GET_BUFFER) {
 +            switch (p->state) {
 +            case STATE_GET_BUFFER:
                  p->result = ff_get_buffer(p->avctx, p->requested_frame, p->requested_flags);
 +                break;
 +            case STATE_GET_FORMAT:
 +                p->result_format = p->avctx->get_format(p->avctx, p->available_formats);
 +                break;
 +            default:
 +                call_done = 0;
 +                break;
 +            }
 +            if (call_done) {
                  p->state  = STATE_SETTING_UP;
                  pthread_cond_signal(&p->progress_cond);
              }
@@@ -601,10 -559,9 +601,10 @@@ int ff_thread_decode_frame(AVCodecConte
       * If we're still receiving the initial packets, don't return a frame.
       */
  
 -    if (fctx->delaying) {
 -        if (fctx->next_decoding >= (avctx->thread_count-1)) fctx->delaying = 0;
 +    if (fctx->next_decoding > (avctx->thread_count-1-(avctx->codec_id == AV_CODEC_ID_FFV1)))
 +        fctx->delaying = 0;
  
 +    if (fctx->delaying) {
          *got_picture_ptr=0;
          if (avpkt->size)
              return avpkt->size;
  void ff_thread_report_progress(ThreadFrame *f, int n, int field)
  {
      PerThreadContext *p;
 -    int *progress = f->progress ? (int*)f->progress->data : NULL;
 +    volatile int *progress = f->progress ? (int*)f->progress->data : NULL;
  
      if (!progress || progress[field] >= n) return;
  
  void ff_thread_await_progress(ThreadFrame *f, int n, int field)
  {
      PerThreadContext *p;
 -    int *progress = f->progress ? (int*)f->progress->data : NULL;
 +    volatile int *progress = f->progress ? (int*)f->progress->data : NULL;
  
      if (!progress || progress[field] >= n) return;
  
@@@ -693,10 -650,6 +693,10 @@@ void ff_thread_finish_setup(AVCodecCont
  
      if (!(avctx->active_thread_type&FF_THREAD_FRAME)) return;
  
 +    if(p->state == STATE_SETUP_FINISHED){
 +        av_log(avctx, AV_LOG_WARNING, "Multiple ff_thread_finish_setup() calls\n");
 +    }
 +
      pthread_mutex_lock(&p->progress_mutex);
      p->state = STATE_SETUP_FINISHED;
      pthread_cond_broadcast(&p->progress_cond);
@@@ -717,7 -670,6 +717,7 @@@ static void park_frame_worker_threads(F
                  pthread_cond_wait(&p->output_cond, &p->progress_mutex);
              pthread_mutex_unlock(&p->progress_mutex);
          }
 +        p->got_frame = 0;
      }
  }
  
@@@ -730,11 -682,7 +730,11 @@@ static void frame_thread_free(AVCodecCo
      park_frame_worker_threads(fctx, thread_count);
  
      if (fctx->prev_thread && fctx->prev_thread != fctx->threads)
 -        update_context_from_thread(fctx->threads->avctx, fctx->prev_thread->avctx, 0);
 +        if (update_context_from_thread(fctx->threads->avctx, fctx->prev_thread->avctx, 0) < 0) {
 +            av_log(avctx, AV_LOG_ERROR, "Final thread update failed\n");
 +            fctx->prev_thread->avctx->internal->is_copy = fctx->threads->avctx->internal->is_copy;
 +            fctx->threads->avctx->internal->is_copy = 1;
 +        }
  
      fctx->die = 1;
  
  
          if (p->thread_init)
              pthread_join(p->thread, NULL);
 +        p->thread_init=0;
  
          if (codec->close)
              codec->close(p->avctx);
@@@ -794,8 -741,7 +794,8 @@@ static int frame_thread_init(AVCodecCon
  
      if (!thread_count) {
          int nb_cpus = av_cpu_count();
 -        av_log(avctx, AV_LOG_DEBUG, "detected %d logical cores\n", nb_cpus);
 +        if ((avctx->debug & (FF_DEBUG_VIS_QP | FF_DEBUG_VIS_MB_TYPE)) || avctx->debug_mv)
 +            nb_cpus = 1;
          // use number of cores + 1 as thread count if there is more than one
          if (nb_cpus > 1)
              thread_count = avctx->thread_count = FFMIN(nb_cpus + 1, MAX_AUTO_THREADS);
  
          if (err) goto error;
  
 -        if (!pthread_create(&p->thread, NULL, frame_worker_thread, p))
 -            p->thread_init = 1;
 +        err = AVERROR(pthread_create(&p->thread, NULL, frame_worker_thread, p));
 +        p->thread_init= !err;
 +        if(!p->thread_init)
 +            goto error;
      }
  
      return 0;
@@@ -906,30 -850,18 +906,30 @@@ void ff_thread_flush(AVCodecContext *av
      }
  }
  
 -int ff_thread_get_buffer(AVCodecContext *avctx, ThreadFrame *f, int flags)
 +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 || !THREAD_SAFE_CALLBACKS(avctx))) {
 +        return 0;
 +    }
 +    return 1;
 +}
 +
 +static int thread_get_buffer_internal(AVCodecContext *avctx, ThreadFrame *f, int flags)
  {
      PerThreadContext *p = avctx->thread_opaque;
      int err;
  
      f->owner = avctx;
  
 +    ff_init_buffer_info(avctx, f->f);
 +
      if (!(avctx->active_thread_type & FF_THREAD_FRAME))
          return ff_get_buffer(avctx, f->f, flags);
  
      if (p->state != STATE_SETTING_UP &&
 -        (avctx->codec->update_thread_context || !avctx->thread_safe_callbacks)) {
 +        (avctx->codec->update_thread_context || !THREAD_SAFE_CALLBACKS(avctx))) {
          av_log(avctx, AV_LOG_ERROR, "get_buffer() cannot be called after ff_thread_finish_setup()\n");
          return -1;
      }
      }
  
      pthread_mutex_lock(&p->parent->buffer_mutex);
 +
      if (avctx->thread_safe_callbacks || (
  #if FF_API_GET_BUFFER
          !avctx->get_buffer &&
          avctx->get_buffer2 == avcodec_default_get_buffer2)) {
          err = ff_get_buffer(avctx, f->f, flags);
      } else {
 +        pthread_mutex_lock(&p->progress_mutex);
          p->requested_frame = f->f;
          p->requested_flags = flags;
          p->state = STATE_GET_BUFFER;
 -        pthread_mutex_lock(&p->progress_mutex);
 -        pthread_cond_signal(&p->progress_cond);
 +        pthread_cond_broadcast(&p->progress_cond);
  
          while (p->state != STATE_SETTING_UP)
              pthread_cond_wait(&p->progress_cond, &p->progress_mutex);
          pthread_mutex_unlock(&p->progress_mutex);
  
      }
 -    if (!avctx->thread_safe_callbacks && !avctx->codec->update_thread_context)
 +    if (!THREAD_SAFE_CALLBACKS(avctx) && !avctx->codec->update_thread_context)
          ff_thread_finish_setup(avctx);
  
      if (err)
      return err;
  }
  
 +enum AVPixelFormat ff_thread_get_format(AVCodecContext *avctx, const enum AVPixelFormat *fmt)
 +{
 +    enum AVPixelFormat res;
 +    PerThreadContext *p = avctx->thread_opaque;
 +    if (!(avctx->active_thread_type & FF_THREAD_FRAME) || avctx->thread_safe_callbacks ||
 +        avctx->get_format == avcodec_default_get_format)
 +        return avctx->get_format(avctx, fmt);
 +    if (p->state != STATE_SETTING_UP) {
 +        av_log(avctx, AV_LOG_ERROR, "get_format() cannot be called after ff_thread_finish_setup()\n");
 +        return -1;
 +    }
 +    pthread_mutex_lock(&p->progress_mutex);
 +    p->available_formats = fmt;
 +    p->state = STATE_GET_FORMAT;
 +    pthread_cond_broadcast(&p->progress_cond);
 +
 +    while (p->state != STATE_SETTING_UP)
 +        pthread_cond_wait(&p->progress_cond, &p->progress_mutex);
 +
 +    res = p->result_format;
 +
 +    pthread_mutex_unlock(&p->progress_mutex);
 +
 +    return res;
 +}
 +
 +int ff_thread_get_buffer(AVCodecContext *avctx, ThreadFrame *f, int flags)
 +{
 +    int ret = thread_get_buffer_internal(avctx, f, flags);
 +    if (ret < 0)
 +        av_log(avctx, AV_LOG_ERROR, "thread_get_buffer() failed\n");
 +    return ret;
 +}
 +
  void ff_thread_release_buffer(AVCodecContext *avctx, ThreadFrame *f)
  {
      PerThreadContext *p = avctx->thread_opaque;
@@@ -1103,7 -1000,7 +1103,7 @@@ int ff_thread_init(AVCodecContext *avct
      validate_thread_parameters(avctx);
  
      if (avctx->active_thread_type&FF_THREAD_SLICE)
-         return avcodec_thread_init(avctx);
+         return thread_init_internal(avctx);
      else if (avctx->active_thread_type&FF_THREAD_FRAME)
          return frame_thread_init(avctx);
  
diff --combined libavfilter/pthread.c
@@@ -1,19 -1,19 +1,19 @@@
  /*
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
@@@ -34,8 -34,6 +34,8 @@@
  
  #if HAVE_PTHREADS
  #include <pthread.h>
 +#elif HAVE_OS2THREADS
 +#include "compat/os2threads.h"
  #elif HAVE_W32THREADS
  #include "compat/w32pthreads.h"
  #endif
@@@ -146,12 -144,13 +146,12 @@@ static int thread_execute(AVFilterConte
      return 0;
  }
  
- static int avfilter_thread_init(ThreadContext *c, int nb_threads)
+ static int thread_init_internal(ThreadContext *c, int nb_threads)
  {
      int i, ret;
  
      if (!nb_threads) {
          int nb_cpus = av_cpu_count();
 -        av_log(c->graph, AV_LOG_DEBUG, "Detected %d logical cores.\n", nb_cpus);
          // use number of cores + 1 as thread count if there is more than one
          if (nb_cpus > 1)
              nb_threads = nb_cpus + 1;
@@@ -208,7 -207,7 +208,7 @@@ int ff_graph_thread_init(AVFilterGraph 
      if (!graph->internal->thread)
          return AVERROR(ENOMEM);
  
-     ret = avfilter_thread_init(graph->internal->thread, graph->nb_threads);
+     ret = thread_init_internal(graph->internal->thread, graph->nb_threads);
      if (ret <= 1) {
          av_freep(&graph->internal->thread);
          graph->thread_type = 0;