Bug 96350

Summary:	vaGetSurfaceBufferWl() is not implemented at VAAPI Gallium state tracker
Product:	Mesa	Reporter:	Egor Zaharov <nexfwall>
Component:	Drivers/Gallium/r600	Assignee:	Default DRI bug account <dri-devel>
Status:	RESOLVED MOVED	QA Contact:	Default DRI bug account <dri-devel>
Severity:	normal
Priority:	medium	CC:	leho, pachoramos1
Version:	git
Hardware:	x86-64 (AMD64)
OS:	Linux (All)
See Also:	https://bugzilla.gnome.org/show_bug.cgi?id=766954
Whiteboard:
i915 platform:		i915 features:

Description Egor Zaharov 2016-06-03 01:12:32 UTC

vaGetSurfaceBufferWl() is not implemented at VAAPI Gallium state tracker, which prevents it's usage under Wayland.

But, after the DRI3 patches for hardware video decoding landed, NOT_IMPLEMENTED return was removed, making the false sense what this will work under Wayland as expected[1]. But it's still don't.

See the bug referenced under "See Also".

Also, I don't know actually, under which component should I file this bug. So redirect me if I had mistaken.

[1]: https://cgit.freedesktop.org/mesa/mesa/tree/src/gallium/state_trackers/va/context.c#n129

Comment 1 leoxsliu 2016-06-07 13:37:24 UTC

First, the change is nothing to do with DRI3.
Second, the change removing "NOT_IMPLEMENTED" is wayland display should be opened with DRM code path, and it's already implemented. the code here is not general switch to turn on the wayland support on vaapi, it's just one of the steps to complete that support and which has been implemented. Sorry for the confusion.

Comment 2 Michel Dänzer 2016-06-08 01:21:00 UTC

Egor, does reverting commit 2f90d11d ("st/va: use drm render node for wayland display type") make totem work on Wayland?

Comment 3 Egor Zaharov 2016-06-09 14:33:38 UTC

(In reply to Michel Dänzer from comment #2)
> Egor, does reverting commit 2f90d11d ("st/va: use drm render node for
> wayland display type") make totem work on Wayland?

No. This makes it just crash with Segfault.

Comment 4 leoxsliu 2016-06-09 19:43:13 UTC

I found an issue with Totem player with DRI3 enabled, which is about glx of gstreamer-vaapi, it renews temporary pixmap each frame. I got a fix and have it sent to mesa-dev lists. But this is glx related, not sure if it will helps your case.

Comment 5 Apostolos B. 2016-09-07 12:47:54 UTC

This affect gstreamer playback in wayland (weston).

gst-play-1.0 --videosink vaapisink some_file.mp4 fails because vaGetSurfaceBufferWl is not implemented.

Comment 6 GitLab Migration User 2019-09-18 19:21:44 UTC

-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/mesa/mesa/issues/587.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.