<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix"><br>
Another FYI: Last week I pushed the fix to the HS24 repository:<br>
<a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/hsx/hsx24/hotspot/rev/9c930c24f9b0">http://hg.openjdk.java.net/hsx/hsx24/hotspot/rev/9c930c24f9b0</a><br>
<br>
This will most likely go in to the next 7update release. With the
current planning this means 7u12.<br>
<br>
Bengt<br>
<br>
<br>
On 2012-09-17 22:08, Bengt Rutisson wrote:<br>
</div>
<blockquote cite="mid:5057834A.7000201@oracle.com" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div class="moz-cite-prefix"><br>
FYI, I pushed this earlier today so the fix is now available in
hotspot-gc:<br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/859cd1a76f8a">http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/859cd1a76f8a</a><br>
<br>
I've also filed a sub CR to backport the fix to hs24/7u12. I
hope we get approval to push it there soon.<br>
<br>
Thanks again for the patch Hal!<br>
Bengt<br>
<br>
On 2012-09-14 01:26, Srinivas Ramakrishna wrote:<br>
</div>
<blockquote
cite="mid:CABzyjynqngYEETuJNERZUta8L4OLAQjWWbipp77a0u5OSGAE+g@mail.gmail.com"
type="cite"><br>
Great, thanks John, all!<br>
<br>
-- ramki<br>
<br>
<div class="gmail_quote">On Fri, Sep 14, 2012 at 12:19 AM, John
Cuthbertson <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:john.cuthbertson@oracle.com" target="_blank">john.cuthbertson@oracle.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#ffffff" text="#000000"> Hi Ramki,<br>
<br>
I don't think there will be an issue with back porting
this to hs24 (which I think is going into 7u12). I just
have to check the dates. There's a couple of other CRs
that we're probably going to want to back port as well.<br>
<br>
I searched the sources for the offending pattern (with and
without whitespace) and only found the instances fixed by
Hal. I think we're OK in that respect. But thanks for
making sure. :)<br>
<br>
JohnC
<div>
<div class="h5"><br>
<br>
On 09/13/12 15:21, Srinivas Ramakrishna wrote:
<blockquote type="cite"><br>
Thanks Bengt; changes look good to me too, Hal,
although i just looked at the changes, and did not<br>
check the code to see if there might be more
instances of the fingered anti-pattern. (I am pretty
sure<br>
you and John and other reviewers have covered that
front well :-)<br>
<br>
Bengt, any chance that the fix will get bkported
also to 7uXX soon, for some suitable value of XX?<br>
<br>
thanks!<br>
-- ramki<br>
<br>
<div class="gmail_quote">On Thu, Sep 13, 2012 at
8:27 PM, Bengt Rutisson <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:bengt.rutisson@oracle.com"
target="_blank">bengt.rutisson@oracle.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="border-left:1px solid
rgb(204,204,204);margin:0pt 0pt 0pt
0.8ex;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div><br>
Hi Ramki,<br>
<br>
Thanks for looking at this!<br>
<br>
Here's a webrev based on the second patch
that Hal sent out:<br>
<a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Ebrutisso/7197906/webrev.02/"
target="_blank">http://cr.openjdk.java.net/~brutisso/7197906/webrev.02/</a><span><font
color="#888888"><br>
<br>
Bengt</font></span>
<div>
<div><br>
<br>
On 2012-09-13 18:12, Srinivas
Ramakrishna wrote:<br>
</div>
</div>
</div>
<div>
<div>
<blockquote type="cite">Hal, thanks for
the catch and the fix!<br>
<br>
Bengt, might it be possible to host this
as a traditional webrev in the usual
place, to simplify review logistics?<br>
<br>
thanks!<br>
-- ramki<br>
<br>
<div class="gmail_quote">On Thu, Sep 13,
2012 at 4:55 PM, Jianhao Mo <span
dir="ltr"><<a
moz-do-not-send="true"
href="mailto:mojianhao@gmail.com"
target="_blank">mojianhao@gmail.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="border-left:1px solid
rgb(204,204,204);margin:0pt 0pt 0pt
0.8ex;padding-left:1ex"><font
face="arial, helvetica,
sans-serif">Hi all,</font>
<div><font face="arial, helvetica,
sans-serif"><br>
</font></div>
<div><font face="arial, helvetica,
sans-serif">Attached please find
the new patch, modified
according to <span
style="color:rgb(34,34,34)">John's
advice.</span></font></div>
<div><font face="arial, helvetica,
sans-serif"><span
style="color:rgb(34,34,34)"><br>
</span></font></div>
<div><font face="arial, helvetica,
sans-serif"><span
style="color:rgb(34,34,34)">Thanks </span></font><span
style="color:rgb(34,34,34);font-family:arial,helvetica,sans-serif">Bengt
again for the help :)</span></div>
<div><font color="#222222"
face="arial, helvetica,
sans-serif"><br>
</font></div>
<div><font color="#222222"
face="arial, helvetica,
sans-serif">Regards,</font></div>
<div><font color="#222222"
face="arial, helvetica,
sans-serif"><br>
</font></div>
<div><font color="#222222"
face="arial, helvetica,
sans-serif">Hal<br>
</font>
<div>
<div><br>
<div class="gmail_quote">2012/9/12
Jianhao Mo <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:mojianhao@gmail.com"
target="_blank">mojianhao@gmail.com</a>></span><br>
<blockquote
class="gmail_quote"
style="border-left:1px
solid
rgb(204,204,204);margin:0pt
0pt 0pt
0.8ex;padding-left:1ex">
<div
style="color:rgb(34,34,34)"><font
face="arial,
helvetica, sans-serif">Hi
all,</font></div>
<div
style="color:rgb(34,34,34)"><font
face="arial,
helvetica, sans-serif"><br>
</font></div>
<div
style="color:rgb(34,34,34)"><font
face="arial,
helvetica, sans-serif">Could
I get a couple of
reviews for this small
patch, please?</font></div>
<div
style="color:rgb(80,0,80)">
<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)"><font
face="arial,
helvetica, sans-serif">CR
link:</font></div>
<div
style="color:rgb(34,34,34)"><a
moz-do-not-send="true"
href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7197906"
style="color:rgb(17,85,204)"
target="_blank"><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)"><font
face="arial,
helvetica, sans-serif">Webrev:</font></div>
<div
style="color:rgb(34,34,34)"><a
moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Ebrutisso/7197906/webrev.01/"
style="color:rgb(17,85,204)"
target="_blank"><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)"><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)">
<font face="arial,
helvetica, sans-serif"><span
style="color:rgb(0,0,0)">Hal Mo <</span><a moz-do-not-send="true"
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)"><font
face="arial,
helvetica, sans-serif"><br>
</font></div>
<div
style="color:rgb(34,34,34)"><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)"><font
face="arial,
helvetica, sans-serif"><br>
</font></div>
<div
style="color:rgb(34,34,34)"><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)"><font
face="arial,
helvetica, sans-serif">The
problem could be
reproduced easily on
machines, that have
enough memory, with:</font></div>
<div
style="color:rgb(80,0,80)">
<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)">
<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)"><font
face="arial,
helvetica, sans-serif"><br>
</font></div>
<div
style="color:rgb(34,34,34)"><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)"><font
face="arial,
helvetica, sans-serif"><br>
</font></div>
<font face="arial,
helvetica, sans-serif">Regards,<br>
Hal</font> </blockquote>
</div>
<br>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</blockquote>
<br>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</blockquote>
<br>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>