Add EventPipe Processor Number support and make NetTrace the default … (#25276)
authorNoah Falk <noahfalk@users.noreply.github.com>
Fri, 21 Jun 2019 22:27:21 +0000 (15:27 -0700)
committerGitHub <noreply@github.com>
Fri, 21 Jun 2019 22:27:21 +0000 (15:27 -0700)
commit961e8d7bd1bdd058ee5f8f34937e9f3f9d80b65b
tree06c18fd142f826966f051c908252089e6e5ec55b
parent205e01fc5020f597ee69643c24fd56268cdf1b50
Add EventPipe Processor Number support and make NetTrace the default … (#25276)

* Add EventPipe Processor Number support and make NetTrace the default format

The EventPipe header now has a Processor Number field. On windows we query the correct value, on other OSes we currently have a -1 placeholder, but the field is written to the format regardless.
NetTrace is now the default format when using the environment variable, and the format must be explicitly configured when using the IPC channel or managed API. A parallel change in the diagnostics repo is changing dotnet-trace and dotnet-counter to specify nettrace format which means .NET devs should see nettrace almost exclusively from now on. If for whatever reason it is needed, NetPerf remains available if a scenario explicitly requests to use it.

* PR feedback + attempting to fix broken tests
19 files changed:
dependencies.props
src/System.Private.CoreLib/src/System/Diagnostics/Eventing/EventPipe.cs
src/System.Private.CoreLib/src/System/Diagnostics/Eventing/EventPipeController.cs
src/System.Private.CoreLib/src/System/Diagnostics/Eventing/EventPipeEventDispatcher.cs
src/inc/clrconfigvalues.h
src/vm/eventpipe.cpp
src/vm/eventpipe.h
src/vm/eventpipeblock.cpp
src/vm/eventpipeblock.h
src/vm/eventpipebuffer.cpp
src/vm/eventpipeconfiguration.cpp
src/vm/eventpipeeventinstance.cpp
src/vm/eventpipeeventinstance.h
src/vm/eventpipeinternal.cpp
src/vm/eventpipeinternal.h
src/vm/eventpipeprotocolhelper.cpp
src/vm/eventpipeprotocolhelper.h
tests/src/tracing/common/TraceConfiguration.cs
tests/src/tracing/regress/GitHub_22247/GitHub_22247.cs