RFR(S/L): 8028537: PPC64: Updated the JDK regression tests to run on AIX
Alan.Bateman at oracle.com
Fri Jan 17 15:50:27 UTC 2014
On 14/01/2014 16:57, Volker Simonis wrote:
> could you please review the following change:
> which, together with the changes from "8031581: PPC64: Addons and fixes for
> AIX to pass the jdk regression tests" and "8031134 : PPC64: implement
> printing on AIX" enables our our port to pass all but the following 7 jtreg
> regression tests on AIX (compared to the Linux/x86_64 baseline from
I've finally got to this one. As the event translation issue is now a
separate issue then I've ignored that part.
I'm not comfortable with the changes to FileDispatcherImpl.c as I don't
think we shouldn't be calling into IO_ or NET_* functions here. I think
I get the issue that you have on AIX (and assume it's the preClose/dup2
that blocks rather than close) but need a bit of time to suggest
alternatives. It may be that it will require an AIX specific
SocketDispatcher. Do you happen to know which tests fail due to this part?
The other changes look okay. There is a typo in the change to
In DatagramChannelImpl.c then you handle connect failing with
EAFNOSUPPORT. I would be tempted to replace the comment to say that it
EAFNOSUPPORT can be ignored on AIX. A minor comment but the indentation
for rv = errno can be fixed (I see the BSD code has it wrong too).
More information about the core-libs-dev