Skip to content

Conversation

@acardona
Copy link

... in Ubuntu 16.04 with CUDA 10.0.

All this commit does is remove code that sets a compatibility mode for CUDA versions earlier than 6050.

All tests pass with ctests.

in Ubuntu 16.04 with CUDA 10.0.

All this commit does is remove code that sets a compatibility mode
for CUDA versions earlier than 6050.
@psteinb
Copy link
Collaborator

psteinb commented Jan 17, 2019

@acardona thank you for the contribution. But why do you want to get rid of something that allows backward compatibility? I agree that CUDA6 and before is quite old, but still I wonder if we instead should try to make cuda10 happy and keep the backward compatibility. Think reproducible science. ;)

@acardona
Copy link
Author

acardona commented Jan 17, 2019 via email

@aschampion
Copy link

While this allows compilation with both CUDA 9.2 and 10.0 (CUDA >= 9.1 will not compile on master otherwise), the resulting how_cufft_works ctest fails for me in both cases. Ubuntu 16.04, Titan Xp, the default 410.79 drivers compiled against CUDA 10.0.

The test passes when compiled against CUDA 8.0 with this branch.

@Xqua
Copy link

Xqua commented Aug 30, 2019

The reason is that Nvidia has removed the function: cufftSetCompatibilityMode
So the compiler fails on this file because this function does not exist in CUDA 10. Nividia said they won't add it back.
The correct fix would be a Cmake fix with 2 files, one for cuda >10 and one for cuda <10.
But I've been removing those lines at every new install recently and was about to PR this exact fix ^^

Also this part:
image

Is not necessary, only the lines with cufftSetCompatibilityMode cause issues.

@tobetz
Copy link

tobetz commented Jun 19, 2021

Thanks for the fixes. It does compile for me,too. (Ubuntu 20.04, Cuda 10.1). But in my case the ctest function part that is finally testing the FFT now fails.
Did any of you checked the test?
Is the library running for you?

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.

5 participants