RFR: 8282477: [x86, aarch64] vmassert(_last_Java_pc == NULL, "already walkable"); fails with async profiler [v3]

Johannes Bechberger duke at openjdk.java.net
Fri Apr 29 11:07:45 UTC 2022


On Fri, 29 Apr 2022 10:58:28 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>>> Though why is the thread passed in as it is unused except in the assert??
>> 
>> I don't know. Removing it would clean up the code.
>
> Right, I also noticed that. Unneeded `Thread::current()` calls are not nice, either. If we can live without the assertion I'd like to see the thread parameter go away.

I removed the assertion.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8209


More information about the hotspot-dev mailing list