omniperf icon indicating copy to clipboard operation
omniperf copied to clipboard

Add custom kernel name shortening interface

Open IanBogle opened this issue 2 years ago • 2 comments

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 avatar Oct 30 '23 17:10 IanBogle

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

srinathv avatar Nov 02 '23 15:11 srinathv

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.

IanBogle avatar Dec 13 '23 00:12 IanBogle