Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Support profiling CPU + Wall clock together #138

Open
be-hase opened this issue Jan 23, 2024 · 2 comments
Open

Support profiling CPU + Wall clock together #138

be-hase opened this issue Jan 23, 2024 · 2 comments

Comments

@be-hase
Copy link

be-hase commented Jan 23, 2024

It seems to be possible to get both since async profiler v3.
async-profiler/async-profiler#740

How about also supporting pyroscope?
@korniltsev
Copy link
Collaborator

korniltsev commented Jan 23, 2024 •
edited

We will look into it.

In the meantime it is already possible to have wall + cpu by using wall. The pyroscope assigns events STATE_RUNNABLE to extra process_cpu profile type.

@korniltsev
Copy link
Collaborator

We also do not recommend wall for continuous profiling because of bigger overhead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@korniltsev @be-hase