<div dir="ltr">Hello,<div><br><div>This is a friendly ping. Could anyone review or sponsor this change? It's just a two-liner change.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-Man</div></div></div>
<br><div class="gmail_quote">On Thu, Nov 30, 2017 at 2:03 PM, Man Cao <span dir="ltr"><<a href="mailto:manc@google.com" target="_blank">manc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I realized that the email attachment is probably dropped by the mailing list, so below is the inlined patch.<div><br></div><div><div><font face="monospace, monospace">--- old/src/hotspot/share/memory/<wbr>metaspace.cpp<span style="white-space:pre-wrap">  </span>2017-11-29 14:56:59.017118444 -0800</font></div><div><font face="monospace, monospace">+++ new/src/hotspot/share/memory/<wbr>metaspace.cpp<span style="white-space:pre-wrap">      </span>2017-11-29 14:56:58.657121375 -0800</font></div><div><font face="monospace, monospace">@@ -3321,9 +3321,6 @@</font></div><div><font face="monospace, monospace">   MinMetaspaceExpansion = align_down_bounded(<wbr>MinMetaspaceExpansion, _commit_alignment);</font></div><div><font face="monospace, monospace">   MaxMetaspaceExpansion = align_down_bounded(<wbr>MaxMetaspaceExpansion, _commit_alignment);</font></div><div><font face="monospace, monospace"> </font></div><div><font face="monospace, monospace">-  CompressedClassSpaceSize = align_down_bounded(<wbr>CompressedClassSpaceSize, _reserve_alignment);</font></div><div><font face="monospace, monospace">-  set_compressed_class_space_<wbr>size(CompressedClassSpaceSize)<wbr>;</font></div><div><font face="monospace, monospace">-</font></div><div><font face="monospace, monospace">   // Initial virtual space size will be calculated at global_initialize()</font></div><div><font face="monospace, monospace">   size_t min_metaspace_sz =</font></div><div><font face="monospace, monospace">       VIRTUALSPACEMULTIPLIER * InitialBootClassLoaderMetaspac<wbr>eSize;</font></div><div><font face="monospace, monospace">@@ -3341,6 +3338,8 @@</font></div><div><font face="monospace, monospace">                   min_metaspace_sz);</font></div><div><font face="monospace, monospace">   }</font></div><div><font face="monospace, monospace"> </font></div><div><font face="monospace, monospace">+  CompressedClassSpaceSize = align_down_bounded(<wbr>CompressedClassSpaceSize, _reserve_alignment);</font></div><div><font face="monospace, monospace">+  set_compressed_class_space_<wbr>size(CompressedClassSpaceSize)<wbr>;</font></div><div><font face="monospace, monospace"> }</font></div><div><font face="monospace, monospace"> </font></div><div><font face="monospace, monospace"> void Metaspace::global_initialize() {</font></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Best,<br clear="all"><div><div class="m_5162284286978002384gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Man</div></div></div><div><div class="h5">
<br><div class="gmail_quote">On Wed, Nov 29, 2017 at 3:21 PM, Man Cao <span dir="ltr"><<a href="mailto:manc@google.com" target="_blank">manc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello,<div><br></div><div>This patch is a follow-up fix for <a href="https://bugs.openjdk.java.net/browse/JDK-8087291" target="_blank">https://bugs.openjdk.java.<wbr>net/browse/JDK-8087291</a></div><div><br></div><div>This patch moves the call to set_compressed_class_space_<wbr>size() after the flag value for CompressedClassSpaceSize is ergo-initialized, fixing the issue that the reserved size for compressed class space and metaspace is excessively large when MaxMetaspaceSize is set to a small value. More discussion about it is available here:</div><div><div><a href="http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2017-November/025200.html" target="_blank">http://mail.openjdk.java.net/p<wbr>ipermail/hotspot-runtime-dev/2<wbr>017-November/025200.html</a></div></div><div><br></div><div>This code patch is attached. Could anyone review and/or sponsor this patch?<br></div><div><br></div><div>Best,</div><div>Man<br></div></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br></div></div>