Integrated: 8337165: Test jdk/jfr/event/gc/stacktrace/TestG1YoungAllocationPendingStackTrace.java failed: IndexOutOfBoundsException: Index 64 out of bounds for length 64

[Erik Gahlin]

On Fri, 26 Jul 2024 17:23:32 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

> Could I have a review of a test fix that increases the stack depth for the AllocationRequiringGC event?
> 
> The problem occurs if the number of frames JFR reports is 64, which can happen with lots of lambda frames. The test then finds a match at the last frame (index 63) and tries to match against the next frame, which leads to an IndexOutOfBoundsException.
> 
> By increasing the stack depth to 256, the stack trace will not be truncated, and the matching of two consecutive frames can happen.
> 
> Testing: 100 *  jdk/jfr/event/gc/stacktrace/*
> 
> Thanks
> Erik

This pull request has now been integrated.

Changeset: 9124a94e
Author:    Erik Gahlin <egahlin at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/9124a94e383f5bc6a3376eecc97ee8bd22f9e420
Stats:     31 lines in 10 files changed: 20 ins; 1 del; 10 mod

8337165: Test jdk/jfr/event/gc/stacktrace/TestG1YoungAllocationPendingStackTrace.java failed: IndexOutOfBoundsException: Index 64 out of bounds for length 64

Reviewed-by: phh

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

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