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

OpenCL: Show ptxas info from verbosity 4 (was 5) #5608

Merged
merged 1 commit into from
Dec 8, 2024

Conversation

magnumripper
Copy link
Member

Level 5 would also show autotune stats, which is not always what I want. Also, we now get compiler warnings and ptxas at the same verbosity level which is logical (to me, at least).

Level 5 would also show autotune stats, which is not always what I want.
Also, we now get compiler warnings and ptxas at the same verbosity level
which is logical (to me, at least).
Copy link
Member

@solardiz solardiz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't mind, although maybe I'd want to see auto-tuning before I see build logs (so levels 4 and 5 swapped in this respect?)

Also, the names of these VERB_* constants are weird. But not to fix here.

@magnumripper
Copy link
Member Author

magnumripper commented Dec 7, 2024

Yes they are weird for hysterical reasons.

I'd love to change verbosity to a bitfield and have verbosity only as "low or high" but settable for specific contexts. So in your case something like -verbosity=+autotune,-nv-verbose (overriding any conf settings - they would be supported for each bit/context). It could even have some kind of backward compatibility so -verbosity=5 would set and unset default bits to a result similar to today. But I'm not sure I'll ever have time to implement any of this (and with "love to implement" I really mean "love to have implemented by someone else" lol.

Another thing I've pondered is to add something like -conf-override=xx so we don't need specific options for overriding config values - would make it a one-for-all canonical code. Like ssh -o option.

@magnumripper magnumripper merged commit af043dd into openwall:bleeding-jumbo Dec 8, 2024
35 of 36 checks passed
@magnumripper magnumripper deleted the ptxas-verb-4 branch December 8, 2024 10:09
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