<div dir="ltr">Hi Stefan,<div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 25, 2016 at 4:58 PM, Stefan Karlsson <span dir="ltr"><<a href="mailto:stefan.karlsson@oracle.com" target="_blank">stefan.karlsson@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Jesper,<span class="gmail-"><br>
<br>
On 2016-11-25 16:18, Jesper Wilhelmsson wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Hi,<br>
<br>
Please review this backout of JDK-8169931 that is causing problems in<br>
the nightlies that has proven difficult to fix.<br>
</blockquote>
<br></span>
The assert introduced in JDK-8169931 found a similar bug but this time in the metaspace initialization code. Using command line flags, there are ways to request a non-humongous chunk size (smaller than the Medium chunk size) that doesn't match any of the fixed chunk sizes.<span class="gmail-"><br>
<br></span></blockquote><div><br></div><div><div>I am currently working on a metaspace change and use your non-static ChunkManager->list_index() a lot. I see that this backout has not yet found its way into the repo, but that will happen shortly I guess. </div><div><br></div><div>The change looked reasonable, I am curious which issues exactly were you guys encountered in the nightly tests?</div><div><br></div><div>Thanks, Thomas</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8170355" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/<wbr>browse/JDK-8170355</a><br>
Webrev: <a href="http://cr.openjdk.java.net/~jwilhelm/8170355/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~jw<wbr>ilhelm/8170355/webrev.00/</a><br>
</blockquote>
<br></span>
The backout looks good.<br>
<br>
Thanks,<br>
StefanK<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks,<br>
/Jesper<br>
</blockquote>
</blockquote></div><br></div></div>