Re: RFR: 8260589: Crash in JfrTraceIdLoadBarrier::load(_jclass*) [v2]

[Denghui Dong]

Hi Markus,

This patch is also worked in JDK 11 but needs some adjustments.
I think a proper review should be done for the backport after this PR is integrated.

Best Regards,
Denghui


------------------------------------------------------------------
From:Markus Grönlund <mgronlun at openjdk.java.net>
Send Time:2021年2月26日(星期五) 23:31
To:hotspot-jfr-dev <hotspot-jfr-dev at openjdk.java.net>
Subject:Re: RFR: 8260589: Crash in JfrTraceIdLoadBarrier::load(_jclass*) [v2]

> Hi Denghui,
> 
> I did not manage to find a solution that would could reduce verbosity, but i did find one, although a bit involved, that lets us keep it local to and integrated in the core of JFR itself.
> 
> Can you do a test run with this suggestion to see if it works for you?
> 
> Thanks
> Markus

Markus Grönlund has updated the pull request incrementally with two additional commits since the last revision:

 - cds hook
 - skip return value

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2738/files
  - new: https://git.openjdk.java.net/jdk/pull/2738/files/2d08073d..ce1ed381

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2738&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2738&range=00-01

  Stats: 6 lines in 2 files changed: 4 ins; 1 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2738.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2738/head:pull/2738

PR: https://git.openjdk.java.net/jdk/pull/2738