RFR(M): 8031997: PPC64: Make the various POLL constants system dependant

[Volker Simonis]

On Fri, Jan 17, 2014 at 4:16 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> On 17/01/2014 13:45, Volker Simonis wrote:
>>
>> :
>> Yes, you're right - it was because of a "VirtualBox Host-Only Network"
>> network device which seems to fool the test. After I disabled it, all
>> tests passed successfully!
>>
>> And what about the change itself :)
>>
> The change itself looks mostly okay.
>
> For naming then I think I would have a slight preference for something like
> pollinValue to getNatvePollin so that it  somewhat consistent with the other
> places where we do this (like in epoll code with eventSize, eventsOffset,
> ...). Naming is subjective of course so this isn't a big issue.
>

Done.

> I suspect you can drop POLLREMOVE, that is only used in the /dev/poll
> Selector and it has its own definition (and shouldn't be compiled on
> anything other than Solaris anyway).
>

Removed POLLREMOVE from sun.nio.ch.Net.

> A minor comment on DatagramChannelImpl, SourceChannelImpl and a few more
> where the replacing PollArrayWrapper.POLL* with Net.POLL* means it is no
> longer necessary to split lines (just might be neater to bring these cases
> back on the one line again).
>

Done.

> I suspect we will be able to drop the changes to the Windows nio_util.h soon
> as these older versions of Windows are not long for this world. I assume
> that by taking on newer VC++ that it won't even be possible to build or run
> either.
>

We still support Server 2003 :(

> -Alan.

Here's the new webrev:

http://cr.openjdk.java.net/~simonis/webrevs/8031997_2/

Built and tested like before. Everything OK.

Is this now ready for push into ppc-aix-port/stage-9?

Regards,
Volker

PS: I've added you as reviewer, but unfortunately after I created the webrev.