Review request for 6829503

Alan Bateman Alan.Bateman at Sun.COM
Mon Apr 20 09:02:41 UTC 2009

David Holmes - Sun Microsystems wrote:
> Hi Alan,
> Alan Bateman said the following on 04/20/09 00:00:
>> As it happens, there was a bug in that code (6526376) that caused NPE 
>> to be thrown so jdk7 b10 is the first build where 
>> IllegalStateException is possible.
> Ah I see. Seems to me that 6526376 took the easy fix and ignored the 
> underlying issue of whether this should be an exceptional circumstance 
> or not. :(
or probably earlier in 4809375 when there was code introduced to 
explicitly handle this case.

> :
> I think there's probably a way to keep the list in java code (for 
> resource management) and also add the fd to list used by the VM. 
> Adding a new "VM shutdown hook" to be executed by the VMThread at the 
> termination safepoint would probably be the best place to deal with this.
That would work but does mean another dependency in the VM code.

I'll create a bug with a link to this discussion so that this issue 
doesn't get forgotten.


More information about the core-libs-dev mailing list