RFR: 8331876: JFR: Move file read and write events to java.base [v3]
Erik Gahlin
egahlin at openjdk.org
Thu May 9 12:01:53 UTC 2024
On Thu, 9 May 2024 07:33:22 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:
>> src/java.base/share/classes/sun/nio/ch/FileChannelImpl.java line 78:
>>
>>> 76:
>>> 77: // Flag that determines if file reads/writes should be traced by JFR
>>> 78: private static boolean jfrTracing;
>>
>> Should the force method be changed to test this flag too?
>>
>> I'm also wondering about the transferXXX methods. We might want to think about these for a separate PR as they have more potential to be outliers than the read/write methods.
>
> I think it would be good to use the flag for all events, but I rather do it as separate PR so this is mostly a mechanical change to remove ASM. It makes it easier to track regressions or improvements.
>
> I can file an enhancement for the transferXXX methods.
"JFR: Add file event support for transfer methods"
https://bugs.openjdk.org/browse/JDK-8331995
"JFR: Add boolean check before loading event classes"
https://bugs.openjdk.org/browse/JDK-8331996
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19129#discussion_r1595353143
More information about the core-libs-dev
mailing list