Units missing from L2 per-channel breakdown

[skyreflectedinmirrors]

I think we just have to copy the units from the base metric into, e.g.:

https://github.com/AMDResearch/omniperf/blob/8af6efc0be5288dadd2e758556154e52f7406e5d/src/omniperf_analyze/configs/gfx90a/1800_L2_cache_per_channel.yaml#L207

[coleramos425]

Could you elaborate @arghdos. Per your screenshot, I'm reading metrics 18.1.1 - 18.1.7 all as "...req per $normUnit". Which seems correct to me

Am I missing something?

[skyreflectedinmirrors]

I guess it's sort of implied by the metric being "L1-L2 read req(uests)", but the unit would actually be "requests per wave" or "cycles per wave" (e.g., this one: https://github.com/AMDResearch/omniperf/blob/8af6efc0be5288dadd2e758556154e52f7406e5d/src/omniperf_analyze/configs/gfx90a/1800_L2_cache_per_channel.yaml#L921) not just "per wave". At a glance, it looks like all the request ones have this. Low priority issue tho

[coleramos425]

Patch merged into dev. Closing issue.