Wavefront launch stats also only normalized by waves

ROCm / rocprofiler-compute

Advanced Profiling and Analytics for AMD Hardware

https://rocm.docs.amd.com/projects/omniperf/en/latest/

MIT License

135 stars 49 forks source link

Wavefront launch stats also only normalized by waves #81

Closed skyreflectedinmirrors closed 1 year ago

skyreflectedinmirrors commented 1 year ago

Similar to https://github.com/AMDResearch/omniperf/issues/68, these values are only normalizable by waves:

https://github.com/AMDResearch/omniperf/blob/main/src/omniperf_analyze/configs/gfx90a/0700_wavefront-launch.yaml#L101

It would be a good idea to check all the YAML files to get this once and for all

coleramos425 commented 1 year ago

Enabled multi-normalization for

Wave Cycles
Dependency Wait Cycles
Issue Wait Cycles
Active Cycles

Others like "Kernel Time (Nanosec)", "Kernel Time (Cycles)", "Instr/wavefront" did not change because their name explicitly defines a static normalization.