Skip to content

removed need to convert from ctf to json#1

Draft
KerianFiter wants to merge 2 commits intoarshadlab:masterfrom
KerianFiter:json-fix
Draft

removed need to convert from ctf to json#1
KerianFiter wants to merge 2 commits intoarshadlab:masterfrom
KerianFiter:json-fix

Conversation

@KerianFiter
Copy link

perf now has a 'to-json' param

@arshadlab
Copy link
Owner

Hi,
Thanks for the efforts. The to-json generates output in a format not compatible with trace2html that's why I didn't use earlier. Can you confirm if the trace.html gets generate correctly either going through to-json or via ctf2ctf path ?

@KerianFiter
Copy link
Author

Hi,
Indeed the to-json output is not compatible sadly. However ctf2ctf seems to have removed the to-ctf param, so then no trace.html is generated at the end.

I'm thinking about converting the trace to a perf file and reading it with Speedscope.

@KerianFiter KerianFiter marked this pull request as draft May 20, 2024 00:08
@arshadlab
Copy link
Owner

Hi,
Perf still have to-ctf option in latest kernel code. See https://github.com/torvalds/linux/blob/master/tools/perf/builtin-data.c
Just that it requires babel trace dev package to be installed else the final build of perf disables this option.

@arshadlab arshadlab force-pushed the master branch 8 times, most recently from 56a13d6 to d4c40cd Compare December 25, 2024 04:34
@arshadlab arshadlab force-pushed the master branch 3 times, most recently from 1125fdc to 341ce05 Compare March 9, 2025 23:56
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.

2 participants