[vlc-devel] [PATCH 1/3] coreaudio: fix possible deadlock on pause
nitz.carola at googlemail.com
Sat Dec 9 16:41:28 CET 2017
Well it is and I can show you the callstack to show you how the unlogical
iOS system does exactly that :) Assuming that the calls are made just like
Thomas described it, are you okay with the patches ?
(It fixes unresponsive UI after starting a different App and users have
complained about that so I really would love to fix that)
If not could you provide constructive feedback on how to do this
On Sat 9. Dec 2017 at 15:58, Rémi Denis-Courmont <remi at remlab.net> wrote:
> Le lauantaina 9. joulukuuta 2017, 15.53.05 EET Thomas Guillem a écrit :
> > 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
> > > > is shut down) and will unstuck flush/play.
> > >
> > > And I don't see how that can work. The interface is not reentrant so
> > > implementation can´t be reentrant either.
> > The interface is not reentrant, but the implementation is with this
> > patch.
> That is not logically possible (except for degenerate implementation such
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel