Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-11-13 Thread Muhammad Faiz
On Sun, Nov 12, 2017 at 8:28 PM, Muhammad Faiz wrote: > On Wed, Nov 1, 2017 at 12:37 AM, Muhammad Faiz wrote: >> On Mon, Oct 30, 2017 at 8:07 PM, Michael Niedermayer >> wrote: >>> On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote: On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-11-12 Thread Muhammad Faiz
On Wed, Nov 1, 2017 at 12:37 AM, Muhammad Faiz wrote: > On Mon, Oct 30, 2017 at 8:07 PM, Michael Niedermayer > wrote: >> On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote: >>> On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: >>> > Fix tsan warnings. >>> > >>> > Signed-off-by: Mu

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-31 Thread Muhammad Faiz
On Mon, Oct 30, 2017 at 8:07 PM, Michael Niedermayer wrote: > On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote: >> On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: >> > Fix tsan warnings. >> > >> > Signed-off-by: Muhammad Faiz >> > --- >> > libavutil/crc.c | 49 +++

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-30 Thread Michael Niedermayer
On Mon, Oct 30, 2017 at 02:14:35PM +0700, Muhammad Faiz wrote: > On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: > > Fix tsan warnings. > > > > Signed-off-by: Muhammad Faiz > > --- > > libavutil/crc.c | 49 + > > 1 file changed, 29 insertions

Re: [FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-30 Thread Muhammad Faiz
On Tue, Oct 24, 2017 at 4:31 PM, Muhammad Faiz wrote: > Fix tsan warnings. > > Signed-off-by: Muhammad Faiz > --- > libavutil/crc.c | 49 + > 1 file changed, 29 insertions(+), 20 deletions(-) Ping. My recent benchmark: old/new: 296320/ 340400 d

[FFmpeg-devel] [PATCH] avutil/crc: use ff_thread_once at av_crc_get_table

2017-10-24 Thread Muhammad Faiz
Fix tsan warnings. Signed-off-by: Muhammad Faiz --- libavutil/crc.c | 49 + 1 file changed, 29 insertions(+), 20 deletions(-) diff --git a/libavutil/crc.c b/libavutil/crc.c index 495732b163..d44550c9c0 100644 --- a/libavutil/crc.c +++ b/libavutil/