Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread Michael Niedermayer
On Sat, Nov 25, 2017 at 11:05:22PM -0300, James Almer wrote: > On 11/25/2017 10:56 PM, Michael Niedermayer wrote: > > On Sat, Nov 25, 2017 at 05:01:55PM +, Rostislav Pehlivanov wrote: > >> Signed-off-by: Rostislav Pehlivanov > >> --- > >> libavcodec/utils.c | 9 +

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread Rostislav Pehlivanov
On 26 November 2017 at 01:56, Michael Niedermayer wrote: > On Sat, Nov 25, 2017 at 05:01:55PM +, Rostislav Pehlivanov wrote: > > Signed-off-by: Rostislav Pehlivanov > > --- > > libavcodec/utils.c | 9 + > > 1 file changed, 5

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread James Almer
On 11/25/2017 10:56 PM, Michael Niedermayer wrote: > On Sat, Nov 25, 2017 at 05:01:55PM +, Rostislav Pehlivanov wrote: >> Signed-off-by: Rostislav Pehlivanov >> --- >> libavcodec/utils.c | 9 + >> 1 file changed, 5 insertions(+), 4 deletions(-) > > LGTM > > The

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread Michael Niedermayer
On Sat, Nov 25, 2017 at 05:01:55PM +, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/utils.c | 9 + > 1 file changed, 5 insertions(+), 4 deletions(-) LGTM The whole lock manager should possibly be simplified using some more

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread Rostislav Pehlivanov
On 25 November 2017 at 18:37, James Almer wrote: > On 11/25/2017 2:01 PM, Rostislav Pehlivanov wrote: > > Signed-off-by: Rostislav Pehlivanov > > --- > > libavcodec/utils.c | 9 + > > 1 file changed, 5 insertions(+), 4 deletions(-) > > > > diff

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread James Almer
On 11/25/2017 2:01 PM, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > libavcodec/utils.c | 9 + > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index e50de6e89b..3a0f3c11f5

[FFmpeg-devel] [PATCH v3 1/3] lavc/utils.c: use C11 atomics for entangled thread handling

2017-11-25 Thread Rostislav Pehlivanov
Signed-off-by: Rostislav Pehlivanov --- libavcodec/utils.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index e50de6e89b..3a0f3c11f5 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -56,6