On Thu, 8 Apr 2021 06:02:35 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:
>> Working on pushing a new version, it will not have addressed all comments >> but many of them. >> I will build and do testing before push, and start deep testing in the >> background. >> >> I also merge with mainline. > > Hi Robbin, > Thank you for the reply and suggestion to help with merge. I made the first > pass through the webrev. The fix looks pretty good in general - nice > simplifications! My estimation is that it should not be a big problem to > merge it with Loom while it can require some effort to do so. I'm going for > another pass to look on details. > Thanks, > Serguei The functions JvmtiSuspendControl::suspend and JvmtiSuspendControl::resume are not really needed anymore. You probably decided to keep them around to simplify merges a little bit. Is that right? ------------- PR: https://git.openjdk.java.net/jdk/pull/3191