Skip to content

Document HPM event 40 and external HPM events 41-50#8

Open
tejassinghbhati wants to merge 2 commits into
bsc-loca:mainfrom
tejassinghbhati:agent/validate-external-hpm-config
Open

Document HPM event 40 and external HPM events 41-50#8
tejassinghbhati wants to merge 2 commits into
bsc-loca:mainfrom
tejassinghbhati:agent/validate-external-hpm-config

Conversation

@tejassinghbhati

@tejassinghbhati tejassinghbhati commented Jul 15, 2026

Copy link
Copy Markdown

Summary

  • document HPM event 40: stalls in the register-rename stage
  • restore documentation for the external HPM events (41-50) exposed through the EXTERNAL_HPM_EVENT_NUM compile-time macro, including which configurations (4, 6, 10) expose each event

Why

top_drac.sv maps pmu_flags.stall_ir to HPM event 40, while events.md
previously stopped at event 39. In addition, the external PMU/HPM events
wired in via EXTERNAL_HPM_EVENT_NUM (L2/L1.5 access and miss, NOC flow and
stall events) were undocumented. Per review feedback on #7, the
configuration-validation RTL was dropped and only the documentation is kept.

Impact

Users configuring hardware performance counters can identify and select the
register-rename stall event and the external L2/L1.5/NOC events correctly.

Validation

  • verified each documentation entry against the event mapping block in
    rtl/top_drac.sv (events 41-44 for any supported width, 45-46 for width 6,
    45-50 for width 10)
  • documentation-only change; no HDL behavior changes

Related to #7

@tejassinghbhati
tejassinghbhati force-pushed the agent/validate-external-hpm-config branch from f666596 to 6071087 Compare July 16, 2026 14:04
@tejassinghbhati tejassinghbhati changed the title Validate external HPM event configurations Document HPM event 40 Jul 16, 2026
Document events 41-50 exposed via the EXTERNAL_HPM_EVENT_NUM macro,
as requested in PR review. Documentation-only change.
@tejassinghbhati tejassinghbhati changed the title Document HPM event 40 Document HPM event 40 and external HPM events 41-50 Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant