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.

Sign up for GitHub

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

Jump to bottom

-lcuda needs to be removed for cuda-11.2 #4085

Closed
RudolfWeeber opened this issue Jan 7, 2021 · 1 comment · Fixed by #4095
Closed

-lcuda needs to be removed for cuda-11.2 #4085

RudolfWeeber opened this issue Jan 7, 2021 · 1 comment · Fixed by #4095
Assignees
@jngrad

Comments

@RudolfWeeber
Copy link
Contributor

At least in the installation done by spack, libcuda is not present, there is just libcudart.
This needs to be changed in cmake/FindCUDACompilerNVCC.cmake.
Don't know if -lcuda is needed for older cuda versions.
@mkuron
Copy link
Member

mkuron commented Jan 21, 2021

libcuda.so is part of the driver, not part of the CUDA SDK. The driver is not installed on the headnode of our cluster because it does not have a GPU, so there is no libcuda.so. There is .../lib64/stubs/libcuda.so, which might be sufficient to link to. Generally, libcudart provides all the cuda[A-Z][A-Za-z]+ functions and libcuda provices all the cu[A-Z][A-Za-z]+ functions, so software like Espresso should not need to link to libcuda.

@jngrad jngrad self-assigned this Jan 22, 2021
@jngrad jngrad mentioned this issue Jan 22, 2021
Merged
@kodiakhq kodiakhq bot closed this as completed in #4095 Jan 29, 2021
kodiakhq bot added a commit that referenced this issue Jan 29, 2021
@kodiakhq
CUDA exception handling (#4095)
e4d6ecc 
Description of changes:
- convert CUDA error codes into runtime errors using the message associated to the error code
- properly handle CUDA errors by halting the flow of the program instead of ignoring them (Barnes-Hut, LB GPU)
- remove superfluous CUDA global variables (partial fix for #2628)
- restore `cuda_gather_gpus()` functionality and convert the `device_list()` getter to a regular function (API change)
- remove unused `libcuda` dependency (fixes #4085)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jngrad jngrad

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CUDA exception handling jngrad/espresso
3 participants
@RudolfWeeber @mkuron @jngrad