[jdk11u-dev] RFR: 8298129: Let checkpoint event sizes grow beyond u4 limit
Ekaterina Vergizova
evergizova at openjdk.org
Mon Feb 6 18:12:36 UTC 2023
I'd like to backport JDK-8298129 to 11u.
It fixes corrupted jfr dumps that can be generated for applications with a large stack trace pool or string pool. Such dumps can not be read by jfr tool or JMC.
17u patch doesn't apply cleanly, minor modifications required:
- jfrCheckpointManager.cpp: is_flushpoint and flushpoint_offset were replaced by checkpoint_type and checkpoint_type_offset by JDK-8226511 that is not backported to 11u. Adapted the changes for flushpoint. Additionally nullptr was replaced with NULL since JDK-8246032 is not backported to 11u
- jfrEncoders.hpp, jfrWriterHost.hpp, jfrWriterHost.inline.hpp: copyright years, minor context difference due to #endif comment, reapplied manually
- ChunkParser.java: minor context difference for the second hunk since JDK-8226511 is not backported to 11u, reapplied manually
Tested with tier1 and jdk/jfr tests.
-------------
Commit messages:
- Backport ea108f504ccb63fc9651e804e3bbba1c108dcead
Changes: https://git.openjdk.org/jdk11u-dev/pull/1708/files
Webrev: https://webrevs.openjdk.org/?repo=jdk11u-dev&pr=1708&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8298129
Stats: 114 lines in 6 files changed: 80 ins; 12 del; 22 mod
Patch: https://git.openjdk.org/jdk11u-dev/pull/1708.diff
Fetch: git fetch https://git.openjdk.org/jdk11u-dev pull/1708/head:pull/1708
PR: https://git.openjdk.org/jdk11u-dev/pull/1708
More information about the jdk-updates-dev
mailing list