-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Closed
Description
#53122 introduced the feature, this tracks the remaining work.
Code
- Introduce code to not check all 32 slots for every callback (I am thinking about keeping a "max used" index and iterating until that is reached)
- Rename ProfControlBlock::FindNextFreeProfilerInfoSlot to indicate that it has side effects and isn't just a getter
- Potential rename to something besides notification profilers, since they can technically have side effects
Testing
- Stress test profilers attaching and detaching
- Run all runtime tests with notification profiler attached
- Test multiple profilers each with an eventpipe session
Documentation
- Write description of feature
- Write and publish sample to dotnet/samples
Reactions are currently unavailable