ROCm / rocprofiler

ROC profiler library. Profiling with perf-counters and derived metrics.
https://rocm.docs.amd.com/projects/rocprofiler/en/latest/
MIT License
132 stars 49 forks source link

adding support for exporting results as Radeon GPU Profiler profile #107

Open Epliz opened 1 year ago

Epliz commented 1 year ago

Hi,

I have recently been using the Radeon GPU Profiler on Windows to optimize OpenCL/HIP kernels on RDNA2, and it has been very very useful thanks to its nice visualization and support of instruction tracing.

Would it be possible for you to consider adding support in rocProfiler for exporting the tracing results in the rgp format expected by the Radeon GPU profiler? I asked your colleagues from RGP at https://github.com/GPUOpen-Tools/radeon_gpu_profiler/issues/84 , and they indicated that rocm doesn't have the required support. Sounds like a challenge to you guys to add that support, don't you think?

Best regards, Epliz

jrmadsen commented 1 year ago

I’m not familiar with this format and it’s unlikely that we will support writing data in that format but you might want to look into our new performance analysis tools AMDResearch/omnitrace and AMDResearch/omniperf

Epliz commented 11 months ago

Seems like it is more than possible, as indicated at https://github.com/GPUOpen-Tools/radeon_gpu_profiler/issues/84#issuecomment-1846308984 . @jrmadsen , can you please share with whoever it might concern?

Epliz commented 4 months ago

Hi, @jrmadsen , @ApoKalipse-V ,

It seems like you added instruction tracing suppport in rocprofilerv2, that's more than appreciated!

Any chance that you might support exporting the traces to RGP format so that they can be loaded in the RGP UI? The current CSV output is not very user friendly...

jrmadsen commented 4 months ago

rocprofiler v2 is no longer being actively developed for numerous reasons. There is a new rocprofiler-sdk that will be released as a beta in ROCm 6.2 and beyond that point, we do not intend to add any new features to rocprof or rocprofv2. However, considerations regarding RGP support in rocprofv3 are being discussed.

ppanchad-amd commented 2 months ago

@Epliz Do you still need assistance with this ticket? If not, please close the ticket. Thanks!

Epliz commented 2 months ago

@ppanchad-amd , I believe it is still very much relevant. Such a feature would be a veryuch needed improvement to the user experience