Merge commit '8385ba53f115401a67a4748c0d107769ebfb2941'
authorClément Bœsch <u@pkh.me>
Wed, 22 Mar 2017 17:01:51 +0000 (18:01 +0100)
committerClément Bœsch <u@pkh.me>
Wed, 22 Mar 2017 17:01:51 +0000 (18:01 +0100)
* commit '8385ba53f115401a67a4748c0d107769ebfb2941':
  mmaldec: convert to stdatomic

Merged-by: Clément Bœsch <u@pkh.me>
1  2 
libavcodec/mmaldec.c

diff --combined libavcodec/mmaldec.c
@@@ -2,20 -2,20 +2,20 @@@
   * MMAL Video Decoder
   * Copyright (c) 2015 Rodger Combs
   *
 - * 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
   */
  
  #include <interface/mmal/util/mmal_util_params.h>
  #include <interface/mmal/util/mmal_default_components.h>
  #include <interface/mmal/vc/mmal_vc_api.h>
+ #include <stdatomic.h>
  
  #include "avcodec.h"
  #include "internal.h"
- #include "libavutil/atomic.h"
  #include "libavutil/avassert.h"
  #include "libavutil/buffer.h"
  #include "libavutil/common.h"
@@@ -55,7 -55,7 +55,7 @@@ typedef struct FFBufferEntry 
  // refcounting for AVFrames, we can free the MMAL_POOL_T only after all AVFrames
  // have been unreferenced.
  typedef struct FFPoolRef {
-     volatile int refcount;
+     atomic_int refcount;
      MMAL_POOL_T *pool;
  } FFPoolRef;
  
@@@ -83,13 -83,11 +83,13 @@@ typedef struct MMALDecodeContext 
      FFBufferEntry *waiting_buffers, *waiting_buffers_tail;
  
      int64_t packets_sent;
-     volatile int packets_buffered;
+     atomic_int packets_buffered;
      int64_t frames_output;
      int eos_received;
      int eos_sent;
      int extradata_sent;
 +    int interlaced_frame;
 +    int top_field_first;
  } MMALDecodeContext;
  
  // Assume decoder is guaranteed to produce output after at least this many
