RFR(S/L): 8028537: PPC64: Updated the JDK regression tests to run on AIX

Volker Simonis volker.simonis at gmail.com
Fri Jan 17 21:10:22 UTC 2014

Hi Alan,

thanks for looking at this. Please find may comments inline:

On Fri, Jan 17, 2014 at 10:48 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> On 15/01/2014 16:42, Volker Simonis wrote:
>> Hi Alan,
>> thanks for the suggestion. That's fine for me. I've copied the empty SCTP
>> stubs from the macosx to the aix directory as well and updated the make file
>> accordingly (in the patch for "8031581: PPC64: Addons and fixes for AIX to
>> pass the jdk regression tests").
>> Therefore, the changes to the three tests:
>> test/com/sun/nio/sctp/SctpChannel/Util.java
>> test/com/sun/nio/sctp/SctpMultiChannel/Util.java
>> test/com/sun/nio/sctp/SctpServerChannel/Util.java
>> can be considered obsolete.
> Thanks, I think this makes the most sense.
> I looked through the rest of this webrev and the update to the tests looks
> fine.

Great, thanks.

> One general comment is that for many of these shell tests (that survive the
> current effort to replace them) is that we could move the Unix handling into
> the match any case so that we don't have to list each of Linux, SunOS,
> Darwin, ...   I think this came up when the OS X port was brought in but
> there wasn't any follow-up on it. I am not suggesting you do this here, it's
> just a comment as I see same change to so many tests.
> A minor comment on SBC.java is that it could just catch
> UnsupportedOperationException on L238, that would avoid needing to check
> os.name.

I agree, that looks much nicer. Done as requested.

> A really minor comment on the updates to ProblemList.txt is that the JMX
> test should probably be in the jdk_jmx section (it's just a convention that
> we've been using, it doesn't of course really matter where tests are
> listed).

Done. Moved the excluded tests down to the jdk_jmx section.

Here's the new webrev:


Can I push this now to ppc-aix-port/stage-9?

Thank you and best regards,

> -Alan

More information about the core-libs-dev mailing list