[Wisp] Remove CoroutineSupport Lock in Java and unnecessary function CoroutineMove #189
Comments
nebula-xm
added a commit
to kuaishou-syslab/dragonwell17
that referenced
this issue
Jan 4, 2024
…coroutineMove. Summary: In Java17, we protect double linked list of coroutine in C++. So there is no need to use Locker in Java to protect it. And also, function moveCoroutine is unnecessary, so we should remove it. Refine nmethods_do mark active. Test Plan: all wisp tests Reviewed-by: yulei Issue: dragonwell-project#189
yuleil
pushed a commit
that referenced
this issue
Jan 26, 2024
…coroutineMove. Summary: In Java17, we protect double linked list of coroutine in C++. So there is no need to use Locker in Java to protect it. And also, function moveCoroutine is unnecessary, so we should remove it. Refine nmethods_do mark active. Test Plan: all wisp tests Reviewed-by: yulei Issue: #189
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary:
In Java17, we protect double linked list of coroutine in C++. So there is no need to use Locker in Java to protect it.
And also, function moveCoroutine is unnecessary, so we should remove it.
Test Plan: all wisp tests
Reviewed-by: yulei
The text was updated successfully, but these errors were encountered: