RFR: JDK-8302320: AsyncGetCallTrace obtains too few frames in sanity test [v3]

Richard Reingruber rrich at openjdk.org
Mon Feb 20 09:11:27 UTC 2023


On Mon, 20 Feb 2023 09:04:39 GMT, Johannes Bechberger <duke at openjdk.org> wrote:

>> The sanity test worked :)
>
> ... and no regression in the other serviceability tests.

Probably it's better to adapt the limit though (sorry).
Problem with this version is that it allows `unextended_sp` outside the address range reserved for the stack. Let's see what other reviewers think.
Suggestion:

  if (!thread->is_in_stack_range_incl(unextended_sp, sp - Interpreter::stackElementSize)) {

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

PR: https://git.openjdk.org/jdk/pull/12535


More information about the serviceability-dev mailing list