<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<span class="new"><b>Background</b>:<br>
ConcurrentGCThread provides incomplete support for an
initialization and termination protocol for GC threads, so missing
parts are duplicated in almost all subclasses.<br>
<br>
<b>Fix</b>:<br>
Move duplicated </span><span class="new">run(), and stop()
methods up from subclasses </span><span class="new">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
ConcurrentG1RefineThread, </span><span class="new">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
ConcurrentMarkThread, G1StringDedupThread, and
G1YoungRemSetSamplingThread to </span><span class="new"><span
class="new">ConcurrentGCThread, as well as declare virtual
methods </span></span><span class="new"><span class="new">run_service()
and stop_service.<br>
<br>
Note that ConcurrentMarkSweepThread is the odd-ball subclass. It
implements it's own termination protocol, it provides it's own
run() and stop() methods, and does not use </span></span><span
class="new"><span class="new">run_service() and stop_service().</span></span><br>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<span class="new"></span><br>
<b>Bug</b>: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8140257">https://bugs.openjdk.java.net/browse/JDK-8140257</a><br>
<b>Webrev</b>:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~drwhite/8140257/webrev.01/">http://cr.openjdk.java.net/~drwhite/8140257/webrev.01/</a><br>
<br>
<b>Tests</b>: jprt, Aurora "nightly" run (I think this is OK)<br>
<a class="moz-txt-link-freetext" href="http://aurora.ru.oracle.com/faces/Batch.xhtml?batchName=1325690.VMSQE.adhoc.JPRT">http://aurora.ru.oracle.com/faces/Batch.xhtml?batchName=1325690.VMSQE.adhoc.JPRT</a>
</body>
</html>