RFR: 8048020 - Regression on java.util.logging.FileHandler (original) (raw)

Daniel Fuchs daniel.fuchs at oracle.com
Tue Jul 1 09:25:33 UTC 2014

On 6/25/14 2:34 PM, Jason Mehrens wrote:

Daniel,

FileChannel.open(WRITE,APPEND) could throw NoSuchFileException during a startup and shutdown race between two VMs. That case needs to either perform a bounded retry or continue and rotate.

Hi Jason, Alan,

Here is an updated version of the fix that does a bounded retry (retries once - and if it fails, proceeds with the next name). I have also added NO_FOLLOW_LINKS - for the case where we try to open an existing Lockfile, and suppressed the Files.isWritable check since that will be taken care of by the call to FileChannel.open.

http://cr.openjdk.java.net/~dfuchs/webrev_8048020/webrev.01/

I also updated the comments to make it clear that the file could not have been locked by another instance of FileHandler (since that would have been taken care of by our global 'locks' synchronization).

best regards,

-- daniel

Jason ---------------------------------------- Date: Tue, 24 Jun 2014 20:34:46 +0200 From: daniel.fuchs at oracle.com To: core-libs-dev at openjdk.java.net CC: jasonmehrens at hotmail.com; Alan.Bateman at oracle.com Subject: RFR: 8048020 - Regression on java.util.logging.FileHandler

Hi, Please find below a patch [1] for [2]: 8048020 - Regression on java.util.logging.FileHandler This corresponds to the problem that was discussed on this list in the following email thread: "Zombie FileHandler locks can exhaust all available log file locks." http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-June/027296.html

[1] http://cr.openjdk.java.net/~dfuchs/webrev8048020/webrev.00/ [2] https://bugs.openjdk.java.net/browse/JDK-8048020 This patch would be a good candidate for being backported on 8, since the issue was introduced in 8. best regards, -- daniel

More information about the core-libs-dev mailing list