Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[browser][MT] Assertion: should not be reached at interp/transform.c:9055 #98758

Open
pavelsavara opened this issue Feb 21, 2024 · 2 comments
Open

[browser][MT] Assertion: should not be reached at interp/transform.c:9055 #98758

pavelsavara opened this issue Feb 21, 2024 · 2 comments
Assignees
@BrzVlad
Labels
arch-wasm WebAssembly architecture area-Codegen-Interpreter-mono area-VM-threading-mono disabled-test The test is disabled in source code against the issue os-browser Browser variant of arch-wasm
Milestone
10.0.0

Comments

@pavelsavara
Copy link
Member

pavelsavara commented Feb 21, 2024 •
edited
Loading

Log

[MONO] * Assertion at D:/a/_work/1/s/src/mono/mono/mini/interp/transform.c:9055, condition `<disabled>' not met

Or locally I also saw

Assertion: should not be reached at C:/Dev/runtime/src/mono/mono/mini/interp/transform.c:347

This is test in src\mono\wasm\Wasm.Build.Tests\Blazor\SimpleMultiThreadedTests.cs

  • Release & multi-threading build of the runtime
  • publish Blazor app with <WasmEnableThreads>true</WasmEnableThreads>
  • Debug version of the app, no AOT.
  • So far we only saw it fail on Windows. Both FF and chrome.
  • Re-link with symbols hides/solves the problem.

Stack trace doesn't help
image

We know this is trying to load assembly Microsoft.AspNetCore.Components.WebAssembly
@pavelsavara pavelsavara added arch-wasm WebAssembly architecture area-Codegen-Interpreter-mono os-browser Browser variant of arch-wasm labels Feb 21, 2024
@pavelsavara pavelsavara added this to the 9.0.0 milestone Feb 21, 2024
@ghost
Copy link

ghost commented Feb 21, 2024

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

Issue Details

Log

[MONO] * Assertion at D:/a/_work/1/s/src/mono/mono/mini/interp/transform.c:9055, condition `<disabled>' not met

This is test in src\mono\wasm\Wasm.Build.Tests\Blazor\SimpleMultiThreadedTests.cs

  • Release & multi-threading build of the runtime
  • publish Blazor app with <WasmEnableThreads>true</WasmEnableThreads>
  • Debug version of the app, no AOT.
  • So far we only saw it fail on Windows. Both FF and chrome.
  • Re-link with symbols hides/solves the problem.

Stack trace doesn't help
image

We know this is trying to load assembly Microsoft.AspNetCore.Components.WebAssembly

Author: pavelsavara
Assignees: BrzVlad
Labels:

arch-wasm, area-Codegen-Interpreter-mono, os-browser
Milestone: 9.0.0

@pavelsavara pavelsavara changed the title [browser][MT] Assertion: should not be reached at interp/transform.c:347 [browser][MT] Assertion: should not be reached at interp/transform.c:9055 Feb 21, 2024
pavelsavara added a commit to pavelsavara/runtime that referenced this issue Feb 21, 2024
@pavelsavara
disable failing test ActiveIssue dotnet#98758
250c8e6
@pavelsavara pavelsavara mentioned this issue Feb 21, 2024
Merged
@pavelsavara
Copy link
Member Author

This is the local fail on my machine, the value of type->type is 153
image

@lewing lewing modified the milestones: 9.0.0, 10.0.0 Jul 30, 2024
@lewing lewing added the disabled-test The test is disabled in source code against the issue label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BrzVlad BrzVlad

Labels
arch-wasm WebAssembly architecture area-Codegen-Interpreter-mono area-VM-threading-mono disabled-test The test is disabled in source code against the issue os-browser Browser variant of arch-wasm
Projects
None yet
Milestone
10.0.0
Development

No branches or pull requests
3 participants
@lewing @pavelsavara @BrzVlad