On Wed, Nov 23, 2016 at 11:21:56AM -0800, Wan-Teh Chang wrote:
> Remove the |checked| variable because the invalid value of -1 for
> |flags| can be used to indicate the same condition. Also rename |flags|
> to |cpu_flags| because there are a local variable and a function
> parameter named |flags| in the same file.
> 
> Add a test program libavutil/tests/cpu_init.c to check whether the
> one-time initialization in av_get_cpu_flags() has data races.
> 
> Co-author: Dmitry Vyukov of Google
> 
> Signed-off-by: Wan-Teh Chang <w...@google.com>
> ---
>  libavutil/Makefile         |  1 +
>  libavutil/cpu.c            | 40 ++++++++++++++------------
>  libavutil/tests/cpu_init.c | 72 
> ++++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 94 insertions(+), 19 deletions(-)
>  create mode 100644 libavutil/tests/cpu_init.c

Patch split and applied

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The bravest are surely those who have the clearest vision
of what is before them, glory and danger alike, and yet
notwithstanding go out to meet it. -- Thucydides

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to