Fix EventActivityIdControl test on MonoVM. (#39178)
authorJohan Lorensson <lateralusx.github@gmail.com>
Tue, 18 Aug 2020 11:00:27 +0000 (13:00 +0200)
committerGitHub <noreply@github.com>
Tue, 18 Aug 2020 11:00:27 +0000 (13:00 +0200)
commit44bac4135a7d74f2ced92f5b4d54263d895f0f41
tree29d4ea9ffc14ec243dfb10e151541f27fd5ed99c
parent907f7da59b40c80941b02ac2a46650adf3f606bc
Fix EventActivityIdControl test on MonoVM. (#39178)

Make EventSource call static version of EventActivityIdControl in
EventPipeEventProvider instead of direct calling low level
EventPipeInternal (now only called from EventPipe* sources). Doing
it that way will also keep EventPipeEventProvider.EventActivityIdControl
in S.P.C meaning that tests like eventactivityidcontrol won't risk
of using methods that has been trimmed away by linker.

Added missing case into Mono's EventPipeInternal::EventActivityIdControl
implementations.

Enabled eventactivityidcontrol test on Mono runtime. Adjusted test to
use static EventPipeEventProvider.EventActivityIdControl version.
src/coreclr/tests/issues.targets
src/libraries/System.Private.CoreLib/src/System/Diagnostics/Tracing/EventPipeEventProvider.cs
src/libraries/System.Private.CoreLib/src/System/Diagnostics/Tracing/EventSource.cs
src/mono/mono/metadata/icall-eventpipe.c
src/tests/tracing/eventactivityidcontrol/EventActivityIdControl.cs