Review request for 6829503
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