<html><head></head><body>There is only one lock involved here, the window lock. It'll obviously lock itself dead if you acquire from a window callback. Like many VLC locks (display lock, aout lock come to mind.)<br><br><div class="gmail_quote">Le 8 novembre 2019 23:13:30 GMT+09:00, Alexandre Janniaux <ajanni@videolabs.io> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">What is the very obvious dead lock?<br><br>A situation with two locks doesn't seem obvious from an<br>external view.<br><br>Regards,<br>--<br>Alexandre Janniaux<br>Videolabs<br><br>On Fri, Nov 08, 2019 at 10:49:22PM +0900, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> This leads to very obvious dead lock.<br><br> Le 8 novembre 2019 19:56:17 GMT+09:00, Thomas Guillem <thomas@gllm.fr> a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">This fixes the window size that was ignored when the size was updated<br>from the<br>open callback of the window plugin.<br><br>Fixes #22674<hr> src/video_output/video_output.c | 6 ++++++<br> 1 file changed, 6 insertions(+)<br><br>diff --git a/src/video_output/video_output.c<br>b/src/video_output/video_output.c<br>index db48538fd9c..ff36250d23d 100644<br>--- a/src/video_output/video_output.c<br>+++ b/src/video_output/video_output.c<br>@@ -424,7 +424,13 @@ void vout_ChangeDisplaySize(vout_thread_t *vout,<br>     assert(!sys->dummy);<br><br>     /* DO NOT call this outside the vout window callbacks */<br>+    vlc_mutex_lock(&sys->window_lock);<br>+    sys->display_cfg.display.width = width;<br>+    sys->display_cfg.display.height = height;<br>+<br>     vlc_mutex_lock(&sys->display_lock);<br>+    vlc_mutex_unlock(&sys->window_lock);<br>+<br>     if (sys->display != NULL)<br>         vout_display_SetSize(sys->display, width, height);<br>     vlc_mutex_unlock(&sys->display_lock);<br>--<br>2.20.1<hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote>--<br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.<br></blockquote><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>