Add context argument to crash dump functions

Co-authored-by: jkotas <6668460+jkotas@users.noreply.github.com>

Author: Copilot

src/coreclr/nativeaot/Runtime/unix/HardwareExceptions.cpp

@@ -559,7 +559,7 @@ void SIGSEGVHandler(int code, siginfo_t *siginfo, void *context)
         RestoreSignalHandler(code, &g_previousSIGSEGV);
     }
 
-    PalCreateCrashDumpIfEnabled(code, siginfo);
+    PalCreateCrashDumpIfEnabled(code, siginfo, context);
 }
 
 // Handler for the SIGFPE signal
@@ -581,7 +581,7 @@ void SIGFPEHandler(int code, siginfo_t *siginfo, void *context)
         RestoreSignalHandler(code, &g_previousSIGFPE);
     }
 
-    PalCreateCrashDumpIfEnabled(code, siginfo);
+    PalCreateCrashDumpIfEnabled(code, siginfo, context);
 }
 
 // Initialize hardware exception handling

src/coreclr/nativeaot/Runtime/unix/PalCreateDump.cpp

@@ -355,13 +355,18 @@ CreateCrashDump(
 Parameters:
     signal - POSIX signal number or 0
     siginfo - signal info or nullptr
+    context - signal context or nullptr
     exceptionRecord - address of exception record or nullptr
 
 (no return value)
 --*/
 void
-PalCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, void* exceptionRecord)
+PalCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, void* context, void* exceptionRecord)
 {
+    // Store context in a volatile variable to prevent optimization
+    volatile void* volatileContext = context;
+    (void)volatileContext;
+
 #if !defined(HOST_MACCATALYST) && !defined(HOST_IOS) && !defined(HOST_TVOS)
     // If enabled, launch the create minidump utility and wait until it completes
     if (g_argvCreateDump[0] != nullptr)
@@ -454,13 +459,13 @@ PalCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, void* exceptionRecor
 void
 PalCreateCrashDumpIfEnabled()
 {
-    PalCreateCrashDumpIfEnabled(SIGABRT, nullptr, nullptr);
+    PalCreateCrashDumpIfEnabled(SIGABRT, nullptr, nullptr, nullptr);
 }
 
 void
 PalCreateCrashDumpIfEnabled(void* pExceptionRecord)
 {
-    PalCreateCrashDumpIfEnabled(SIGABRT, nullptr, pExceptionRecord);
+    PalCreateCrashDumpIfEnabled(SIGABRT, nullptr, nullptr, pExceptionRecord);
 }
 
 /*++

src/coreclr/nativeaot/Runtime/unix/PalCreateDump.h

@@ -7,5 +7,5 @@
 
 extern bool PalCreateDumpInitialize();
 extern void PalCreateCrashDumpIfEnabled();
-extern void PalCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo = nullptr, void* exceptionRecord = nullptr);
+extern void PalCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo = nullptr, void* context = nullptr, void* exceptionRecord = nullptr);
 extern void PalCreateCrashDumpIfEnabled(void* pExceptionRecord);

src/coreclr/pal/src/exception/signal.cpp

@@ -433,7 +433,7 @@ static void invoke_previous_action(struct sigaction* action, int code, siginfo_t
             // Shutdown and create the core dump before we restore the signal to the default handler.
             PROCNotifyProcessShutdown(IsRunningOnAlternateStack(context));
 
-            PROCCreateCrashDumpIfEnabled(code, siginfo, true);
+            PROCCreateCrashDumpIfEnabled(code, siginfo, context, true);
 
             // Restore the original and restart h/w exception.
             restore_signal(code, action);
@@ -462,7 +462,7 @@ static void invoke_previous_action(struct sigaction* action, int code, siginfo_t
 
     PROCNotifyProcessShutdown(IsRunningOnAlternateStack(context));
 
-    PROCCreateCrashDumpIfEnabled(code, siginfo, true);
+    PROCCreateCrashDumpIfEnabled(code, siginfo, context, true);
 }
 
 /*++
@@ -851,7 +851,7 @@ static void sigterm_handler(int code, siginfo_t *siginfo, void *context)
         DWORD val = 0;
         if (enableDumpOnSigTerm.IsSet() && enableDumpOnSigTerm.TryAsInteger(10, val) && val == 1)
         {
-            PROCCreateCrashDumpIfEnabled(code, siginfo, false);
+            PROCCreateCrashDumpIfEnabled(code, siginfo, context, false);
         }
     }
 

src/coreclr/pal/src/include/pal/process.h

@@ -182,11 +182,12 @@ VOID PROCNotifyProcessShutdown(bool isExecutingOnAltStack = false);
 Parameters:
   signal - POSIX signal number
   siginfo - POSIX signal info or nullptr
+  context - signal context or nullptr
   serialize - allow only one thread to generate core dump
 
 (no return value)
 --*/
-VOID PROCCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, bool serialize);
+VOID PROCCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, void* context, bool serialize);
 
 #ifdef __cplusplus
 }

src/coreclr/pal/src/thread/process.cpp

@@ -2716,23 +2716,32 @@ PAL_GenerateCoreDump(
 Parameters:
   signal - POSIX signal number
   siginfo - POSIX signal info or nullptr
+  context - signal context or nullptr
   serialize - allow only one thread to generate core dump
 
 (no return value)
 --*/
 #ifdef HOST_ANDROID
 #include <minipal/log.h>
 VOID
-PROCCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, bool serialize)
+PROCCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, void* context, bool serialize)
 {
+    // Store context in a volatile variable to prevent optimization
+    volatile void* volatileContext = context;
+    (void)volatileContext;
+
     // TODO: Dump all managed threads callstacks into logcat and/or file?
     // TODO: Dump stress log into logcat and/or file when enabled?
     minipal_log_write_fatal("Aborting process.\n");
 }
 #else
 VOID
-PROCCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, bool serialize)
+PROCCreateCrashDumpIfEnabled(int signal, siginfo_t* siginfo, void* context, bool serialize)
 {
+    // Store context in a volatile variable to prevent optimization
+    volatile void* volatileContext = context;
+    (void)volatileContext;
+
     // If enabled, launch the create minidump utility and wait until it completes
     if (!g_argvCreateDump.empty())
     {
@@ -2821,7 +2830,7 @@ PROCAbort(int signal, siginfo_t* siginfo)
     // Do any shutdown cleanup before aborting or creating a core dump
     PROCNotifyProcessShutdown();
 
-    PROCCreateCrashDumpIfEnabled(signal, siginfo, true);
+    PROCCreateCrashDumpIfEnabled(signal, siginfo, nullptr, true);
 
     // Restore all signals; the SIGABORT handler to prevent recursion and
     // the others to prevent multiple core dumps from being generated.