[vlc-devel] [PATCH 5/7] vout/control: keep the lock instead of setting a held flag
remi at remlab.net
Fri Aug 14 17:30:59 CEST 2020
Le perjantaina 14. elokuuta 2020, 12.25.43 EEST Steve Lhomme a écrit :
> On 2020-08-14 11:02, Thomas Guillem wrote:
> > Why is it not needed anymore?
> I don't know why it was needed before. Maybe I'm missing something.
> > vout_control_Push() will now have to wait that controls are processed.
> I don't see how it's different from now.
> It needs the lock to push a command.
As Thomas already pointed out, the difference is that a control request needs
to wait for the control queue to be unheld by vout_control_Release() before it
can push a control request.
For filter changes, the correct solution is ostensibly to take the out of the
vout control queue, and process them synchronously with the filter lock. They
won't need the display lock.
Which gets back to my earlier point: it's about time to get rid of the vout
control queue for good. I fear that you're wasting your time trying to
simplify a goner.
More information about the vlc-devel