-
Notifications
You must be signed in to change notification settings - Fork 32
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
Load NVML lib at runtime #12
Labels
Comments
Looks like rust-lang/rust-bindgen#1541 has been resolved and is incorporated into the 0.5.6 release of bindgen |
Wonderful! Thanks for the heads up. I'll give the new bindgen feature a poke tonight and see what using it looks like in practice. |
Merged
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now this crate can only be used as an optional compile-time dependency on Linux and / or Windows. This makes it impossible both to build a binary that can be distributed cross-platform and to build a binary that can decide at runtime whether or not it is possible to get GPU info.
This crate needs to be moved to a dlopen-style approach where symbols are obtained at runtime, something that is unfortunately going to be a lot of work. It'll hopefully become somewhat tractable if and when this bindgen issue is resolved.
If I have the time and motivation + bindgen or some other tool makes the process more streamlined, I will invest the time into re-working the crate.
(Note that I may also end up dropping a lot of the more niche API surface upon making this migration in order to focus my time on supporting the more commonly-used functionality.)
The text was updated successfully, but these errors were encountered: