Move TypeSystemLog::OnKeywordsChanged from EtwCallback to EtwCallbackCommon to enable...
authorJohn Salem <josalem@microsoft.com>
Thu, 22 Aug 2019 23:46:24 +0000 (16:46 -0700)
committerGitHub <noreply@github.com>
Thu, 22 Aug 2019 23:46:24 +0000 (16:46 -0700)
Commit migrated from https://github.com/dotnet/coreclr/commit/16e83a73cb2df7e2434a7332f7608fc39286ac72

src/coreclr/src/vm/eventtrace.cpp

index 85aa7c4da89b7d4f65e16f724f008b6b156e63d5..0c288c93dfc9859ba15ddd0425db6372c9cea1ac 100644 (file)
@@ -4269,6 +4269,12 @@ VOID EtwCallbackCommon(
 #endif // !defined(FEATURE_PAL)
         ETW::GCLog::ForceGC(l64ClientSequenceNumber);
     }
+    // TypeSystemLog needs a notification when certain keywords are modified, so
+    // give it a hook here.
+    if (g_fEEStarted && !g_fEEShutDown && bIsPublicTraceHandle)
+    {
+        ETW::TypeSystemLog::OnKeywordsChanged();
+    }
 }
 
 // Individual callbacks for each EventPipe provider.
@@ -4488,13 +4494,6 @@ extern "C"
 
         EtwCallbackCommon(providerIndex, ControlCode, Level, MatchAnyKeyword, FilterData, false);
 
-        // TypeSystemLog needs a notification when certain keywords are modified, so
-        // give it a hook here.
-        if (g_fEEStarted && !g_fEEShutDown && bIsPublicTraceHandle)
-        {
-            ETW::TypeSystemLog::OnKeywordsChanged();
-        }
-
         // A manifest based provider can be enabled to multiple event tracing sessions
         // As long as there is atleast 1 enabled session, IsEnabled will be TRUE
         // Since classic providers can be enabled to only a single session,