mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-12-23 12:43:46 +02:00
2170017a1c
Make the one-time initialization in av_get_cpu_flags() thread-safe. The static variables |flags|, |cpuflags_mask|, and |checked| in libavutil/cpu.c are read and written using normal load and store operations. These are considered as data races. The fix is to use atomic load and store operations. Remove the |checked| variable because the invalid value of -1 for |flags| can be used to indicate the same condition. Rename |flags| to |cpu_flags| and move it to file scope. The fix can be verified by running the libavutil/tests/cpu_init.c test program under ThreadSanitizer: ./configure --toolchain=clang-tsan make libavutil/tests/cpu_init libavutil/tests/cpu_init There should be no warnings from ThreadSanitizer. Co-author: Dmitry Vyukov of Google, who suggested the data race fix. Signed-off-by: Wan-Teh Chang <wtc@google.com> |
||
---|---|---|
.. | ||
checkasm | ||
fate | ||
filtergraphs | ||
ref | ||
.gitignore | ||
audiogen.c | ||
base64.c | ||
fate-run.sh | ||
fate.sh | ||
lavf-regression.sh | ||
Makefile | ||
md5.sh | ||
reference.pnm | ||
regression-funcs.sh | ||
rotozoom.c | ||
tiny_psnr.c | ||
utils.c | ||
videogen.c |