<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Thank you for review, Dan.<br>
<br>
Tao<br>
<br>
On 7/2/13 1:32 PM, Daniel D. Daugherty wrote:
<blockquote cite="mid:51D338D2.9020609@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<tt>Adding </tt><tt><a moz-do-not-send="true"
class="moz-txt-link-abbreviated"
href="mailto:hotspot-runtime-dev@openjdk.java.net">hotspot-runtime-dev@openjdk.java.net</a>
to this thread...<br>
<br>
> <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Etamao/7122222_hsx24/webrev.01/">http://cr.openjdk.java.net/~tamao/7122222_hsx24/webrev.01/</a><br>
<br>
make/linux/makefiles/vm.make<br>
No comments.<br>
<br>
make/solaris/makefiles/vm.make<br>
No comments.<br>
<br>
src/os/solaris/vm/os_solaris.inline.hpp<br>
No comments.<br>
<br>
Also compared the two patch files:<br>
7122222GCLogLimitedTo2GFor32Bit_hsx24.patch<br>
7122222GCLogLimitedTo2GFor32Bit.patch<br>
<br>
In HSX-25, the Makefile construct looks like:<br>
<br>
CXXFLAGS/ostream.o += -D_FILE_OFFSET_BITS=64<br>
<br>
In HSX-24, the Makefile construct looks like:<br>
<br>
ostream.o: CXXFLAGS += -D_FILE_OFFSET_BITS=64<br>
<br>
which threw me for a loop for a minute... I realized that<br>
your construct matches the existing rules for vm_version.o<br>
in each release so I think the HSX-24 backport is good.<br>
</tt></blockquote>
I should've mentioned the point that I just would like to keep
consistency with the existing way of passing cxxflag to vm_version.o
in order to make the changeset simple to come up.<br>
<blockquote cite="mid:51D338D2.9020609@oracle.com" type="cite"><tt>
<br>
Hopefully, you double checked builds logs on both Linux<br>
and Solaris and made sure that '-D_FILE_OFFSET_BITS=64'<br>
is only passed where you expect it.<br>
</tt></blockquote>
Yes, it is only passed "-D_FILE_OFFSET_BITS=64" to ostream.o<br>
<br>
As I mentioned, the changeset passed JPRT. <br>
<br>
Also, I manually tested whether the change would solve the CR's
problem as expected. It's doing its right job on Linux-i586,
Solaris-i586 and Solaris-sparc.<br>
<blockquote cite="mid:51D338D2.9020609@oracle.com" type="cite"><tt>
<br>
Thumbs up!<br>
<br>
</tt><tt>Dan<br>
<br>
</tt><br>
<blockquote cite="mid:51D284DB.5080107@oracle.com" type="cite">
<div class="moz-forward-container"> -------- Original Message
--------
<table class="moz-email-headers-table" border="0"
cellpadding="0" cellspacing="0">
<tbody>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Subject:
</th>
<td>Re: hsx24 backport: Request for review: 7122222: GC
log is limited to 2G for 32-bit</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Date:
</th>
<td>Mon, 01 Jul 2013 23:05:46 -0700</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">From:
</th>
<td>Tao Mao <a moz-do-not-send="true"
class="moz-txt-link-rfc2396E"
href="mailto:tao.mao@oracle.com"><tao.mao@oracle.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Organization:
</th>
<td>Oracle Corporation</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">To:
</th>
<td><a moz-do-not-send="true"
class="moz-txt-link-abbreviated"
href="mailto:hotspot-gc-dev@openjdk.java.net">hotspot-gc-dev@openjdk.java.net</a>
<a moz-do-not-send="true"
class="moz-txt-link-rfc2396E"
href="mailto:hotspot-gc-dev@openjdk.java.net"><hotspot-gc-dev@openjdk.java.net></a>,
<a moz-do-not-send="true"
class="moz-txt-link-abbreviated"
href="mailto:build-dev@openjdk.java.net">build-dev@openjdk.java.net</a></td>
</tr>
</tbody>
</table>
<br>
<br>
<pre>The format of passing cxxflag to a specific file is different in hsx24
than in hsx25. I've changed the setting accordingly in order to pass
through the compilation.
Please review the new webrev below and ignore webrev.00.
hsx24 webrev:
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Etamao/7122222_hsx24/webrev.01/">http://cr.openjdk.java.net/~tamao/7122222_hsx24/webrev.01/</a>
(original) hsx25 webrev:
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Etamao/7122222/webrev.00/">http://cr.openjdk.java.net/~tamao/7122222/webrev.00/</a>
Test:
Builds were tested on Linux-i586, Solaris-i586, and Solaris-sparc.
Builds were successful and they all passed test of the gc-log size limit
of 2G.
Thanks.
Tao
On 7/1/13 1:18 PM, Tao Mao wrote:
> The hsx25 fix has been pushed already. Then I got the
> 7u40-critical-approved.
>
> When I tried backporting to hsx24, it didn't apply to the hsx24 repo
> cleanly. Thus, I made the patch manually (copy-and-paste style) and
> now need some quick reviews to get it in since it's 7u40 related P3 CR.
>
> hsx24 webrev:
> <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Etamao/7122222_hsx24/webrev.00/">http://cr.openjdk.java.net/~tamao/7122222_hsx24/webrev.00/</a>
>
> (original) hsx25 webrev:
> <a moz-do-not-send="true" class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Etamao/7122222/webrev.00/">http://cr.openjdk.java.net/~tamao/7122222/webrev.00/</a>
>
> Thanks.
> Tao
</pre>
<br>
</div>
<br>
</blockquote>
<br>
</blockquote>
</body>
</html>