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

Fix possible dead lock in wasm_cluster_spawn_exec_env #2882

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

wenyongh
Copy link
Contributor

@wenyongh wenyongh commented Dec 8, 2023

Fix a deadlock issue like:

wasm_runtime_spawn_exec_env
wasm_cluster_spawn_exec_env (hold cluster->lock)
wasm_runtime_instantiate_internal
wasm_instantiate
execute_post_instantiate_functions
wasm_exec_env_set_module_inst (grab the lock again)

Refer to #2879.

@wenyongh wenyongh merged commit b008ab4 into bytecodealliance:main Dec 8, 2023
383 checks passed
yamt added a commit to yamt/wasm-micro-runtime that referenced this pull request Dec 8, 2023
this would expose the deadlock bug, which has been fixed by
bytecodealliance#2882
wenyongh pushed a commit that referenced this pull request Dec 8, 2023
this would expose the deadlock bug, which has been fixed by
#2882
@wenyongh wenyongh deleted the fix_grab_cluster_lock branch December 17, 2023 08:45
@yamt yamt mentioned this pull request Dec 28, 2023
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
…ce#2882)

Fix a deadlock issue like:

wasm_runtime_spawn_exec_env
wasm_cluster_spawn_exec_env (hold cluster->lock)
wasm_runtime_instantiate_internal
wasm_instantiate
execute_post_instantiate_functions
wasm_exec_env_set_module_inst (grab the lock again)

Refer to bytecodealliance#2879.
victoryang00 pushed a commit to victoryang00/wamr-aot-gc-checkpoint-restore that referenced this pull request May 27, 2024
this would expose the deadlock bug, which has been fixed by
bytecodealliance#2882
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants