<div dir="ltr">Never mind, I am not yet awake it seems :)<div><br></div><div>I see now there was a REDO right after the backout, so my question is pointless. </div><div><br></div><div>Kind Regards, Thomas</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 14, 2016 at 8:20 AM, Thomas Stüfe <span dir="ltr"><<a href="mailto:thomas.stuefe@gmail.com" target="_blank">thomas.stuefe@gmail.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">Hi Stefan,<div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">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="m_-70658628182572226gmail-"><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="m_-70658628182572226gmail-"><br>
<br></span></blockquote><div><br></div></span><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><span class=""><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="m_-70658628182572226gmail-">
<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></span></div><br></div></div>
</blockquote></div><br></div>