[FFmpeg-devel] [PATCH] vp9: fix explicit memory order for report_progress.

Clément Bœsch u at pkh.me
Tue Sep 12 22:18:18 EEST 2017


On Tue, Sep 12, 2017 at 03:16:43PM -0400, Ronald S. Bultje wrote:
> ---
>  libavcodec/vp9.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c
> index 66ccb6c..6b5de19 100644
> --- a/libavcodec/vp9.c
> +++ b/libavcodec/vp9.c
> @@ -73,7 +73,7 @@ static int vp9_alloc_entries(AVCodecContext *avctx, int n) {
>  
>  static void vp9_report_tile_progress(VP9Context *s, int field, int n) {
>      pthread_mutex_lock(&s->progress_mutex);
> -    atomic_fetch_add_explicit(&s->entries[field], n, memory_order_relaxed);
> +    atomic_fetch_add_explicit(&s->entries[field], n, memory_order_release);
>      pthread_cond_signal(&s->progress_cond);
>      pthread_mutex_unlock(&s->progress_mutex);
>  }

tested, works, LGTM

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170912/e1db94d0/attachment.sig>


More information about the ffmpeg-devel mailing list