RFR: 8312293: SIGSEGV in jfr.internal.event.EventWriter.putUncheckedByte after JDK-8312086
Greetings, please help review this fix. The problem description and fix are described in detail in the JIRA issue. Testing: jdk_jfr, microbenchmarks Thanks Markus ------------- Commit messages: - 8312293 Changes: https://git.openjdk.org/jdk/pull/14962/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14962&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8312293 Stats: 3 lines in 2 files changed: 1 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk/pull/14962.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/14962/head:pull/14962 PR: https://git.openjdk.org/jdk/pull/14962
On Thu, 20 Jul 2023 19:59:50 GMT, Markus Grönlund <mgronlun@openjdk.org> wrote:
Greetings,
please help review this fix. The problem description and fix are described in detail in the JIRA issue.
Testing: jdk_jfr, microbenchmarks
Thanks Markus
Looks reasonable. ------------- Marked as reviewed by egahlin (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/14962#pullrequestreview-1540079337
On Thu, 20 Jul 2023 19:59:50 GMT, Markus Grönlund <mgronlun@openjdk.org> wrote:
Greetings,
please help review this fix. The problem description and fix are described in detail in the JIRA issue.
Testing: jdk_jfr, microbenchmarks
Thanks Markus
This pull request has now been integrated. Changeset: 59f66a3b Author: Markus Grönlund <mgronlun@openjdk.org> URL: https://git.openjdk.org/jdk/commit/59f66a3b839c5b77c2770fa0bdd76f399d2d86d6 Stats: 3 lines in 2 files changed: 1 ins; 0 del; 2 mod 8312293: SIGSEGV in jfr.internal.event.EventWriter.putUncheckedByte after JDK-8312086 Reviewed-by: egahlin ------------- PR: https://git.openjdk.org/jdk/pull/14962
participants (2)
-
Erik Gahlin
-
Markus Grönlund