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

[Ronald S. Bultje]

Hi,

On Tue, Sep 12, 2017 at 3:18 PM, Clément Bœsch <u at pkh.me> wrote:

> 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


TY, pushed.

Ronald