[vlc-devel] [PATCH 1/3] coreaudio: fix possible deadlock on pause

Thomas Guillem thomas at gllm.fr
Sat Dec 9 15:53:05 CET 2017



On Sat, Dec 9, 2017, at 15:48, Rémi Denis-Courmont wrote:
> Le lauantaina 9. joulukuuta 2017, 15.28.43 EET Thomas Guillem a écrit :
> > Here, ca_Pause is the pause callback for macOS and iOS audio outputs.
> > With the patch 2/2, this function can be called from a random thread
> > when the audio session is disabled (and when the audio render callback
> > is shut down) and will unstuck flush/play.
> 
> And I don't see how that can work. The interface is not reentrant so its 
> implementation can´t be reentrant either.

The interface is not reentrant, but the implementation is with this
patch.

If you prefer, we can add a new function (like ca_SetAlive()) called
from the notification thread and leave the ca_Pause not reentrant but
it's all the same to me, just an implementation detail.

> 
> -- 
> 雷米‧德尼-库尔蒙
> https://www.remlab.net/
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list