Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mesa: add vulkan-drivers=virtio #23090

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

john-peterson
Copy link

virgl_test_server --venus &
VN_DEBUG=all vkcube

virgl_test_server --venus &
VN_DEBUG=all vkcube
@licy183
Copy link
Member

licy183 commented Feb 2, 2025

I have no objection to ship such a driver, but please keep in mind that this will not be useful as it is expected.

Please split this vulkan driver in a seperate mesa-vulkan-icd-virtio.subpackage.sh as other vulkan driver does, and bump revision

@xNul
Copy link

xNul commented Feb 2, 2025

I would also like to lend my support for this. Will the virglrenderer and virglrenderer-android packages also have to be updated since they provide virgl_test_server and virgl_test_server_android?

@xNul
Copy link

xNul commented Feb 2, 2025

xMeM@da44e3b and xMeM@9bdcceb may be useful to reference

@twaik
Copy link
Member

twaik commented Feb 2, 2025

virgl_test_server_android does not have codepath to wrap android's libvulkan.

@john-peterson
Copy link
Author

done

…gle bionic packet has this patch

use BIONIC instead of ANDROID i still have no idea what the point is with this remake i wish all my drivers had a GNU libc version i can't even load them from glibc is it possible have you been able to load the Vulkan driver from glibc all drivers are bionic

try to focus on the important adding virpipe to this packet

only glibc has virpipe GALLIUM_DRIVER=virpipe has no effect here sigh
@licy183
Copy link
Member

licy183 commented Feb 2, 2025

libbthread doesn't work on higher Android. See #23092 (comment). Please revert it to the signal-based pthread_cancel workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants