Remove some unnecessary indirection to DiagnosticServerAdapter/EventPipeAdapter in nativeaot (#90132)

Author: elinor-fung

src/coreclr/nativeaot/Runtime/CommonMacros.h

@@ -64,6 +64,8 @@
 
 #ifndef __GCENV_BASE_INCLUDED__
 
+#include <cstdint>
+
 //
 // This macro returns val rounded up as necessary to be a multiple of alignment; alignment must be a power of 2
 //

src/coreclr/nativeaot/Runtime/DisabledEventPipeInterface.cpp

@@ -1,14 +1,14 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
-void EventPipeAdapter_Initialize() {}
+void EventPipe_Initialize() {}
 
-bool DiagnosticServerAdapter_Initialize() { return false; }
-void DiagnosticServerAdapter_PauseForDiagnosticsMonitor() {}
+bool DiagnosticServer_Initialize() { return false; }
+void DiagnosticServer_PauseForDiagnosticsMonitor() {}
 
-void EventPipeAdapter_FinishInitialize() {}
+void EventPipe_FinishInitialize() {}
 
 void EventPipe_ThreadShutdown() { }
 
-void EventPipeAdapter_Shutdown() {}
-bool DiagnosticServerAdapter_Shutdown() { return false; }
+void EventPipe_Shutdown() {}
+bool DiagnosticServer_Shutdown() { return false; }

src/coreclr/nativeaot/Runtime/EnabledEventPipeInterface.cpp

@@ -1,17 +1,18 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
-#include "eventpipeadapter.h"
-#include "diagnosticserveradapter.h"
+#include <eventpipe/ep.h>
+#include <eventpipe/ep-rt-aot.h>
+#include <eventpipe/ds-server.h>
 
-void EventPipeAdapter_Initialize() { EventPipeAdapter::Initialize(); }
+void EventPipe_Initialize() { ep_init(); }
 
-bool DiagnosticServerAdapter_Initialize() { return DiagnosticServerAdapter::Initialize(); }
-void DiagnosticServerAdapter_PauseForDiagnosticsMonitor() { DiagnosticServerAdapter::PauseForDiagnosticsMonitor();}
+bool DiagnosticServer_Initialize() { return ds_server_init(); }
+void DiagnosticServer_PauseForDiagnosticsMonitor() { ds_server_pause_for_diagnostics_monitor(); }
 
-void EventPipeAdapter_FinishInitialize() { EventPipeAdapter::FinishInitialize(); }
+void EventPipe_FinishInitialize() { ep_finish_init(); }
 
 void EventPipe_ThreadShutdown() { ep_rt_aot_thread_exited(); }
 
-void EventPipeAdapter_Shutdown() { EventPipeAdapter::Shutdown(); }
-bool DiagnosticServerAdapter_Shutdown() { return DiagnosticServerAdapter::Shutdown(); }
+void EventPipe_Shutdown() { ep_shutdown(); }
+bool DiagnosticServer_Shutdown() { return ds_server_shutdown(); }

src/coreclr/nativeaot/Runtime/EventPipeInterface.h

@@ -5,19 +5,18 @@
 #define EVENTPIPE_INTERFACE_H
 
 // Initialize EventPipe
-void EventPipeAdapter_Initialize();
+void EventPipe_Initialize();
 
 // Initialize DS
-bool DiagnosticServerAdapter_Initialize();
-void DiagnosticServerAdapter_PauseForDiagnosticsMonitor();
+bool DiagnosticServer_Initialize();
+void DiagnosticServer_PauseForDiagnosticsMonitor();
 
-
-void EventPipeAdapter_FinishInitialize();
+void EventPipe_FinishInitialize();
 
 void EventPipe_ThreadShutdown();
 
-void EventPipeAdapter_Shutdown();
-bool DiagnosticServerAdapter_Shutdown();
+void EventPipe_Shutdown();
+bool DiagnosticServer_Shutdown();
 
 void EventTracing_Initialize();
 #endif //EVENTPIPE_INTERFACE_H

src/coreclr/nativeaot/Runtime/PalRedhawk.h

@@ -17,6 +17,10 @@
 
 #include <sal.h>
 #include <stdarg.h>
+#ifdef TARGET_UNIX
+#include <pthread.h>
+#endif
+
 #include "CommonTypes.h"
 #include "CommonMacros.h"
 #include "gcenv.structs.h" // CRITICAL_SECTION

src/coreclr/nativeaot/Runtime/StackFrameIterator.h

@@ -4,7 +4,12 @@
 #ifndef __StackFrameIterator_h__
 #define __StackFrameIterator_h__
 
+#include "CommonMacros.h"
 #include "ICodeManager.h"
+#include "PalRedhawk.h" // NATIVE_CONTEXT
+#include "regdisplay.h"
+
+#include "forward_declarations.h"
 
 struct ExInfo;
 typedef DPTR(ExInfo) PTR_ExInfo;
@@ -22,6 +27,7 @@ struct EHEnum
     EHEnumState m_state;
 };
 
+class StackFrameIterator;
 EXTERN_C FC_BOOL_RET FASTCALL RhpSfiInit(StackFrameIterator* pThis, PAL_LIMITED_CONTEXT* pStackwalkCtx, CLR_BOOL instructionFault);
 EXTERN_C FC_BOOL_RET FASTCALL RhpSfiNext(StackFrameIterator* pThis, uint32_t* puExCollideClauseIdx, CLR_BOOL* pfUnwoundReversePInvoke);
 

src/coreclr/nativeaot/Runtime/diagnosticserveradapter.h

@@ -1,23 +1,20 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
-#include "common.h"
-#include "eventpipeadapter.h"
+#include "CommonTypes.h"
+#include "CommonMacros.h"
+#include "PalRedhawk.h"
 
-#ifdef FEATURE_PERFTRACING
+#include <eventpipe/ep.h>
 
-#include "gcenv.h"
-#include "regdisplay.h"
-#include "StackFrameIterator.h"
-#include "thread.h"
-#include "SpinLock.h"
+#ifdef FEATURE_PERFTRACING
 
 struct EventPipeEventInstanceData;
 
 struct EventPipeSessionInfo;
 
 EXTERN_C NATIVEAOT_API uint64_t __cdecl RhEventPipeInternal_Enable(
-    LPCWSTR outputFile,
+    const WCHAR* outputFile,
     EventPipeSerializationFormat format,
     uint32_t circularBufferSizeInMB,
     /* COR_PRF_EVENTPIPE_PROVIDER_CONFIG */ const void * pProviders,
@@ -31,7 +28,7 @@ EXTERN_C NATIVEAOT_API void __cdecl RhEventPipeInternal_Disable(uint64_t session
 }
 
 EXTERN_C NATIVEAOT_API intptr_t __cdecl RhEventPipeInternal_CreateProvider(
-    LPCWSTR providerName,
+    const WCHAR* providerName,
     EventPipeCallback pCallbackFunc,
     void* pCallbackContext)
 {
@@ -50,7 +47,7 @@ EXTERN_C NATIVEAOT_API intptr_t __cdecl RhEventPipeInternal_DefineEvent(
     return 0;
 }
 
-EXTERN_C NATIVEAOT_API intptr_t __cdecl RhEventPipeInternal_GetProvider(LPCWSTR providerName)
+EXTERN_C NATIVEAOT_API intptr_t __cdecl RhEventPipeInternal_GetProvider(const WCHAR* providerName)
 {
     return 0;
 }

src/coreclr/nativeaot/Runtime/disabledeventpipeinternal.cpp

@@ -1,7 +1,9 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
-#include "eventpipeadapter.h"
+#include "CommonTypes.h"
+#include "CommonMacros.h"
+#include "PalRedhawk.h"
 
 #ifdef FEATURE_PERFTRACING
 
@@ -78,7 +80,7 @@ EXTERN_C NATIVEAOT_API void __cdecl NativeRuntimeEventSource_LogThreadPoolIOPack
 {
 }
 
-EXTERN_C NATIVEAOT_API void __cdecl NativeRuntimeEventSource_LogExceptionThrown(LPCWSTR exceptionTypeName, LPCWSTR exceptionMessage, void* faultingIP, HRESULT hresult)
+EXTERN_C NATIVEAOT_API void __cdecl NativeRuntimeEventSource_LogExceptionThrown(const WCHAR* exceptionTypeName, const WCHAR* exceptionMessage, void* faultingIP, HRESULT hresult)
 {
 }
 

src/coreclr/nativeaot/Runtime/disabledruntimeeventinternal.cpp

@@ -107,7 +107,6 @@ list(APPEND GEN_EVENTPIPE_SOURCES
 list(APPEND AOT_EVENTPIPE_MANAGED_TO_NATIVE_SOURCES
    ${RUNTIME_DIR}/eventpipeinternal.cpp
    ${RUNTIME_DIR}/eventpipeadapter.h
-   ${RUNTIME_DIR}/diagnosticserveradapter.h
    ${RUNTIME_DIR}/EnabledEventPipeInterface.cpp
    ${RUNTIME_DIR}/runtimeeventinternal.cpp
 )