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
geekbench5: add compute benchmark support #89124
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you! 😻
Still works after the update in #94418. |
/marvin opt-in |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normally the nvidia
dependency should be made optional, but since the package is already unfree that is probably okay. Thank you!
ln -s ${ocl-icd}/lib/libOpenCL.so $out/lib/ | ||
ln -s ${ocl-icd}/lib/libOpenCL.so.1 $out/lib/ | ||
ln -s ${vulkan-loader}/lib/libvulkan.so $out/lib/ | ||
ln -s ${vulkan-loader}/lib/libvulkan.so.1 $out/lib/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only noticing this after merging: Do you specify version 1 on purpose, or should this maybe link all versions instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's what geekbench5 tried to open: #87990 (comment)
the opencl issue don't look related to a library missing #94982
do you think it could be an improvement?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is actually just one version, with different names.
ls -la ${ocl-icd}/lib/
-r-xr-xr-x 1 nobody nogroup 994 Jan 1 1970 libOpenCL.la
lrwxrwxrwx 1 nobody nogroup 18 Jan 1 1970 libOpenCL.so -> libOpenCL.so.1.0.0
lrwxrwxrwx 1 nobody nogroup 18 Jan 1 1970 libOpenCL.so.1 -> libOpenCL.so.1.0.0
-r-xr-xr-x 1 nobody nogroup 150576 Jan 1 1970 libOpenCL.so.1.0.0
dr-xr-xr-x 2 nobody nogroup 4096 Jan 1 1970 pkgconfig
ls -la ${vulkan-loader}/lib/
lrwxrwxrwx 1 nobody nogroup 14 Jan 1 1970 libvulkan.so -> libvulkan.so.1
lrwxrwxrwx 1 nobody nogroup 20 Jan 1 1970 libvulkan.so.1 -> libvulkan.so.1.2.131
-r-xr-xr-x 1 nobody nogroup 458168 Jan 1 1970 libvulkan.so.1.2.131
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the usual naming convention is that libsomething.so
will link to the latest version, libsomething.so.n
will link to version n
. So in this case, it might break if libvulkan
2 is released.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, i understand. but it will not improve the current situation if all files are copied.
i will add that later
Motivation for this change
I want to use the compute benchmark.
Fixes #87990
Devices are detected now:
OpenCL benchmark fails, but Vulkan works.
cc maintainer @michalrus
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)