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.

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

Integrate with WebAssembly multithreading #48768

Closed
SteveSandersonMS opened this issue Jun 13, 2023 · 5 comments · Fixed by #52724
Closed

Integrate with WebAssembly multithreading #48768

SteveSandersonMS opened this issue Jun 13, 2023 · 5 comments · Fixed by #52724
Assignees
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly feature-blazor-wasm-threading Pillar: Dev Experience

Comments

@SteveSandersonMS
Copy link
Member

As multithreading is added to the core .NET WebAssembly runtime, we need to ensure this doesn't lead to invalid/dangerous scenarios for Blazor components.

  • Min bar: if the renderer is accessed from the wrong thread, throw
  • Ideal: sync context with semantics equivalent to Blazor Server or WebView, so InvokeAsync works

This is going to come in late so we should take a risk-averse position on how much we attempt to implement here.

@SteveSandersonMS SteveSandersonMS added area-blazor Includes: Blazor, Razor Components feature-full-stack-web-ui Full stack web UI with Blazor labels Jun 13, 2023
@SteveSandersonMS SteveSandersonMS added this to the 8.0-rc1 milestone Jun 13, 2023
@SteveSandersonMS SteveSandersonMS changed the title WebAssembly multithreading: prevent invalid usage of Blazor renderer Integrate with WebAssembly multithreading Jun 13, 2023
@SteveSandersonMS SteveSandersonMS added feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly enhancement This issue represents an ask for new feature or an enhancement to an existing one and removed feature-full-stack-web-ui Full stack web UI with Blazor labels Jun 13, 2023
@pavelsavara
Copy link
Member

Related dotnet/runtime#86759

@mkArtakMSFT
Copy link
Member

@MackinnonBuck, @pavelsavara is there any remaining work here for our team, or should this be closed already?

@pavelsavara
Copy link
Member

this needs core review #48991

@mkArtakMSFT mkArtakMSFT modified the milestones: 8.0-rc1, .NET 9 Planning Jul 14, 2023
@mkArtakMSFT
Copy link
Member

@lewing, @pavelsavara, @maraf at this point, given all the other other critical issues on our plate, I've moved this out of .NET 8 release.

@ghost
Copy link

ghost commented Dec 21, 2023

Thanks for contacting us.

We're moving this issue to the .NET 9 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly feature-blazor-wasm-threading Pillar: Dev Experience
Projects
None yet
4 participants