<div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">Hi all,</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif"><br>
</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">Could I get a couple of reviews for this small patch, please?</font></div><div class="im" style="color:rgb(80,0,80);background-color:rgb(255,255,255)">
<div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">7197906: BlockOffsetArray::power_to_cards_back() needs to handle > 32 bit shifts</font></div><div><font face="arial, helvetica, sans-serif"><br>
</font></div></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">CR link:</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><a href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7197906" target="_blank" style="color:rgb(17,85,204)"><font face="arial, helvetica, sans-serif">http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7197906</font></a></div>
<div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">Webrev:</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><a href="http://cr.openjdk.java.net/~brutisso/7197906/webrev.01/" target="_blank" style="color:rgb(17,85,204)"><font face="arial, helvetica, sans-serif">http://cr.openjdk.java.net/~brutisso/7197906/webrev.01/</font></a></div>
<div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><span style="color:rgb(0,0,0)"><font face="arial, helvetica, sans-serif">Contributed-by: </font></span></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)">
<font face="arial, helvetica, sans-serif"><span style="color:rgb(0,0,0)">Hal Mo <</span><a href="mailto:kungu.mjh@taobao.com" target="_blank">kungu.mjh@taobao.com</a><span style="color:rgb(0,0,0)">> from Alibaba Group(with OCA) </span>
</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif"><br></font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">It may take a while before the CR link is publicly accessible.</font></div>
<div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif"><br></font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">In blockOffsetTable.hpp, there is an unexpected data truncation in power_to_cards_back(uint i), which may lead to crashing the VM on very large GC heaps.</font></div>
<div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">The problem could be reproduced easily on machines, that have enough memory, with:</font></div><div class="im" style="color:rgb(80,0,80);background-color:rgb(255,255,255)">
<div><font face="arial, helvetica, sans-serif">$ java -Xmx135g -XX:+UseConcMarkSweepGC</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)">
<font face="arial, helvetica, sans-serif">The proposed patch fixes this problem, and builds on all OpenJDK supported platforms.</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif"><br>
</font></div><div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif">I'd like to thank Bengt Rutisson for the initial review and help preparing the CR and Webrev.</font></div>
<div style="color:rgb(34,34,34);background-color:rgb(255,255,255)"><font face="arial, helvetica, sans-serif"><br></font></div><font face="arial, helvetica, sans-serif">Regards,<br>Hal</font>