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

offwaketime #331

Merged
merged 2 commits into from Jan 30, 2016
Merged

offwaketime #331

merged 2 commits into from Jan 30, 2016

Conversation

brendangregg
Copy link
Member

No description provided.

4ast added a commit that referenced this pull request Jan 30, 2016
@4ast
Merge pull request #331 from brendangregg/master
bf9d207 
offwaketime
@4ast 4ast merged commit bf9d207 into iovisor:master Jan 30, 2016
@4ast
Copy link
Member

4ast commented Jan 30, 2016

This is awesome!
Is there a way to make the time easier to interpret?
Currently it's "The number beneath the stacks is the total time spent from the blocking event to the wakeup event. This is summed for all occurances with the same stack pairs." but it's hard to translate that to wall-clock nsec.

@brendangregg
Copy link
Member Author

The time is the total time. Conceptually, maybe we can explain it as an off-CPU analogue to CPU sampling: imagine we're sampling when threads are off-CPU, and showing the totals in different paths.

Time gets more complex with more waker stacks (chain graphs), and I'm putting more thought into it...

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@brendangregg @4ast