On Tue, 24 Jun 2025 21:15:06 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
> Update SA to know about JfrRecorderThread, which was made a JavaThread in JDK > 25 by [JDK-8352251](https://bugs.openjdk.org/browse/JDK-8352251). > > I'm also fixing ClhsdbJstackWithConcurrentLock, which was also failing with > JFR enabled, but for a different reason (specified heap size was too small). > > Testing (in progress): > > - [x] tier1 ci > - [x] tier1 ci with -XX:StartFlightRecording > - [ ] tier5 ci This looks good except the copyright header update in last file. test/hotspot/jtreg/serviceability/sa/ClhsdbJstackWithConcurrentLock.java line 2: > 1: /* > 2: * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved. This seems to be an incorrect way to update the copyright header. ------------- Marked as reviewed by sspitsyn (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/25960#pullrequestreview-2956718918 PR Review Comment: https://git.openjdk.org/jdk/pull/25960#discussion_r2165838975
