<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p>Hi David,</p>
<p>Here is a link to stalled code review where I wanted to fix
problem by avoiding globals. <br>
</p>
<p>What is your opinion?</p>
<p><br>
</p>
<p>Thank you,</p>
<p>Alex<br>
</p>
<p><br>
</p>
<div class="moz-forward-container"><br>
<br>
-------- Forwarded Message --------
<table class="moz-email-headers-table" border="0" cellpadding="0"
cellspacing="0">
<tbody>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Subject:
</th>
<td>Request for review JDK-8169634 - Logging fails with
'pure virtual method called'</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Date: </th>
<td>Tue, 20 Dec 2016 17:16:10 -0500</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">From: </th>
<td>Alexander Harlap <a class="moz-txt-link-rfc2396E" href="mailto:alexander.harlap@oracle.com"><alexander.harlap@oracle.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Organization:
</th>
<td>Oracle Corporation</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">To: </th>
<td><a class="moz-txt-link-abbreviated" href="mailto:hotspot-gc-dev@openjdk.java.net">hotspot-gc-dev@openjdk.java.net</a></td>
</tr>
</tbody>
</table>
<br>
<br>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<p>Please review suggested change for <a moz-do-not-send="true"
class="issue-link" data-issue-key="JDK-8169634"
href="https://bugs.openjdk.java.net/browse/JDK-8169634"
id="key-val" rel="4906770">JDK-8169634</a> - Logging fails
with 'pure virtual method called'</p>
<p>Change is located at <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Eaharlap/8169634/webrev.00/">http://cr.openjdk.java.net/~aharlap/8169634/webrev.00/</a></p>
<p>Reported problem points to access virtual method of of
statically initialized object MarkSweep::follow_root_closure
after calling global destructor for that object (from exit()).</p>
<p>Currently AllStatic class MarkSweep declares few statically
allocated closures and Stack<..> objects.</p>
<p>Suggested change converts MarkSweep class and all its closures
and stacks<..> to be the stack allocated objects.</p>
<p>Alex<br>
</p>
<p><br>
</p>
<p><br>
</p>
</div>
</body>
</html>