RFR (S): JDK-8038924: Test bit-instructions fails with unexpected exit value on sparc

[Filipp Zhinkin]

Vladimir,

sorry for a late response.

On 05/13/2014 11:21 PM, Vladimir Kozlov wrote:
> On 5/13/14 11:49 AM, Filipp Zhinkin wrote:
>> Hi all,
>>
>> please review fix forJDK-8038924.
>
> Filipp, please, provide the link to the bug report in you RFR.
https://bugs.openjdk.java.net/browse/JDK-8038924

Sorry for that, I'l take it in mind for future RFRs.
>
>>
>> CLI tests on BMI-related options previously took all externally passed
>> VM options,
>> add options that should be tested and launch new VM with such
>> combination of options.
>>
>> Unfortunately, some options like IgnoreUnrecognizedVMOptions, can affect
>> processing of tested flags and in such case tests fail.
>>
>> I've updated these tests so now they do not use externally passed 
>> options.
>
> I don't think it is right solution. Some external flags could be 
> important (DeoptimizeALot for example).
>
> You can add -XX:-IgnoreUnrecognizedVMOptions to tests which you expect 
> to fail (on platforms which does not have the feature). It will 
> nullify this flag passed from CL.
In general I agree with that point - tests should be able to use as much 
externally passed options as possible.
However, in this particular case I believe that it would be better to 
ignore such options, because these tests only launch a new VM with tests 
options and '-version' flag,
so we'll not be able to catch issues that we can't find with other BMI 
tests. And if CLI tests will not use externally passed options it will 
be easier to maintain it.

Note that other BMI-related tests use all externally passed options, so 
we can catch any issues related to JVM behaviour using it.

So I think that it will be better to avoid usage of externally passed 
options in CLI-tests and update compiler/intrinsics/bmi tests to run it 
on all platforms.

Thanks,
Filipp.
>
> Regards,
> Vladimir
>
>> Also, I've fix silly issue with String::format in
>> BMICommandLineOptionTestBase.java.
>>
>> Webrev: 
>> http://cr.openjdk.java.net/~iignatyev/fzhinkin/8038924/webrev.00/
>> Testing: manual and automated.
>>
>> Thanks,
>> Filipp.