Skip to content

Handle uppercase cbr state in perf output#378

Open
Kevin-Li-2025 wants to merge 1 commit into
janestreet:masterfrom
Kevin-Li-2025:kevin/cbr-uppercase-state
Open

Handle uppercase cbr state in perf output#378
Kevin-Li-2025 wants to merge 1 commit into
janestreet:masterfrom
Kevin-Li-2025:kevin/cbr-uppercase-state

Conversation

@Kevin-Li-2025

Copy link
Copy Markdown

Fixes #324.

Summary

Root cause

perf_cbr_event_re only allowed lowercase letters in the optional text between the event name and the frequency payload. On Ubuntu 24.04 / perf 6.8, perf can emit uppercase state text such as D, so parsing raised before extracting the frequency.

Validation

I could not run the OCaml expect suite locally because this machine does not have ocaml, dune, or opam installed.

Signed-off-by: Kevin-Li-2025 <2242139@qq.com>
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.

"exception raised while parsing perf output" with "cbr: 36 freq: 3629 MHz (450%)"

1 participant