Skip to content

Feature/support CUDA <12.3 compatibility - #209

Closed
giostrives wants to merge 1 commit into
Niko1221:mainfrom
giostrives:feature/support_2080ti
Closed

giostrives wants to merge 1 commit into
Niko1221:mainfrom
giostrives:feature/support_2080ti

Conversation

@giostrives

@giostrives giostrives commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

2080ti support was recently added, this adds.

guards the cudaFuncGetName call for CUDA <12.3 compatibility

@MahirZukic

Copy link
Copy Markdown

This should be closed as it is already addressed in #87 which got merged.

…A 12.0 failed)

It names kernels only for the STRATA_VERIFY_NODES debug dump.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@giostrives
giostrives force-pushed the feature/support_2080ti branch from 7732cc8 to e08ffbd Compare September 30, 2026 07:51
@giostrives giostrives changed the title Feature/support 2080ti Feature/support 2080ti with CUDA <12.3 compatibility Sep 30, 2026
@giostrives

giostrives commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor Author

This should be closed as it is already addressed in #87 which got merged.

I just noticed it was, still I have an old CUDA and I needed an extra if/else.

@giostrives giostrives changed the title Feature/support 2080ti with CUDA <12.3 compatibility Feature/support CUDA <12.3 compatibility Sep 30, 2026
@Niko1221

Copy link
Copy Markdown
Owner

Correct: main still called it unguarded on CUDA. Merged for 0.1.28 (in testing); CUDA older than 12.3 just gets generic names in that debug listing.

@Niko1221

Copy link
Copy Markdown
Owner

Released in 0.1.28. Thanks @giostrives!

@Niko1221 Niko1221 closed this Sep 30, 2026
@giostrives

Copy link
Copy Markdown
Contributor Author

Thank you, btw, I suggest anyone using this to update their CUDA: I got a +4/7% speedup by using 13.4 (using my other multi-gpu patch).
9U34r

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.

3 participants