Integrated: 8298129: Let checkpoint event sizes grow beyond u4 limit

Markus Grönlund mgronlun at openjdk.org
Thu Dec 8 12:54:27 UTC 2022


On Thu, 8 Dec 2022 10:55:07 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

> Greetings,
> 
> This change pre-calculates the event size for checkpoint / constant pool events (event id 1) before serialization to disk and circumvents the need to reserve a limited checkpoint event size up front, to be later filled in, using a padded write of the actual size. It lets us represent small checkpoint events more effectively, but more importantly, it allows for larger checkpoint / constant pool events because the event size pre-reservation no longer restricts them. The parser side will continue to read checkpoint event sizes as varints, but will expose the checkpoint event size as a long instead of an int.
> 
> Testing: jdk_jfr, stress testing
> 
> Thanks
> Markus

This pull request has now been integrated.

Changeset: ea108f50
Author:    Markus Grönlund <mgronlun at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/ea108f504ccb63fc9651e804e3bbba1c108dcead
Stats:     139 lines in 6 files changed: 84 ins; 17 del; 38 mod

8298129: Let checkpoint event sizes grow beyond u4 limit

Reviewed-by: egahlin

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

PR: https://git.openjdk.org/jdk/pull/11584


More information about the hotspot-jfr-dev mailing list