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 Would a simple forward declaration of `class JfrRecordThread` in vmStructs.cpp avoid the need to move the class definition to the header file? Otherwise fix looks good. Thanks ------------- Marked as reviewed by dholmes (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/25960#pullrequestreview-2956528984
