RFR: 8318682: SA decoding of scalar replaced objects is broken [v5]

[Tom Rodriguez]

On Wed, 17 Jan 2024 19:52:47 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> No.  ClhsdbTestAllocationMerge is using `jstack -v` which doesn't work with ZGC because the `-v` part needs to iterate the heap looking for AbstractQueuedSynchronizers which doesn't work with ZGC and the SA.
>
> It's not -v that is making it do that. It does it without -v also. It can be a very slow operation, and I recently was running into cases where even after 10 minutes it was not done, so I gave up waiting. I'm about to file a CR to make it so it does not do that by default, and add a -l argument to optionally enable it. This will make it consistent with what bin/jstack is doing. I already have the fix ready. Just need to get the CR filed and and publish the PR.

Ok.  I was assuming that the `-v` option was like the `-l` option.  Anyway, TestDebugInfoDecode doesn't use jstack so it doesn't need the problem list entry.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17407#discussion_r1456412077