Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Add support for writing EventPipeEvents to ProfAPI#19157

Closed
mjsabby wants to merge 1 commit intodotnet:masterfrom
mjsabby:eventPipeToProfApi
Closed

Add support for writing EventPipeEvents to ProfAPI#19157
mjsabby wants to merge 1 commit intodotnet:masterfrom
mjsabby:eventPipeToProfApi

Conversation

@mjsabby
Copy link

@mjsabby mjsabby commented Jul 27, 2018

This augments the EventPipe framework to send data to the Profiler APIs in addition to any other receivers that may be configured.

This change does not add or remove any requirement on how the eventing system is initialized. Although it does provide a provision to the Profiler APIs that despite whatever the eventing system may be initialized to, it can eject itself when it needs to.

@mjsabby
Copy link
Author

mjsabby commented Jul 27, 2018

@brianrob @noahfalk

@discostu105
Copy link

I'd also be interested in that feature. Talked about something similar a long time ago in #4382 (route GC ETW events directly to profiler to avoid NT kernel roundtrip).

@georgschausberger
Copy link

Is there any update? I'm highly interested into this too!

@mjsabby
Copy link
Author

mjsabby commented Nov 26, 2018

I met @noahfalk recently, he can confirm the status, but I've asked it be at least considered for 3.0, given at least one implementation works (although it may not be the final one).

Can you (@discostu105 @georgschausberger) describe your scenarios? Is it just for GC events or would you like all the events so that if you were building a monitoring tool a lot more can be gleaned with preexisting runtime instrumentation vs. for example you adding your own instrumentation via IL rewriting?

@georgschausberger
Copy link

georgschausberger commented Nov 27, 2018

@mjsabby the main purpose is building a monitoring tool which should be able to use all kinds of events. Furthermore synchronous events would be much appreciated to have a chance of thread affine time tracking.

@davmason
Copy link

davmason commented Nov 5, 2019

Closing this in preparation for repo consolidation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants