<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Hi.<br>
Here is updated webrevs<br>
<span style="font-weight: bold;color: #cc0000;"> </span><a
href="http://cr.openjdk.java.net/%7Eiignatyev/anzakharov/8037924/webrev.14/">http://cr.openjdk.java.net/~iignatyev/anzakharov/8037924/webrev.14/</a><br>
<a
href="http://cr.openjdk.java.net/%7Eiignatyev/anzakharov/8037925/webrev.07/">http://cr.openjdk.java.net/~iignatyev/anzakharov/8037925/webrev.07/</a><br>
<br>
Thanks.<br>
<br>
<div class="moz-cite-prefix">On 10.04.2014 16:25, Erik Helin wrote:<br>
</div>
<blockquote cite="mid:53468DBB.3050504@oracle.com" type="cite">Given
that the cleanup issue exists and is being worked on, I'm fine
with this patch.
<br>
<br>
Thanks,
<br>
Erik
<br>
<br>
On 2014-04-10 14:22, Erik Helin wrote:
<br>
<blockquote type="cite">Andrey has updated the patch:
<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ehelin/8037925/webrev.06/">http://cr.openjdk.java.net/~ehelin/8037925/webrev.06/</a>
<br>
<br>
The new patch does no longer change the testlibrary. There is
already an
<br>
issue filed for fixing these tests and the previous testlibrary
changes:
<br>
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8039489">https://bugs.openjdk.java.net/browse/JDK-8039489</a>.
<br>
<br>
Thanks,
<br>
Erik
<br>
<br>
On 2014-04-02 19:02, Andrey Zakharov wrote:
<br>
<blockquote type="cite">Hi, Jon
<br>
On 02.04.2014 01:21, Jon Masamitsu wrote:
<br>
<blockquote type="cite">Seems like an out-of-memory (OOME) can
be thrown here (or anywhere below
<br>
this block of code). Will an OOME be considered a pass,
fail or
<br>
test-not-run?
<br>
</blockquote>
Yes, its really an question for me, from one point of view, we
should
<br>
catch OOM and move forward to exit(0),
<br>
from another point - such things should be handled by test
harness. Test
<br>
should have some @tag about 1G memory its used and harness
should skip
<br>
test and notify about this if machine memory is below needed.
<br>
Maybe there is another right way.
<br>
<br>
</blockquote>
</blockquote>
</blockquote>
<br>
</body>
</html>