PATCH: ShutdownHooks and LoggingDeadlock2

David Holmes - Sun Microsystems David.Holmes at Sun.COM
Mon Apr 13 22:58:58 UTC 2009


I concur with the caution here. When looking at the lazy initialization 
change, it never occurred to me that first-use could actually occur 
after shutdown had commenced.

David

Martin Buchholz said the following on 04/14/09 06:57:
> On Mon, Apr 13, 2009 at 10:56, Mandy Chung <Mandy.Chung at sun.com> wrote:
> 
>> With the fix for 6829503, ApplicationShutdownHook and DeleteOnExitHook are
>> both lazily initialized.  Since it is possible that a file could be added to
>> the list for DeletedOnExit during shutdown, perhaps we also need to change
>> the Shutdown.add method to allow DeleteOnExitHook to be added during
>> shutdown.
> 
> Shutdown is very tricky, and it seems like a big change to allow
> shutdown hooks to be registered after shutdown has commenced.
> 
> That's not a no vote, but please be very careful fixing this.
> 
> I don't have a high degree of confidence that my own fix is correct,
> but it is conservative and it fixes the test failure.
> 
> Martin



More information about the core-libs-dev mailing list