[MT][browser] `TaskIDFromExternalContextTest` fails, Task has a negative ID

[ilonatommy]

Failing after enabling all MT library tests (#91536).

[FAIL] System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest
 Task within Parallel.For must have non-negative Id
 Expected: False
 Actual:   True
    at System.Threading.Tasks.Tests.ParallelForTests.<>c.<TaskIDFromExternalContextTest>b__4_1(Int32 i)
    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2[[System.Object, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e],[System.Int32, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
 --- End of stack trace from previous location ---
    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2[[System.Object, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e],[System.Int32, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
    at System.Threading.Tasks.TaskReplicator.Run[RangeWorker](ReplicatableUserAction`1 action, ParallelOptions options, Boolean stopOnFirstFailure)
    at System.Threading.Tasks.Parallel.ForWorker[Object,Int32](Int32 fromInclusive, Int32 toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.For(Int32 fromInclusive, Int32 toExclusive, Action`1 body)
    at System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest()
    at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
    at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

Error Blob

{
  "ErrorMessage": "[FAIL] System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest",
  "ErrorPattern": "",
  "BuildRetry": false,
  "ExcludeConsoleLog": false
}

Build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=523904

Known issue validation

Build: 🔎 https://dev.azure.com/dnceng-public/public/_build/results?buildId=523904
Error message validated: [FAIL] System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest
Result validation: ✅ Known issue matched with the provided build.
Validation performed at: 1/11/2024 6:33:01 PM UTC

Report

Build	Definition	Test	Pull Request
540124	dotnet/runtime	System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest	#97441
536247	dotnet/runtime	WasmTestOnBrowser-System.Threading.Tasks.Parallel.Tests.WorkItemExecution	#97270
535842	dotnet/runtime	WasmTestOnBrowser-System.Threading.Tasks.Parallel.Tests.WorkItemExecution	#97270
533238	dotnet/runtime	WasmTestOnBrowser-System.Threading.Tasks.Parallel.Tests.WorkItemExecution	#96841
523904	dotnet/runtime	WasmTestOnBrowser-System.Threading.Tasks.Parallel.Tests.WorkItemExecution	#96618

Summary

24-Hour Hit Count	7-Day Hit Count	1-Month Count
1	4	5

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Failing after enabling all MT library tests (#91536).

[FAIL] System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest
 Task within Parallel.For must have non-negative Id
 Expected: False
 Actual:   True
    at System.Threading.Tasks.Tests.ParallelForTests.<>c.<TaskIDFromExternalContextTest>b__4_1(Int32 i)
    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2[[System.Object, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e],[System.Int32, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
 --- End of stack trace from previous location ---
    at System.Threading.Tasks.Parallel.<>c__DisplayClass19_0`2[[System.Object, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e],[System.Int32, System.Private.CoreLib, Version=9.0.0.0, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]].<ForWorker>b__1(RangeWorker& currentWorker, Int64 timeout, Boolean& replicationDelegateYieldedBeforeCompletion)
    at System.Threading.Tasks.TaskReplicator.Run[RangeWorker](ReplicatableUserAction`1 action, ParallelOptions options, Boolean stopOnFirstFailure)
    at System.Threading.Tasks.Parallel.ForWorker[Object,Int32](Int32 fromInclusive, Int32 toExclusive, ParallelOptions parallelOptions, Action`1 body, Action`2 bodyWithState, Func`4 bodyWithLocal, Func`1 localInit, Action`1 localFinally)
   at System.Threading.Tasks.Parallel.For(Int32 fromInclusive, Int32 toExclusive, Action`1 body)
    at System.Threading.Tasks.Tests.ParallelForTests.TaskIDFromExternalContextTest()
    at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
    at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

Author:	ilonatommy
Assignees:	-
Labels:	arch-wasm, disabled-test, area-Build-mono, area-VM-threading-mono
Milestone:	-

[pavelsavara]

@lambdageek said "maybe we didn't implement Volatile.Read for MT? or maybe the jiterp doesn't implement it?"

cc @kg

[lambdageek]

I'm probably wrong about that 😄

Volatile.Read turns into an indirect load from a volatile local. which turns into interp_emit_memory_barrier which turns into mono_memory_barrier () at runtime.

and that is... __sync_synchronize () which is a compiler intrinsic. According to WebAssembly/tool-conventions#59 I guess that turns into an atomic.fence instruction

[kg]

I don't remember ever implementing atomics in the jiterpreter period

[lambdageek]

yea it looks like MINT_MONO_MEMORY_BARRIER is just an unsupported opcode for the jiterp. so that doesn't enter into it