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

Add custom kernel name shortening interface #196

Open
IanBogle opened this issue Oct 30, 2023 · 2 comments
Open

Add custom kernel name shortening interface #196

IanBogle opened this issue Oct 30, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@IanBogle
Copy link

IanBogle commented Oct 30, 2023

Is your feature request related to a problem? Please describe.
Users that profile Kokkos and Raja-based applications would like to be able to "nickname" functions for easier display.

Specifically, we've gotten feedback from code teams at Sandia National Labs that this would increase the usability of Omniperf for them.

Describe the solution you'd like
Allow users to provide a file that contains exact kernel names to match, and shortened strings to replace them with.

Describe alternatives you've considered
Pointing to an alternate .csv file could work, but seems pretty fragile/potentially frustrating.
Manually editing the existing dispatch to kernel name mapping doesn't work for all cases: roofline generation will still show the long kernel names on the kernel legend.

@IanBogle IanBogle added the enhancement New feature or request label Oct 30, 2023
@srinathv
Copy link

srinathv commented Nov 2, 2023

I like this suggestion. I would like to understand if somthing like this already exists.

@IanBogle
Copy link
Author

I encountered a team using roctx-rename and ROCP_RENAME_KERNEL to rename kernels in traces using roctx-region. This could serve as a workaround, except it only changes the .json output, not the .csv output. Using the same environment variable with omniperf did not result in shortened kernel names due to the .json/.csv issue, but also omniperf's rocprof call doesn't seem to use roctx-trace.

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

No branches or pull requests

2 participants