<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>
Hi Jesper,<br>
<br>
I assume that you are talking about
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
nsk/regression/b4212983.<br>
<br>
In that case I agree that the test is bad. It will not detect if
we start allowing -Xmx0m since it only checks the error message
for the initial heap size.<br>
<br>
It is probably too late by now, but I would suggest filing a bug
that matches the test failure to make this a know failure. The bug
should be on hotspot/test and say that this test needs to be split
up into two test cases. One for -Xms0m and -Xmx0m.<br>
<br>
If you do that I think you can push hotspot-gc to main. But maybe
we have already missed the snapshot deadline?<br>
<br>
Bengt<br>
<br>
<br>
<br>
On 5/9/13 12:41 PM, Jesper Wilhelmsson wrote:<br>
</div>
<blockquote cite="mid:518B7D3C.5080201@oracle.com" type="cite">Hi,
<br>
<br>
There is a failure in the nightlies due to change 4612 where the
argument parsing was changed somewhat.
<br>
<br>
The test starts a JVM with invalid heap size flags and looks for a
specific error message. However, since there are more than one
invalid flag on the command line (-Xms0m -Xmx0m) we now check the
flags in a different order than before and a different error
message is printed causing the test to fail.
<br>
<br>
I find this test extremely unstable even though it is in our
"stable" GC baseline. If we want to check for invalid arguments,
there should only be one invalid argument on the command line.
<br>
<br>
I'm waiting for a response from SQE on this and in the mean time I
will not push hotspot-gc to main. I don't feel comfortable enough
around mercurial to attempt to exclude this changeset from the
push. Especially since there is nothing wrong in the change.
<br>
<br>
I have a job in the jprt queue now to pull main to gc.
<br>
<br>
Let me know if there is something in hotspot-gc that needs to be
pushed to main this week.
<br>
/Jesper
<br>
</blockquote>
<br>
</body>
</html>