Bug 486391

Summary: QMplay2 crashes when using Vaapi on Wayland
Product: [Plasma] kwin Reporter: Mike Lothian <mike>
Component: wayland-genericAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: crash CC: kde, nate, xaver.hugl
Priority: NOR    
Version First Reported In: master   
Target Milestone: ---   
Platform: Other   
OS: Linux   
URL: https://gitlab.freedesktop.org/mesa/mesa/-/issues/11097
Latest Commit: https://invent.kde.org/plasma/kwin/-/commit/0f4123951599cfdc99b53cde6911990158151293 Version Fixed/Implemented In: 6.1
Sentry Crash Report:
Attachments: WAYLAND_DEBUG=1

Description Mike Lothian 2024-05-01 10:14:56 UTC 
This is using Plasma 6.1 git, KF6 git, Mesa git and QMplay2 from git

QMPlay2 
libva info: VA-API version 1.21.0
libva info: Trying to open /usr/lib64/va/drivers/radeonsi_drv_video.so
libva info: Found init function __vaDriverInit_1_21
libva info: va_openDriver() returns 0
VA-API :: Initialized device: renderD129
wp_linux_drm_syncobj_surface_v1@107: error 4: explicit sync is used, but no acquire point is set
[01 May 2024 11:04:45.448] The Wayland connection experienced a fatal error: Protocol error


This appears to be triggered by the landing of https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28874

The error message about the no acquire point I think is from the much older https://invent.kde.org/plasma/kwin/-/merge_requests/4693

I'm raising a bug with the Mesa folk too, but I'm not 100% sure if this is a Mesa, Kwin or QMplay2 bug
Comment 1 David Edmundson 2024-05-01 14:59:58 UTC 
As you have a git master of everything, could you attach output of

WAYLAND_DEBUG=1  qplay2

It'll help confirm which side the bug is
Comment 2 Mike Lothian 2024-05-01 18:05:26 UTC 
I have a feeling its related to the vulkan-{header,loader,etc etc} being updated to 1.3.280
Comment 3 Mike Lothian 2024-05-01 18:05:51 UTC 
Created attachment 169082 [details]
WAYLAND_DEBUG=1
Comment 4 Zamundaaa 2024-05-01 21:04:33 UTC 
I had the bug id wrong at first, https://invent.kde.org/plasma/kwin/-/merge_requests/5683 fixes this
Comment 5 Mike Lothian 2024-05-02 00:53:59 UTC 
It does indeed, feel free to add my:

Tested-by: Mike Lothian <mike@fireburn.co.uk>
Comment 6 Zamundaaa 2024-05-21 23:57:50 UTC 
Git commit 0f4123951599cfdc99b53cde6911990158151293 by Xaver Hugl.
Committed on 21/05/2024 at 22:26.
Pushed by zamundaaa into branch 'master'.

waylandserver: set render backend for linuxdrmsyncobj directly

The code would previously never set the render backend, causing the use of explicit sync
to fail

M  +0    -4    src/platformsupport/scenes/opengl/abstract_egl_backend.cpp
M  +4    -0    src/wayland_server.cpp

https://invent.kde.org/plasma/kwin/-/commit/0f4123951599cfdc99b53cde6911990158151293