<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=windows-1252">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi everyone,
<br>
<div class="moz-forward-container"><br>
A trivial fix is still waiting for reviewers...<br>
<br>
Bug: <a class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8132334">https://bugs.openjdk.java.net/browse/JDK-8132334</a>
<br>
Webrev:
<a class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Edfazunen/8132334/webrev/">http://cr.openjdk.java.net/~dfazunen/8132334/webrev/</a>
<br>
<br>
Comments: w/a for known jtreg issue described:
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/CODETOOLS-7900024">https://bugs.openjdk.java.net/browse/CODETOOLS-7900024</a>
<br>
<br>
Thanks,<br>
Dima<br>
-------- Forwarded 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: RFR (XXS): 8132334: [TESTBUG]
gc/metaspace/TestCapacityUntilGCWrapAround.java
Compilation failed</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Date: </th>
<td>Tue, 28 Jul 2015 21:07:50 +0300</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">From: </th>
<td>Dmitry Fazunenko <a class="moz-txt-link-rfc2396E" href="mailto:dmitry.fazunenko@oracle.com"><dmitry.fazunenko@oracle.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">To: </th>
<td>Kim Barrett <a class="moz-txt-link-rfc2396E" href="mailto:kim.barrett@oracle.com"><kim.barrett@oracle.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">CC: </th>
<td>hotspot-gc-dev <a class="moz-txt-link-rfc2396E" href="mailto:hotspot-gc-dev@openjdk.java.net"><hotspot-gc-dev@openjdk.java.net></a></td>
</tr>
</tbody>
</table>
<br>
<br>
<pre>
On 28.07.2015 4:40, Kim Barrett wrote:
> On Jul 27, 2015, at 9:12 PM, Dmitry Fazunenko <a class="moz-txt-link-rfc2396E" href="mailto:dmitry.fazunenko@oracle.com"><dmitry.fazunenko@oracle.com></a> wrote:
>> On 27.07.2015 21:49, Kim Barrett wrote:
>>> On Jul 24, 2015, at 12:23 PM, Dmitry Fazunenko <a class="moz-txt-link-rfc2396E" href="mailto:dmitry.fazunenko@oracle.com"><dmitry.fazunenko@oracle.com></a> wrote:
>>>> Hi everyone,
>>>>
>>>> Can I have reviews for the trivial change that must fix the compilation problem?
>>>> <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8132334">https://bugs.openjdk.java.net/browse/JDK-8132334</a>
>>>>
>>>> webrev:
>>>> <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dfazunen/8132334/webrev/">http://cr.openjdk.java.net/~dfazunen/8132334/webrev/</a>
>>>>
>>>> Comments: In some circumstances jtreg doesn't see wb classes if they weren't built explicitly.
>>> Under what circumstances does this occur? I see other tests that look very similar to the pre-change version of this test - what makes this test so special?
>>>
>> I don't have an answer. I just heard about such problem, but I didn't find the bug. Might be because it's not filed yet. The hypothesis is the concurrency might cause such effect.
> Without an understanding of why this change is needed here but not
> elsewhere, I'm concerned that we're either
>
> 1. papering over one instance of a more widespread problem, or
> 2. not really solving the problem with this test after all.
>
I found the jtreg bug:
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/CODETOOLS-7900024">https://bugs.openjdk.java.net/browse/CODETOOLS-7900024</a>
It's about 3 years old and still un-triaged, so I won't expect it will
be fixed soon.
But the problem could be easily worked around and this change does it.
Thanks,
Dima
</pre>
<br>
</div>
<br>
</body>
</html>