You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
release all those proxies when we detach the worker.
for the proxies of objects, we would reset the GCHandle/JSHandle to -1 and if some stray setTimeout callback uses it, we will throw.
for the proxies of functions, mark it disposed and it would throw if you try to call it.
for the proxies of Task/Promise
same for proxies in the other direction.
at the end of detach, we should assert that have empty FinalizationRegistry of that worker.
at the end of detach, we should assert that s_csOwnedObjects is empty
we need a JS callback or event when a pthread detaches
the FinalizationRegistry callback invokes managed code, so we need to check when is the last moment when the thread is still attached enough that we can run some managed code.
The text was updated successfully, but these errors were encountered:
Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.
Issue Details
release all those proxies when we detach the worker.
for the proxies of objects, we would reset the GCHandle/JSHandle to -1 and if some stray setTimeout callback uses it, we will throw.
for the proxies of functions, mark it disposed and it would throw if you try to call it.
for the proxies of Task/Promise
same for proxies in the other direction.
at the end of detach, we should assert that have empty FinalizationRegistry of that worker.
at the end of detach, we should assert that s_csOwnedObjects is empty
we need a JS callback or event when a pthread detaches
the FinalizationRegistry callback invokes managed code, so we need to check when is the last moment when the thread is still attached enough that we can run some managed code.
ghost
added
the
in-pr
There is an active PR which will close this issue when it is merged
label
Jun 26, 2023
ghost
added
in-pr
There is an active PR which will close this issue when it is merged
and removed
in-pr
There is an active PR which will close this issue when it is merged
labels
Jul 11, 2023
release all those proxies when we detach the worker.
The text was updated successfully, but these errors were encountered: