<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Thanks Thomas.<br>
Coleen<br>
<br>
<div class="moz-cite-prefix">On 3/23/18 11:02 AM, Thomas Stüfe
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAA-vtUzRnCHuXeX5uaBORXEcBuq4TwZeNhVWvkL63JFO6d4A9w@mail.gmail.com">
<div>
<div dir="auto">Hi Coleen,</div>
<div dir="auto"><br>
</div>
<div dir="auto">This looks good.</div>
<div dir="auto"><br>
</div>
<div dir="auto">Thanks, Thomas </div>
<div dir="auto"><br>
</div>
<br>
<div class="gmail_quote">
<div>On Fri 23. Mar 2018 at 15:29, <<a
href="mailto:coleen.phillimore@oracle.com"
moz-do-not-send="true">coleen.phillimore@oracle.com</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">Summary:
The function isn't needed any longer.<br>
<br>
I believe the function existed from an earlier day when<br>
Metaspace::contains walked the CLDG, so you needed a
function to walk<br>
the unloaded list as well.<br>
<br>
Tested with Per's GC test suite and mach5 tier1-5 (until it
failed in<br>
infra for some platforms), and my own local testing like:<br>
<br>
jtreg -exclude:open/test/hotspot/jtreg/ProblemList.txt<br>
open/test/hotspot/jtreg:hotspot_all_no_apps<br>
<br>
(and the closed jtreg tests too).<br>
<br>
open webrev at <a
href="http://cr.openjdk.java.net/%7Ecoleenp/8199007.01/webrev"
rel="noreferrer" target="_blank" moz-do-not-send="true">http://cr.openjdk.java.net/~coleenp/8199007.01/webrev</a><br>
bug link <a
href="https://bugs.openjdk.java.net/browse/JDK-8199007"
rel="noreferrer" target="_blank" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8199007</a><br>
<br>
Thanks,<br>
Coleen<br>
</blockquote>
</div>
</div>
</blockquote>
<br>
</body>
</html>