RFR: 8360312: Serviceability Agent tests fail with JFR enabled due to unknown thread type JfrRecorderThread

[David Holmes]

On Tue, 24 Jun 2025 21:15:06 GMT, Chris Plummer <cjplummer at 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