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

Alan Bateman Alan.Bateman at oracle.com
Fri Jan 17 15:16:54 UTC 2014

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 

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).

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).

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.


More information about the core-libs-dev mailing list