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.

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

Non-existant CUDA device '12(Or 11)' specified in -d option #90

Open
Cerunnir opened this issue Dec 16, 2017 · 5 comments
Open

Non-existant CUDA device '12(Or 11)' specified in -d option #90

Cerunnir opened this issue Dec 16, 2017 · 5 comments
Labels

Comments

@Cerunnir
Copy link

I have an issue controlling what cards CCminer will use above number 9. Im able to use -d 1,2,3,,,9.

But when I get to 10, it just runs GPU 0.

When I try -d 11 or 12 I get this error:

Non-existant CUDA device '12' specified in -d option

If I just run it without -d at all, it detect all 13 nvidia GPU's and run fine. (Problem with 8+ of the same brand GPU is fixed in windows 10).

To me it seems like CCminer can use all my GPU's fine, but the -d command does not support numbers above 9.

Based on pooler cpuminer 2.3.2 and the tpruvot@github fork
CUDA support by Christian Buchner, Christian H. and DJM34
Includes optimizations implemented by sp-hash, klaust, tpruvot and tsiv.

[2017-12-16 22:57:29] NVML GPU monitoring enabled.
[2017-12-16 22:57:29] Non-existant CUDA device '12' specified in -d option
[2017-12-16 22:57:29] stopping 0 threads
[2017-12-16 22:57:29] resetting GPUs

@KlausT KlausT added the bug label Dec 17, 2017
@KlausT
Copy link
Owner

KlausT commented Dec 17, 2017

ok, I think I have found the bug

@Cerunnir
Copy link
Author

Hello, thanks for looking into it. It does seem to apply to every ccminer out there that ive tested.

Any ways to spread this to the other developers without individually telling them in their github?

@KlausT
Copy link
Owner

KlausT commented Dec 17, 2017

The closest thing would be be the bitcointalk forum.
Here for example:
https://bitcointalk.org/index.php?topic=770064.0

@Cerunnir
Copy link
Author

Cerunnir commented Dec 17, 2017 via email

@KlausT
Copy link
Owner

KlausT commented Dec 17, 2017

The old code was checking the length of the numbers.
If it was longer than one character then you got an error.
The exact code change you can see here:
caf27b4
Now the code isn't checking the length anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants