RFR(M): 8068013: [TESTBUG] Aix support in hotspot jtreg tests

Filipp filipp.zhinkin at gmail.com
Wed Jan 7 13:09:36 UTC 2015

Hi Goetz,

you have to update TestMutuallyExclusivePlatformPredicates test [1], because you've added mutually exclusive predicate. Otherwise that test will fail.

Also, I'd prefer to see switch instead of if-else chain in DynLibDcmdTest, but I'm fine with leaving it as it is now.

Otherwise change looks good.


[1] http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/file/acfdd92cedaa/test/testlibrary_tests/TestMutuallyExclusivePlatformPredicates.java#l48

-----Исходное сообщение-----
От: "Lindenmaier, Goetz" <goetz.lindenmaier at sap.com>
Отправлено: ‎07.‎01.‎2015 14:03
Кому: "'hotspot-dev at openjdk.java.net'" <hotspot-dev at openjdk.java.net>
Тема: RE: RFR(M): 8068013: [TESTBUG] Aix support in hotspot jtreg tests


my first upload missed the fix in Platform.shouldSAAttach():

+        if (isAix()) {
+            return false;   // SA not implemented.

I added that to the webrev.


From: Lindenmaier, Goetz
Sent: Mittwoch, 7. Januar 2015 11:14
To: 'hotspot-dev at openjdk.java.net'
Subject: RFR(M): 8068013: [TESTBUG] Aix support in hotspot jtreg tests


A row of hotspot jtreg tests fail because support to test for Aix is missing.
To pass them, I would like to submit the following fixes:

* runtime/6888954/vmerrors.sh: AIX can read from address '0', then throws SIGILL.
* serviceability/dcmd/DynLibDcmdTest.java: Just add check for Aix.
* test/test_env.sh: Add support for AIX. Sort OSes alphabetical.
* test/testlibrary/com/oracle/java/testlibrary/Platform.java: add isAix(). Sort OSes alphabetical.

Please review this change, and I please need a sponsor.

Thanks and best regards,

More information about the hotspot-dev mailing list