RFR(S): 8058891: serviceability/dcmd/CodelistTest.java - fails on all platforms
nils.eliasson at oracle.com
Wed Sep 24 11:25:57 UTC 2014
On 2014-09-23 23:24, Vladimir Kozlov wrote:
> On 9/23/14 6:14 AM, Nils Eliasson wrote:
>> Please review this small fix. It contains:
>> 1) Adding compiler dcmd tests to TEST.groups/hotspot_compiler_3
> I think they belong to hotspot_serviceability group. Please, move it
ok moved them.
>> 2) Moving compiler dcmd tests to subfolder compiler and adding @library
>> tag to find and build DcmdUtil
> Was MethodIdentifierParser.java just moved and no change was done in it?
>> 3) Fixing CompilerQueueTest to work in client, vintage server (no
>> tiered), and with interpreter only
> Could you explain 3) changes? I see only new throws which will causes
> test failure.
Are you sure you are looking at the right file?
The while loop will now match zero or many queue outputs.
> Should we also test interpreter mode in CodeCacheTest.java?:
Yes, I will add that. Also the CompilerQueueTest should be testen with
interpreter since it failed there earlier. I will add that too.
> @run main/othervm -Xint -XX:+SegmentedCodeCache CodeCacheTest
>> 4) Preventing CodelistTest from trying to reflect MethodHandles
> Explain change.
CodelistTest validates the printed methods by reflecting them.
MethodHandle.invokeToStatic can not be reflected. The tests skips all
methods of MethodHandle now. This bug was not caught earlier because
then we saw no methodHandles during start up.
New webrev: http://cr.openjdk.java.net/~neliasso/8058891/webrev.04/
>> Tests run: Test group hotspot_compiler_3 on the 9 major platforms.
More information about the hotspot-compiler-dev