[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    RE: [8u] RFR: 8173361: various crashes in JvmtiExport::post_compiled_method_load
From:       hedongbo <hedongbo () huawei ! com>
Date:       2021-12-30 3:58:13
Message-ID: 348b7a3fcfef4d66bb8f0a4c085e20d9 () huawei ! com
[Download RAW message or body]

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@openjdk.java.net>
Cc: 'serviceability-dev@openjdk.java.net' <serviceability-dev@openjdk.java.=
net>
Subject: [8u] RFR: 8173361: various crashes in JvmtiExport::post_compiled_m=
ethod_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 ex=
ist in 8.
I added the CLDClosure* to ServiceThread::oops_do because JDK- 8154580[3] d=
oes 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





[Attachment #3 (text/html)]

<html xmlns:v="urn:schemas-microsoft-com:vml" \
xmlns:o="urn:schemas-microsoft-com:office:office" \
xmlns:w="urn:schemas-microsoft-com:office:word" \
xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" \
xmlns="http://www.w3.org/TR/REC-html40"> <head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
	{font-family:"Cambria Math";
	panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
	{font-family:DengXian;
	panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
	{font-family:DengXian;
	panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0cm;
	margin-bottom:.0001pt;
	text-align:justify;
	font-size:11.0pt;
	font-family:DengXian;}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:#0563C1;
	text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
	{mso-style-priority:99;
	color:#954F72;
	text-decoration:underline;}
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
	{mso-style-priority:99;
	mso-style-link:"\7EAF\6587\672C \5B57\7B26";
	margin:0cm;
	margin-bottom:.0001pt;
	font-size:11.0pt;
	font-family:"Calibri",sans-serif;}
p.msonormal0, li.msonormal0, div.msonormal0
	{mso-style-name:msonormal;
	mso-margin-top-alt:auto;
	margin-right:0cm;
	mso-margin-bottom-alt:auto;
	margin-left:0cm;
	font-size:11.0pt;
	font-family:"Calibri",sans-serif;}
span.a
	{mso-style-name:"\7EAF\6587\672C \5B57\7B26";
	mso-style-priority:99;
	mso-style-link:\7EAF\6587\672C;
	font-family:"Calibri",sans-serif;}
span.EmailStyle20
	{mso-style-type:personal;
	font-family:DengXian;
	color:windowtext;}
span.EmailStyle21
	{mso-style-type:personal-reply;
	font-family:DengXian;
	color:windowtext;}
.MsoChpDefault
	{mso-style-type:export-only;
	font-size:10.0pt;}
@page WordSection1
	{size:612.0pt 792.0pt;
	margin:72.0pt 90.0pt 72.0pt 90.0pt;}
div.WordSection1
	{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal">This problem has affected the customer's use at present. Can \
anyone help to take a look? Thanks.<o:p></o:p></p> <p \
class="MsoNormal"><o:p>&nbsp;</o:p></p> <p class="MsoNormal">Thanks,<o:p></o:p></p>
<p class="MsoNormal">hedongbo<o:p></o:p></p>
<p class="MsoNormal"><o:p>&nbsp;</o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal" align="left" style="text-align:left"><b><span \
style="font-family:&quot;Calibri&quot;,sans-serif">From:</span></b><span \
style="font-family:&quot;Calibri&quot;,sans-serif"> hedongbo <br>
<b>Sent:</b> Friday, December 17, 2021 9:35 AM<br>
<b>To:</b> jdk8u-dev &lt;jdk8u-dev@openjdk.java.net&gt;<br>
<b>Cc:</b> 'serviceability-dev@openjdk.java.net' \
&lt;serviceability-dev@openjdk.java.net&gt;<br> <b>Subject:</b> [8u] RFR: 8173361: \
various crashes in JvmtiExport::post_compiled_method_load<o:p></o:p></span></p> \
</div> </div>
<p class="MsoNormal" align="left" style="text-align:left"><o:p>&nbsp;</o:p></p>
<p class="MsoPlainText">Hi,<o:p></o:p></p>
<p class="MsoNormal"><o:p>&nbsp;</o:p></p>
<p class="MsoNormal" style="text-indent:12.0pt">Please review the backport of \
JDK-8173361 to 8u.<o:p></o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt">Bug: <a \
href="https://bugs.openjdk.java.net/browse/JDK-8173361"> \
https://bugs.openjdk.java.net/browse/JDK-8173361</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt">11u commit: <a \
href="https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/99bdef096320"> \
https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/99bdef096320</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt">8u webrev: <a \
href="https://cr.openjdk.java.net/~dongbohe/8173361/webrev.00/"> \
https://cr.openjdk.java.net/~dongbohe/8173361/webrev.00/</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt">This patch doesn't apply \
cleanly.<o:p></o:p></p> <p class="MsoNormal" style="text-indent:12.0pt">I turned \
NoSafepointVerifier to No_Safepoint_Verifier because JDK- 8146690[1] changed the \
naming convention.<o:p></o:p></p> <p class="MsoNormal" style="text-indent:12.0pt">I \
removed the mutexLocker.cpp changeset because JDK-8047290[2] &nbsp;does not exist in \
8.<o:p></o:p></p> <p class="MsoNormal" style="text-indent:12.0pt">I added the \
CLDClosure* to ServiceThread::oops_do because JDK- 8154580[3] does not exist in \
8.<o:p></o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt">Tested with tier1. No regression in tests.<o:p></o:p></p> \
<p class="MsoNormal" style="text-indent:12.0pt">Follow-up fix JDK-8235218[4] is \
planned to be backported as well.<o:p></o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt">[1] <a \
href="https://bugs.openjdk.java.net/browse/JDK-8146690"> \
https://bugs.openjdk.java.net/browse/JDK-8146690</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt">[2] <a \
href="https://bugs.openjdk.java.net/browse/JDK-8047290"> \
https://bugs.openjdk.java.net/browse/JDK-8047290</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt">[3] <a \
href="https://bugs.openjdk.java.net/browse/JDK-8154580"> \
https://bugs.openjdk.java.net/browse/JDK-8154580</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt">[4] <a \
href="https://bugs.openjdk.java.net/browse/JDK-8235218"> \
https://bugs.openjdk.java.net/browse/JDK-8235218</a><o:p></o:p></p> <p \
class="MsoNormal" style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p \
class="MsoPlainText">Thanks,<o:p></o:p></p> <p \
class="MsoPlainText">hedongbo<o:p></o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p class="MsoNormal" \
style="text-indent:12.0pt"><o:p>&nbsp;</o:p></p> <p \
class="MsoNormal">&nbsp;&nbsp;&nbsp; <o:p></o:p></p> </div>
</body>
</html>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic