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.

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

Correct EventPipe comparison info #42456

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Conversation

noahfalk
Copy link
Member

@noahfalk noahfalk commented Sep 4, 2024

The EventPipe docs were saying things about LTTng that were either incorrect or misleading. I think changing the comparison to use perf_events makes it more effective.

  • LTTng does support tracing kernel events, but as written users are likely to get the wrong idea. Perfcollect, the tool we recommend to when using LTTng with .NET, uses perf_events rather than LTTng to perform the kernel tracing. Perfcollect only uses lttng for its user-mode tracing capability.
  • LTTng doesn't support native callstacks as far as I am aware.

Internal previews

📄 File 🔗 Preview link
docs/core/diagnostics/eventpipe.md docs/core/diagnostics/eventpipe

The EventPipe docs were saying things about LTTng that were either incorrect or misleading. I think changing the comparison to use perf_events makes it more effective.

- LTTng does support tracing kernel events, but as written users are likely to get the wrong idea. Perfcollect, the tool we recommend to when using LTTng with .NET, uses perf_events rather than LTTng to perform the kernel tracing. Perfcollect only uses lttng for its user-mode tracing capability.
- LTTng doesn't support native callstacks as far as I am aware.
@noahfalk
Copy link
Member Author

noahfalk commented Sep 4, 2024

PTAL @brianrob @dotnet/dotnet-diag

@noahfalk
Copy link
Member Author

noahfalk commented Sep 4, 2024

fyi @Maoni0

Copy link
Member

@Maoni0 Maoni0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for fixing this!

Copy link
Member

@tommcdon tommcdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM- thanks!

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM as well. I'll :shipit: now.

@BillWagner BillWagner merged commit c98fecf into dotnet:main Sep 4, 2024
10 checks passed
@brianrob
Copy link
Member

brianrob commented Sep 4, 2024

I see this is already merged, but thanks for including me and thanks for fixing @noahfalk!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants