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

virgl-angle: add venus interface and on device build #23104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

john-peterson
Copy link

@john-peterson john-peterson commented Feb 4, 2025

dead lock or starvation

virgl-angle --venus --angle-null &
VN_DEBUG=all vkcube
MESA-VIRTIO: debug: stuck in ring seqno wait with iter at 4096

if you need GL you have to use the GNU build the bionic virpipe driver is missing. sorry I can't fix everything guys need some assistance

GALLIUM_DRIVER=virpipe $PREFIX/glibc/bin/glmark2

@john-peterson
Copy link
Author

john-peterson commented Feb 4, 2025

deleted

@john-peterson john-peterson force-pushed the vinkel branch 2 times, most recently from bddbd52 to c347c0d Compare February 4, 2025 23:04
@john-peterson john-peterson changed the title virgl_angle: add venus and on device build virgl-angle: add venus and on device build Feb 4, 2025
@john-peterson john-peterson changed the title virgl-angle: add venus and on device build virgl-angle: add venus interface and on device build Feb 4, 2025
@john-peterson john-peterson force-pushed the vinkel branch 5 times, most recently from 31f48f9 to 86a74aa Compare February 4, 2025 23:54
todo fix this dead lock

virgl-angle --venus --angle-null &
VN_DEBUG=all vkcube
MESA-VIRTIO: debug: stuck in ring seqno wait with iter at 4096
@hansm629
Copy link

hansm629 commented Feb 5, 2025

@john-peterson
You are preparing a great request.
Great.

Will this package be available on all GPUs regardless of GPU device?

# Move our virglrenderer binary to regular bin directory.
mv $_INSTALL_PREFIX/bin/virgl_test_server $TERMUX_PREFIX/bin/virgl_test_server_android
mkdir $TERMUX_PKG_MASSAGEDIR/$TERMUX_PREFIX/bin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not use TERMUX_PKG_MASSAGEDIR, unless you know what you are doing. I haven't found any need to use it.

ninja -C virglrenderer-build install -j $TERMUX_PKG_MAKE_PROCESSES

# i have to say everything twice as usual
patchelf --set-rpath $_INSTALL_PREFIX/lib $_INSTALL_PREFIX/bin/virgl_test_server
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

patchelf doesn't exist in the host builder image. This will break the normal build progress when cross-compling.

CXX=c++
CROSS=
# need android 30+
# vim $PREFIX/include/sys/mman.h -c /memfd_create
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

memfd_create should be patched to syscall(SYS_memfd_create, ...).


if $TERMUX_ON_DEVICE_BUILD; then
local _INSTALL_PREFIX=$TERMUX_PKG_MASSAGEDIR/$TERMUX_PREFIX/opt/$TERMUX_PKG_NAME
CC=clang
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to set CC and CXX. $CCTERMUX_HOST_PLATFORM-clang/clang++ also exist when on-device build.

@licy183
Copy link
Member

licy183 commented Feb 6, 2025

To be honest, I don't see any reason why this package should support on-device build....

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.

3 participants