Skip to content

Replace ocaml.1 with ocaml_1 in provider name for stapsdt elf notes #638

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged

Conversation

gretay-js
Copy link
Contributor

We piggyback on the provider field of stapsdt elf notes to indicate the version of ocaml-probes. (The experiemental mechanism for enabling ocaml-probes is separate from systemtap and subject to change so it was useful to have a version number, see #60 (comment).)

Looks like perf parses the provider field of stapsdt elf notes expecting its syntax to be a C symbol name (i.e., dots are not allowed). For example, if test.exe has any ocaml-probes, then perf record test.exe sometimes prints this warning:

Semantic error :sdt_ocaml.1 is bad for event name -it must follow C symbol-naming rule.

It seems to generate perf.data just fine if we don't use these probes with -e.
Thanks @Xyene for reporting this issue.

Looks like some versions of perf parse provider expecting it to have
syntax of a C function name, i.e., dots are not allowed.
@gretay-js gretay-js requested a review from xclerc as a code owner May 3, 2022 17:24
@gretay-js gretay-js merged commit a7cd4f8 into oxcaml:main May 4, 2022
mshinwell pushed a commit that referenced this pull request May 20, 2022
…638)

Looks like some versions of perf parse provider expecting it to have
syntax of a C function name, i.e., dots are not allowed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants