[release/6.0] Fix AwaitableSocketAsyncEventArgs reorderings on weaker memory models (#51663)

* Fix AwaitableSocketAsyncEventArgs reorderings on weaker memory models

* add comment

---------

Co-authored-by: antonfirsov <[email protected]>

Author: github-actions[bot]

src/Servers/Kestrel/Transport.Sockets/src/Internal/SocketAwaitableEventArgs.cs

@@ -20,7 +20,11 @@ internal class SocketAwaitableEventArgs : SocketAsyncEventArgs, IValueTaskSource
 
         private readonly PipeScheduler _ioScheduler;
 
-        private Action<object?>? _continuation;
+    // There are places where we read the _continuation field and then read some other state which we assume to be consistent
+    // with the value we read in _continuation. Without a fence, those secondary reads could be reordered with respect to the first.
+    // https://github.com/dotnet/runtime/pull/84432
+    // https://github.com/dotnet/aspnetcore/issues/50623
+    private volatile Action<object?>? _continuation;
 
         public SocketAwaitableEventArgs(PipeScheduler ioScheduler)
             : base(unsafeSuppressExecutionContextFlow: true)