<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<tt>Adding </tt><tt><a 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 class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tamao/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>
<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>
<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>
</body>
</html>