@@@ -98,7 -96,8 +98,8 @@@
  
  static void ffmmal_poolref_unref(FFPoolRef *ref)
  {
-     if (ref && avpriv_atomic_int_add_and_fetch(&ref->refcount, -1) == 0) {
+     if (ref &&
+         atomic_fetch_add_explicit(&ref->refcount, -1, memory_order_acq_rel) == 1) {
          mmal_pool_destroy(ref->pool);
          av_free(ref);
      }
@@@ -134,7 -133,7 +135,7 @@@ static int ffmmal_set_ref(AVFrame *fram
          return AVERROR(ENOMEM);
      }
  
-     avpriv_atomic_int_add_and_fetch(&ref->pool->refcount, 1);
+     atomic_fetch_add_explicit(&ref->pool->refcount, 1, memory_order_relaxed);
      mmal_buffer_header_acquire(buffer);
  
      frame->format = AV_PIX_FMT_MMAL;
@@@ -165,14 -164,14 +166,14 @@@ static void ffmmal_stop_decoder(AVCodec
          ctx->waiting_buffers = buffer->next;
  
          if (buffer->flags & MMAL_BUFFER_HEADER_FLAG_FRAME_END)
-             avpriv_atomic_int_add_and_fetch(&ctx->packets_buffered, -1);
+             atomic_fetch_add(&ctx->packets_buffered, -1);
  
          av_buffer_unref(&buffer->ref);
          av_free(buffer);
      }
      ctx->waiting_buffers_tail = NULL;
  
-     av_assert0(avpriv_atomic_int_get(&ctx->packets_buffered) == 0);
+     av_assert0(atomic_load(&ctx->packets_buffered) == 0);
  
      ctx->frames_output = ctx->eos_received = ctx->eos_sent = ctx->packets_sent = ctx->extradata_sent = 0;
  }
@@@ -204,7 -203,7 +205,7 @@@ static void input_callback(MMAL_PORT_T 
          FFBufferEntry *entry = buffer->user_data;
          av_buffer_unref(&entry->ref);
          if (entry->flags & MMAL_BUFFER_HEADER_FLAG_FRAME_END)
-             avpriv_atomic_int_add_and_fetch(&ctx->packets_buffered, -1);
+             atomic_fetch_add(&ctx->packets_buffered, -1);
          av_free(entry);
      }
      mmal_buffer_header_release(buffer);
@@@ -276,14 -275,13 +277,14 @@@ static int ffmal_update_format(AVCodecC
      int ret = 0;
      MMAL_COMPONENT_T *decoder = ctx->decoder;
      MMAL_ES_FORMAT_T *format_out = decoder->output[0]->format;
 +    MMAL_PARAMETER_VIDEO_INTERLACE_TYPE_T interlace_type;
  
      ffmmal_poolref_unref(ctx->pool_out);
      if (!(ctx->pool_out = av_mallocz(sizeof(*ctx->pool_out)))) {
          ret = AVERROR(ENOMEM);
          goto fail;
      }
-     ctx->pool_out->refcount = 1;
+     atomic_store(&ctx->pool_out->refcount, 1);
  
      if (!format_out)
          goto fail;
      if ((status = mmal_port_format_commit(decoder->output[0])))
          goto fail;
  
 +    interlace_type.hdr.id = MMAL_PARAMETER_VIDEO_INTERLACE_TYPE;
 +    interlace_type.hdr.size = sizeof(MMAL_PARAMETER_VIDEO_INTERLACE_TYPE_T);
 +    status = mmal_port_parameter_get(decoder->output[0], &interlace_type.hdr);
 +    if (status != MMAL_SUCCESS) {
 +        av_log(avctx, AV_LOG_ERROR, "Cannot read MMAL interlace information!\n");
 +    } else {
 +        ctx->interlaced_frame = (interlace_type.eMode != MMAL_InterlaceProgressive);
 +        ctx->top_field_first = (interlace_type.eMode == MMAL_InterlaceFieldsInterleavedUpperFirst);
 +    }
 +
      if ((ret = ff_set_dimensions(avctx, format_out->es->video.crop.x + format_out->es->video.crop.width,
                                          format_out->es->video.crop.y + format_out->es->video.crop.height)) < 0)
          goto fail;
          avctx->sample_aspect_ratio.num = format_out->es->video.par.num;
          avctx->sample_aspect_ratio.den = format_out->es->video.par.den;
      }
 +    if (format_out->es->video.frame_rate.num && format_out->es->video.frame_rate.den) {
 +        avctx->framerate.num = format_out->es->video.frame_rate.num;
 +        avctx->framerate.den = format_out->es->video.frame_rate.den;
 +    }
  
      avctx->colorspace = ffmmal_csp_to_av_csp(format_out->es->video.color_space);
  
@@@ -377,9 -361,6 +378,9 @@@ static av_cold int ffmmal_init_decoder(
      case AV_CODEC_ID_MPEG2VIDEO:
          format_in->encoding = MMAL_ENCODING_MP2V;
          break;
 +    case AV_CODEC_ID_MPEG4:
 +        format_in->encoding = MMAL_ENCODING_MP4V;
 +        break;
      case AV_CODEC_ID_VC1:
          format_in->encoding = MMAL_ENCODING_WVC1;
          break;
@@@ -542,7 -523,7 +543,7 @@@ static int ffmmal_add_packet(AVCodecCon
  
          if (!size) {
              buffer->flags |= MMAL_BUFFER_HEADER_FLAG_FRAME_END;
-             avpriv_atomic_int_add_and_fetch(&ctx->packets_buffered, 1);
+             atomic_fetch_add(&ctx->packets_buffered, 1);
          }
  
          if (!buffer->length) {
@@@ -607,7 -588,7 +608,7 @@@ static int ffmmal_fill_input_port(AVCod
              mmal_buffer_header_release(mbuffer);
              av_buffer_unref(&buffer->ref);
              if (buffer->flags & MMAL_BUFFER_HEADER_FLAG_FRAME_END)
-                 avpriv_atomic_int_add_and_fetch(&ctx->packets_buffered, -1);
+                 atomic_fetch_add(&ctx->packets_buffered, -1);
              av_free(buffer);
          }
  
@@@ -626,9 -607,6 +627,9 @@@ static int ffmal_copy_frame(AVCodecCont
      MMALDecodeContext *ctx = avctx->priv_data;
      int ret = 0;
  
 +    frame->interlaced_frame = ctx->interlaced_frame;
 +    frame->top_field_first = ctx->top_field_first;
 +
      if (avctx->pix_fmt == AV_PIX_FMT_MMAL) {
          if (!ctx->pool_out)
              return AVERROR_UNKNOWN; // format change code failed with OOM previously
@@@ -689,7 -667,7 +690,7 @@@ static int ffmmal_read_frame(AVCodecCon
          // excessive buffering.
          // We also wait if we sent eos, but didn't receive it yet (think of decoding
          // stream with a very low number of frames).
-         if (avpriv_atomic_int_get(&ctx->packets_buffered) > MAX_DELAYED_FRAMES ||
+         if (atomic_load(&ctx->packets_buffered) > MAX_DELAYED_FRAMES ||
              (ctx->packets_sent && ctx->eos_sent)) {
              // MMAL will ignore broken input packets, which means the frame we
              // expect here may never arrive. Dealing with this correctly is
@@@ -823,13 -801,6 +824,13 @@@ AVHWAccel ff_mpeg2_mmal_hwaccel = 
      .pix_fmt    = AV_PIX_FMT_MMAL,
  };
  
 +AVHWAccel ff_mpeg4_mmal_hwaccel = {
 +    .name       = "mpeg4_mmal",
 +    .type       = AVMEDIA_TYPE_VIDEO,
 +    .id         = AV_CODEC_ID_MPEG4,
 +    .pix_fmt    = AV_PIX_FMT_MMAL,
 +};
 +
  AVHWAccel ff_vc1_mmal_hwaccel = {
      .name       = "vc1_mmal",
      .type       = AVMEDIA_TYPE_VIDEO,
@@@ -872,5 -843,4 +873,5 @@@ static const AVOption options[]=
  
  FFMMAL_DEC(h264, AV_CODEC_ID_H264)
  FFMMAL_DEC(mpeg2, AV_CODEC_ID_MPEG2VIDEO)
 +FFMMAL_DEC(mpeg4, AV_CODEC_ID_MPEG4)
  FFMMAL_DEC(vc1, AV_CODEC_ID_VC1)