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

Add support for writing EventPipeEvents to ProfAPI #19157

Closed
wants to merge 1 commit into from

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
Member

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