<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Sounds good. I'll take a stab at it.</div><div><br></div><div>Does initiating the AdaptiveSizePolicy class for G1, or create a subclass G1AdaptiveSizePolicy sound like an acceptable high-level approach?</div><div>I know it is a non-goal to make UseAdaptiveSizePolicy work for G1, but most logic for UseGCOverheadLimit is in the AdaptiveSizePolicy class.<br></div><div>This seems the approach that reuse most existing code and does not affect ParallelGC.<br></div><div><br></div>A side story is that we have a Google-local patch since JDK6 that makes UseGCOverheadLimit work properly for CMS.<div>UseAdaptiveSizePolicy doesn't work and is disabled by default for CMS, but UseGCOverheadLimit depends on logic in the CMSAdaptiveSizePolicy class (which was removed in JDK9) to properly collect major GC overhead.</div><div><br></div><div>-Man<br></div><br></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 12, 2018 at 12:10 AM Thomas Schatzl <<a href="mailto:thomas.schatzl@oracle.com">thomas.schatzl@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Man,<br>
<br>
On Thu, 2018-10-11 at 14:51 -0700, Man Cao wrote:<br>
> Hi all,<br>
> <br>
> I noticed that UseGCOverheadLimit does not really work for G1.<br>
> I created an RFE: <a href="https://bugs.openjdk.java.net/browse/JDK-8212084" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8212084</a><br>
> <br>
> I also search JBS and mail archive, but didn't find any plan to<br>
> implement UseGCOverheadLimit for G1. Is this a desirable feature to<br>
> be implemented in the future?<br>
> <br>
<br>
  if it is something you need, feel free to implement. I see no<br>
particular reason why it shouldn't be supported.<br>
<br>
Thanks,<br>
  Thomas<br>
<br>
<br>
</blockquote></div>