Re: RFR 8064924: Update java.net.URL to work with modules

2015-02-06 Thread Alan Bateman

On 04/02/2015 15:11, Chris Hegarty wrote:

Agreed. Updated in-place

http://cr.openjdk.java.net/~chegar/8064924/03/specdiff/overview-summary.html 

I think the approach and naming is good. A few small comments on the 
wording:


1. "used to locate URLStreamHandlerProvider providers" - the wording 
hints as further redirection, maybe it would be better as 
"URLStreamHandlerProvider implementations".


2. "the ordering that providers are located" - maybe this should be "the 
order that providers are located".


3. "Some protocol, that are fundamental ...". Here's a re-worded 
statement to consider:


"Some protocol handlers, for example those used for loading platform 
classes or classes on the class path, may not be overridden. The details 
of such restrictions, and when those restrictions apply (during 
initialization of the runtime for example), are implementation specific 
and therefore not specified".


One other thing in this area is setURLStreamHandlerFactory(null). Long 
standing behavior is to remove the system-wide URLStreamHandlerFactory, 
should this continue?


-Alan






Re: [8u-dev] Request for approval: JDK-6933879: URISyntaxException when non-alphanumeric characters are present in scope_id

2015-02-06 Thread Konstantin Shefov

Hi Sean,

I have tried to use different symbols in networking interface name in 
Linux. I have added a comment to 
https://bugs.openjdk.java.net/browse/JDK-8071458


I am able to use symbols: "-", "+", "=", "#", "@".
I was UNABLE to use: "?", "~", "$", "*", "!", "[", "(", "&", "^", "|", 
"/", ":" and "\".


My network interface name in Linux is now "eth0.1_-+=#@55" (used 
"ifconfig" command).


On 23.01.2015 16:47, Seán Coffey wrote:

Konstantin,

can you hold off pushing this fix to jdk8u for the moment ? It's a P4 
and could have behavioural consequences (something we try and avoid in 
update releases). I see JDK-8071458 was logged to track IPv6 scope 
specifications. Let's see how this soaks into JDK 9 over coming days 
and let's see how JDK-8071458 progresses.


regards,
Sean.

On 23/01/15 12:56, Konstantin Shefov wrote:

Hello,

Please approve the direct backport of the bug fix to 8u-dev

Patch applies cleanly to JDK 8u.

The bug:https://bugs.openjdk.java.net/browse/JDK-6933879
The webrev: http://cr.openjdk.java.net/~kshefov/6933879/webrev.01/

JDK 9 changeset: 
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/801eb37fc6c4
Review thread: 
http://mail.openjdk.java.net/pipermail/net-dev/2015-January/008834.html


Thanks
-Konstantin






RFR 8072466: Deadlock when starting MulticastSocket and DatagramSocket

2015-02-06 Thread Ivan Gerasimov

Hello!

There has been a deadlock in jdk-net code noticed on Windows.

In the bug description there's a stack snippet showing that one of the 
deadlocked threads stuck in the native initialization code of 
DualStackPlainDatagramSocketImpl and the other -- in initializing of 
TwoStacksPlainDatagramSocketImpl.


I suspect that the reason might be due to unordered initialization:
AbstractPlainDatagramSocketImpl, the parent of both classes above, 
explicitly loads TwoStacksPlainDatagramSocketImpl from the native init().
Thus, when both classes are being initialized concurrently, it may end 
with a clash.



Another issue noticed by Mark Sheppard is that the Windows version of 
DefaultDatagramSocketImplFactory.createDatagramSocketImpl() isn't 
synchronized, but reads/modifies shared data.

Thus, I removed modification and declared all the accessed fields final.

Would you please help review the proposed fix?

BUGURL: https://bugs.openjdk.java.net/browse/JDK-8072466
WEBREV: http://cr.openjdk.java.net/~igerasim/8072466/0/webrev/

The build went fine on all the platforms, all the tests from 
(net|nio|io) passed Okay.


Sincerely yours,
Ivan