RFR: 8348347: Cleanup JavaThread subclass support in SA [v2]

Serguei Spitsyn sspitsyn at openjdk.org
Tue Feb 11 23:14:18 UTC 2025


On Thu, 6 Feb 2025 22:44:48 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Cleanup SA JavaThread support. Details in first comment:
>> 
>> Testing:
>> - Tier1
>> - Tier2 svc
>> - Tier3
>> - Tier5 svc
>> - Local testing of debuggee using graal java compiler threads. Verified that the compiler threads shows up in jstack output if the property is set.
>
> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
> 
>   update comments

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/runtime/JavaThread.java line 54:

> 52:   private static long oopPtrSize;
> 53: 
> 54:   // For accessing platform dependent functionality

Nit: It is better to have dot at the end of comment.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23456#discussion_r1951735651


More information about the serviceability-dev mailing list