clvk stopped working on Android after adding buffer device address support

[truboxl]

~ $ OCL_ICD_DEBUG=15 clinfo
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:708): __initClIcd: OPENCL_VENDOR_PATH unset or empty. Using hard-coded path '/data/data/com.termux/files/usr/etc/OpenCL/vendors'
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:714): __initClIcd: OCL_ICD_VENDORS empty or not defined, using vendors path '/data/data/com.termux/files/usr/etc/OpenCL/vendors'
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:733): __initClIcd: Reading icd list from '/data/data/com.termux/files/usr/etc/OpenCL/vendors'
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:201): _find_num_icds: return: 1/0x1
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:232): _open_driver: Considering file '/data/data/com.termux/files/usr/etc/OpenCL/vendors/clvk.icd'
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:206): _load_icd: Loading ICD '/data/data/com.termux/files/usr/lib/clvk/libOpenCL.so'
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:213): _load_icd: error while dlopening the IDL: 'dlopen failed: cannot locate symbol "vkGetBufferDeviceAddress" referenced by "/data/data/com.termux/files/usr-aarch64/lib/clvk/libOpenCL.so"...',
  => skipping ICD
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:264): _open_driver: return: 0/0x0
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:277): _open_drivers: return: 0/0x0
ocl-icd(/home/builder/.termux-build/ocl-icd/src/ocl_icd_loader.c:1021): clGetPlatformIDs: return: -1001/0xfffffffffffffc17
Number of platforms                               0

The latest commit 6ccb8a0 stopped working on my Android 11 device.
I am not sure whether its because of building clvk myself differently from the guide.
But using the older commit does work and I will temporarily revert my build to that commit.
I will checkout the GitHub Actions artifacts later to see if it can reproduce or not. (that thing is huge! ~2GB)

[kpet]

@truboxl Thanks for the report! I didnt know you were packaging clvk for Android, that's great to see :). I've opened #496 which should fix the issue you're seeing. This issue will automatically close once the PR is merged as I'm very confident it fixes the issue described here but if that's not the case feel free to reopen the issue. Also, if there's anything we can do to help you package clvk on Android or provide a better user experience, don't hesitate to create further issues. Thanks!

[truboxl]

Hi @kpet
I think you will find https://android.googlesource.com/platform/frameworks/native/+/master/vulkan/libvulkan/libvulkan.map.txt helpful on understanding the extent to support Android.

    vkGetBufferDeviceAddress; # introduced=31

In this case API 31 translates to Android 12. Do you allow PR that workaround old Android version? Usually older version dont have the proper symbols in their system libvulkan.so.

It maybe helpful to have at least some form of release version. This will be a great help for downstream distro to package clvk. Currently I am using some cryptic versioning while packaging clvk, eg: 0.0.20230218.101959gfb7b75ce 😂

[kpet]

@truboxl

Do you allow PR that workaround old Android version?

Yes, definitely open to making changes so clvk can work with more Android versions. Feel free to create issues or open PRs. I can't promise someone would pick up the issues but it's always good to know what would be useful.

It maybe helpful to have at least some form of release version.

Yes, I agree we need to do better there :). I've created #499. Thoughts welcome!