<div dir="ltr">Hi<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 27, 2016 at 3:56 PM, Marc-André Lureau <span dir="ltr"><<a href="mailto:mlureau@redhat.com" target="_blank">mlureau@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi<br>
<span class=""><br>
----- Original Message -----<br>
> When using GtkDrawingArea and EGL was not used, it was still initialized.<br>
> This produced warning messages on systems where EGL is not supported.<br>
<br>
</span>Not sure it's really a big problem since we require GL anyway.<br>
<br>
Which system doesn't support egl?<span class=""><br></span></blockquote><div><br></div><div>For example when running the client in virtual machine.</div><div>The point is, that we shouldn't initialize EGL, if we are not going to use it.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
><br>
> Move spice_egl_init from drawing_area_realize to gl_scanout.<br>
> Drawing_area_realize is no longer needed.<br>
> ---<br>
> Changes since v1:<br>
>  - changed from recursive call to if check<br>
>  - moved set_egl_enabled(display, true); after initialisation is finished<br>
> ---<br>
<br>
</span>Have you checked the behaviour on wayland?<br></blockquote><div><br></div><div>Yes, it's working correctly in wayland.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Because spice_egl_init() is already called in gl_area_realize in this case.<br></blockquote><div><br></div><div>Well spice_egl_init() sets context_ready to true, so it should never be called twice.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Furthermore, there is no guarantee that the widget is realized when spice_display_widget_gl_scanout() is called, which would results in other warnings/criticals messages.<span class=""><br></span></blockquote><div><br></div><div>This could potentionally be a problem. Does spice_egl_realize_display() require the widget to be fully realized before the call?</div></div><br>Also wouldn't it be better, to move all of the EGL code to GtkGLArea, instead of GtkDrawArea even for X11?<br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Lukas Venhoda</div>
</div></div>