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

[Severin Gehwolf]

Hi,

On Thu, 2022-03-17 at 08:16 +0000, hedongbo wrote:
> Hi 8u-maintainers,
> 
> Excuse me, can this fix be merged into 8u332 or 8u342?

At this point you need to create a git PR at:
https://github.com/openjdk/jdk8u-dev/ and get it integrated for 8u342
that way. 8u332 is in rampdown and this fix doesn't qualify as a
critical fix.

Thanks,
Severin

> 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
>     
>     
>     
>     
>     
>