Skip to content

Handle uppercase cbr state in perf output#378

Open
yinli-systems wants to merge 1 commit into
janestreet:masterfrom
yinli-systems:kevin/cbr-uppercase-state
Open

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

Conversation

@yinli-systems

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