RFR: 8359593: JFR: Instrumentation of java.lang.String corrupts recording [v2]

ExE Boss duke at openjdk.org
Sun Jun 15 21:58:28 UTC 2025


On Sun, 15 Jun 2025 18:41:18 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

>> Could I have review of PR that excludes problematic String-related methods when tracing java.lang.String.
>> 
>> Testing: jdk/jdk/jfr
>> 
>> Thanks
>> Erik
>
> Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add missing copyright header

src/jdk.jfr/share/classes/jdk/jfr/internal/tracing/ExcludeList.java line 38:

> 36:         "jdk/internal/misc/Unsafe",
> 37:         "java/lang/StringLatin1",
> 38:         "java/lang/StringUTF16"

**Java** array initialiser expressions can have trailing commas:
Suggestion:

        "java/lang/StringUTF16",

(wish the same applied to method parameters)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25814#discussion_r2148837431


More information about the hotspot-jfr-dev mailing list