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

Added option to select GPU to CUDA benchmark. #56

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mattkjames7
Copy link

I had a bit of fun with this nifty little utility, I had to make a small modification to allow me to benchmark multiple GPUs on the same system. It's a really minor change, just added an optional --gpu argument e.g.

./mixbench-cuda --gpu 2

It still defaults to GPU 0 if no argument is passed.

@ekondis
Copy link
Owner

ekondis commented Oct 24, 2024

Thank you for this.
Regarding the --gpu argument, I'm just a little hesitant from the fact that this diverges from the way the device is specified in the OpenCL & SYCL implementations. In the latter, the device index is passed directly without a named argument. I understand that using a named argument is a better practice but in this case I would prefer without it.

@mattkjames7
Copy link
Author

So you would prefer a call something more like this?

./mixbench-cuda 2

I'll make an edit when I get a chance!

@ekondis
Copy link
Owner

ekondis commented Oct 24, 2024

So you would prefer a call something more like this?

./mixbench-cuda 2

I'll make an edit when I get a chance!

Yes, exactly. To maintain a somewhat similar invocation pattern between all benchmarks.

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.

2 participants