RFR (S) CR 8023638: Add the regression test for 8006997

[Daniel D. Daugherty]

 > http://cr.openjdk.java.net/~shade/8023638/webrev.01/

hotspot/src/share/vm/runtime/arguments.cpp
     Thanks for fixing the typos.

hotspot/test/runtime/contended/Options.java
     Thanks for updating the test.

Thumbs up.

Dan


On 8/26/13 7:12 AM, Aleksey Shipilev wrote:
> Hi Dan,
>
> On 08/24/2013 12:14 AM, Daniel D. Daugherty wrote:
>> On 8/23/13 5:24 AM, Aleksey Shipilev wrote:
>>> Here's the webrev:
>>>     http://cr.openjdk.java.net/~shade/8023638/webrev.00/
>> hotspot/test/runtime/contended/Options.java
>>      I count 11 Java process invocations. Will the default timeout
>>      of 120 seconds be enough for a fastdebug '-server -Xcomp'
>>      run on something like Solaris SPARC.
> Linux i5/x86_64 laptop passes this test in 5 seconds. I think there is
> enough headroom for slower machines.
>
>>      The fragments:
>>
>>          "must be the between"
>>          "must be the multiple of 8"
>>
>>      probably match the real error messages, but those messages
>>      aren't quite proper English. I expected:
>>
>>          "must be in between"
>>          "must be a multiple of 8"
> Yes. Unfortunately, this is what VM outputs. Does it make sense to fix
> the error messages in VM along with the regression test? If so, here is
> the new webrev:
>    http://cr.openjdk.java.net/~shade/8023638/webrev.01/
>
> -Aleksey.