Skip to content

Added option to select GPU to CUDA benchmark.#56

Open
mattkjames7 wants to merge 2 commits intoekondis:masterfrom
mattkjames7:master
Open

Added option to select GPU to CUDA benchmark.#56
mattkjames7 wants to merge 2 commits intoekondis:masterfrom
mattkjames7:master

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