John Salem

Handle case where shutdown deadlocks on m_lock

1

*Opening as draft for feedback and test coverage* fixes https://github.com/dotnet/runtime/issues/51579 > Specifically, the order of events that cause this to occur is as follows with the chronological order of events...

https://github.com/microsoft/perfview/blob/8a34d2d64bc958902b2fa8ea5799437df57d8de2/src/FastSerialization/StreamReaderWriter.cs#L791-L814 The above code in `StreamReaderWriter` can cause latency for EventPipe. If you have infrequent events, e.g., EventCounters, and an event block ends up a length not divisible by 8,...

Unconditional printing to stderr

1

https://github.com/Sysinternals/SysinternalsEBPF/blob/b9035cf7780c9c733f4be06ff29f7b4fc447b395/telemetryLoader.c#L1426 The library unconditionally prints to `stderr`, which can be problematic for consumption. If you have a TUI for instance, this can corrupt the TUI output if the user doesn't...

telemetryStart should have a stop mechanism other than signals

1

Currently, `teletryStart` is a fully blocking call. There appears to be no way to stop tracing without sending a signal that would cause `perf_buffer__poll` to return `Start->Stop->Fini pattern would be...