[8u] RFR: 8173361: various crashes in JvmtiExport::post_compiled_method_load

[hedongbo]

Hi 8u-maintainers,

Excuse me, can this fix be merged into 8u332 or 8u342?

Thanks,
Hedongbo


-----Original Message-----
From: hedongbo 
Sent: Wednesday, January 5, 2022 10:20 AM
To: 'Serguei Spitsyn' <serguei.spitsyn at oracle.com>; jdk8u-dev <jdk8u-dev at openjdk.java.net>
Cc: serviceability-dev at openjdk.java.net
Subject: RE: [8u] RFR: 8173361: various crashes in JvmtiExport::post_compiled_method_load

Thank you for your review, Serguei.

I have added jdk8u-fix-request label in JDK-8173361[1], and now I need to wait for an 8u maintainer to add jdk8u-fix-yes.
After that, the change can be commited by the committer, the recommended process is on the wiki of jdk8u#Contributing[2].

[1] https://bugs.openjdk.java.net/browse/JDK-8146690
[2] https://wiki.openjdk.java.net/display/jdk8u/Main

Thanks,
Hedongbo

-----Original Message-----
From: Serguei Spitsyn <serguei.spitsyn at oracle.com> 
Sent: Wednesday, January 5, 2022 7:05 AM
To: hedongbo <hedongbo at huawei.com>; jdk8u-dev <jdk8u-dev at openjdk.java.net>
Cc: serviceability-dev at openjdk.java.net
Subject: Re: [8u] RFR: 8173361: various crashes in JvmtiExport::post_compiled_method_load

Hi Hedongbo,

The backport looks okay to me.
But I do not see where I can mark it as approved.
Do you have a PR (Pool Request) or a backport bug number?

Thanks,
Serguei


On 12/29/21, 7:58 PM, "jdk8u-dev on behalf of hedongbo" <jdk8u-dev-retn at openjdk.java.net on behalf of hedongbo at huawei.com> wrote:

    This problem has affected the customer's use at present. Can anyone help to take a look? Thanks.
    
    Thanks,
    hedongbo
    
    From: hedongbo
    Sent: Friday, December 17, 2021 9:35 AM
    To: jdk8u-dev <jdk8u-dev at openjdk.java.net>
    Cc: 'serviceability-dev at openjdk.java.net' <serviceability-dev at openjdk.java.net>
    Subject: [8u] RFR: 8173361: various crashes in JvmtiExport::post_compiled_method_load
    
    
    Hi,
    
    Please review the backport of JDK-8173361 to 8u.
    Bug: https://bugs.openjdk.java.net/browse/JDK-8173361
    11u commit: https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/99bdef096320
    8u webrev: https://cr.openjdk.java.net/~dongbohe/8173361/webrev.00/
    
    This patch doesn't apply cleanly.
    I turned NoSafepointVerifier to No_Safepoint_Verifier because JDK- 8146690[1] changed the naming convention.
    I removed the mutexLocker.cpp changeset because JDK-8047290[2]  does not exist in 8.
    I added the CLDClosure* to ServiceThread::oops_do because JDK- 8154580[3] does not exist in 8.
    
    Tested with tier1. No regression in tests.
    Follow-up fix JDK-8235218[4] is planned to be backported as well.
    
    [1] https://bugs.openjdk.java.net/browse/JDK-8146690
    [2] https://bugs.openjdk.java.net/browse/JDK-8047290
    [3] https://bugs.openjdk.java.net/browse/JDK-8154580
    [4] https://bugs.openjdk.java.net/browse/JDK-8235218
    
    
    Thanks,
    
    hedongbo