<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi all,<br>
<br>
Please, review this patch to remove the do_code_roots parameter from
SharedHeap::process_strong_roots.<br>
<br>
The changes done are:<br>
- Change the code to rely on the ScannningOption so parameter
instead of do_code_roots.<br>
- Change GenMarkSweep and G1MarkSweep to adjust the code roots with
the help of process_strong_roots instead of doing it as a separate
phase after process_strong_roots.<br>
- Removed the unused
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
FalseClosure.<br>
<br>
After this patch the adjust phase of the GenMarkSweep and
G1MarkSweep will use the generic code in process_strong_roots, which
mark/claim the nmethods before they are processed. Before the patch
these two Serial Old GC adjust phases skipped the mark/claim part.
No noticeable Serial Old GC time increases were found when this
patch was performance tested.<br>
<br>
This cleanup is needed/wanted for G1 Class Unloading.<br>
<br>
Webrev:<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~stefank/8034761/webrev.00/">http://cr.openjdk.java.net/~stefank/8034761/webrev.00/</a><br>
<br>
RFE:<br>
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8034761">https://bugs.openjdk.java.net/browse/JDK-8034761</a><br>
<br>
thanks,<br>
StefanK<br>
</body>
</html>