Re: [PATCH] ui/sdl2: Fix assumption of EGL presence at runtime

["Alex Bennée" <alex.bennee@linaro.org>]

Mohamed Mediouni <mohamed@unpredictable.fr> writes:

(Add Stefan to CC)

>> On 9. Apr 2026, at 14:08, Alex Bennée <alex.bennee@linaro.org> wrote:
>> 
>> Anthony Roberts <anthony.roberts@linaro.org> writes:
>> 
>>> The original commit had a section of code which worked on the assumption
>>> that if OpenGL was enabled at build, it was present on the end user machine,
>>> and calls could be made to it. This is not always the case (such as Windows
>>> on Arm devices).
>>> 
>>> This line should have also included a runtime check.
>>> 
>>> This commit moves the relevant line to inside a runtime check for OpenGL.
>>> 
>>> Fixes: 52053b7e0a0e ("ui/sdl2: Implement dpy dmabuf functions")
>>> Resolves: https://gitlab.com/qemu-project/qemu/-/work_items/3347
>>> Signed-off-by: Anthony Roberts <anthony.roberts@linaro.org>
>> 
>> Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
>> 
>> -- 
>> Alex Bennée
>> Virtualisation Tech Lead @ Linaro
>> 
>
> Hello,
>
> As a related note, modern QEMU is built assuming EGL which Windows does
> not ship as a system library - and doesn’t support a WGL-only configuration...
>
> For those distributing QEMU binaries on Windows, they should bring an
> EGL implementation along such as the one in mingw-w64-angleproject 
> or Mesa for Windows if they wish to support OpenGL functionality
> instead of having crashes.

Stefan handles the semi-official windows binaries. 

Generally our windows coverage is poor as most developers can only use
the mingw cross compilers (if they even check) and we have one Windows
x86_64 runner via Cirrus which doesn't have bare metal access to the
hypervisor.

It would be nice to have better testing coverage in CI but that would
require someone to fund some bare metal runners and volunteer to keep
them updated.

>
> Wonder what’s the best place to document that requirement…

docs/system/devices/virtio/virtio-gpu.rst

has a fairly in-depth breakdown of the various moving parts you need.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro