<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">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/~coleenp/8199007.01/webrev" rel="noreferrer" target="_blank">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">https://bugs.openjdk.java.net/browse/JDK-8199007</a><br>
<br>
Thanks,<br>
Coleen<br>
</blockquote></div></div>