RFR(M): 8068013: [TESTBUG] Aix support in hotspot jtreg tests
dmitry.samersoff at oracle.com
Wed Jan 21 13:23:08 UTC 2015
Looks good for me (not a reviewer).
This function is not intended to control whether SA is implemented or
not, it's just to check couple of well known conditions when attach
So it's better to return True (as it happens by default) for Aix and
manage SA related problem on different layers (ProblemList.txt, etc).
On 2015-01-07 13:52, Lindenmaier, Goetz wrote:
> Thanks Fredrik!
> Best regards,
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-bounces at openjdk.java.net] On Behalf Of Fredrik Arvidsson
> Sent: Mittwoch, 7. Januar 2015 11:18
> To: hotspot-dev at openjdk.java.net
> Subject: Re: RFR(M): 8068013: [TESTBUG] Aix support in hotspot jtreg tests
> The changes to the DynLibDcmdTest look good to me. I am not a reviewer
> though, but I wrote that test some time ago.
> On 2015-01-07 11:14, Lindenmaier, Goetz wrote:
>> 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,
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.
More information about the hotspot-dev