pal init: Move InitializeFlushProcessWriteBuffers() after VIRTUALInitialize() (#107100)

A fixup of commit 27ee590 that's broken on platforms which don't
support membarrier() syscall: GetVirtualPageSize() is called in the
fallback path of InitializeFlushProcessWriteBuffers() and attempts to
mmap() zero bytes.

Move InitializeFlushProcessWriteBuffers() after VIRTUALInitialize() but
before the first thread is created.

Fixes #106892
Fixes #106722

Co-authored-by: Haris Okanovic <[email protected]>

Author: harisokanovic

src/coreclr/pal/src/init/pal.cpp

@@ -359,16 +359,6 @@ Initialize(
             goto CLEANUP0a;
         }
 
-        if (flags & PAL_INITIALIZE_FLUSH_PROCESS_WRITE_BUFFERS)
-        {
-            // Initialize before first thread is created for faster load on Linux
-            if (!InitializeFlushProcessWriteBuffers())
-            {
-                palError = ERROR_PALINIT_INITIALIZE_FLUSH_PROCESS_WRITE_BUFFERS;
-                goto CLEANUP0a;
-            }
-        }
-
         // The gSharedFilesPath is allocated dynamically so its destructor does not get
         // called unexpectedly during cleanup
         gSharedFilesPath = InternalNew<PathCharString>();
@@ -616,6 +606,17 @@ Initialize(
             goto CLEANUP10;
         }
 
+        if (flags & PAL_INITIALIZE_FLUSH_PROCESS_WRITE_BUFFERS)
+        {
+            // Initialize before first thread is created for faster load on Linux
+            if (!InitializeFlushProcessWriteBuffers())
+            {
+                ERROR("Unable to initialize flush process write buffers\n");
+                palError = ERROR_PALINIT_INITIALIZE_FLUSH_PROCESS_WRITE_BUFFERS;
+                goto CLEANUP10;
+            }
+        }
+
         if (flags & PAL_INITIALIZE_SYNC_THREAD)
         {
             //