[vlc-devel] Handling deadlock between display:Open() and vout_window_ReportSize()

Rémi Denis-Courmont remi at remlab.net
Fri Dec 13 18:06:35 CET 2019

Le perjantaina 13. joulukuuta 2019, 18.54.22 EET Alexandre Janniaux a écrit :
> On Fri, Dec 13, 2019 at 06:08:38PM +0200, Rémi Denis-Courmont wrote:
> > 	Hi,
> > 
> > There's an obvious lock inversion in that patch that will break all
> > platforms.
> For the inversion lock, the inversion is obvious in the
> current codebase, but is impossible if you enable with the
> right state because of the ordering of events and state, as
> far as I checked. It's not impossible that it's imperfect
> though, I would have submitted a patch if I were sure of
> what I was doing.

We use strict lock ordering in VLC like pretty much everywhere else. Anything 
else will confuse developers, reviewers, sanitizers and analyzers too much to 


More information about the vlc-devel mailing list