Skip to content

avcodec: lock less

Thomas Guillem requested to merge tguillem/vlc:va-locked into master

Don't protect va with a lock. Indeed, it doesn't need any protection as it is created from ffmpeg_GetFormat(). Ffmpeg already take care of the synchronisation of all threads when get_format() need to be called.

Tested with tsan on ffmpeg 4 5 6 master.

⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⠿⠿⠿⣿⣿⣿
⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⠏⠀⠀⠀⠀⠀⠻⣿
⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡆⠀⠀⣤⣤⣤⣶⣿
⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⡇⠀⠀⢹⣿⣿⣿⣿
⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣧⣀⠀⢸⣿⣿⣿⣿
⣿⣉⣥⣤⣌⠻⣿⣿⡿⠋⣉⣤⣬⡉⠛⢿⣿⣿⣿⠟⢉⣤⣌⡙⠻⢾⣿⣿⣿⣿
⣿⣿⣿⣿⣿⣦⣈⣉⣠⣾⣿⣿⣿⣿⣷⣦⣈⣉⣁⣴⣿⣿⣿⡟⠲⣤⣉⣉⣁⣿
⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣇⠀⠀⣿⣿⣿⣿
⣿⣿⣿⣿⣿⣿⡿⠟⠛⠉⠉⠀⠀⠀⠀⠀⠀⠀⠈⠉⠙⠛⠻⠟⠀⠀⣿⣿⣿⣿
⣿⡟⠛⠛⠋⠁⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣿⣿⣿⣿
⣿⣿⣷⣄⠀⢀⣠⡄⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⣀⠀⠀⣰⣿⣿⣿⣿
⣿⣿⣿⣿⣷⡿⠋⠀⠀⣿⠀⠀⠀⠀⠀⠀⠀⠀⣴⠃⠀⠀⢿⣤⣾⣿⣿⣿⣿⣿
⣿⣿⣿⣿⠏⠀⠀⠀⢰⡿⢶⣤⣤⣤⣤⣤⣤⣾⠃⠀⠀⠀⣼⠇⢹⣿⣿⣿⣿⣿
⣿⣿⣿⣿⠀⠀⣀⣴⡟⠁⣠⣿⣿⣿⣿⣿⣿⣿⠀⠀⢀⣴⡟⢀⣿⣿⣿⣿⣿⣿
⣿⣿⣿⣿⣶⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⣦⣴⣿⣿⣿⣿⣿⣿⣿⣿⣿⣿⠀
Edited by Thomas Guillem

Merge request reports

Loading