<html><body><div style="color:#000; background-color:#fff; font-family:times new roman, new york, times, serif;font-size:12pt"><font face="Arial" size="2"><b><span style="font-weight:bold;"></span></b></font>On 31Aug2011 00:04, chris coleman wrote:<br><div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"><div style="font-family: times new roman, new york, times, serif; font-size: 12pt;">| Sebastian wrote:<br>| > On Tue, 30 Aug 2011 11:20:58 -0700 (PDT), chris coleman wrote:<br>| > > After a crash (steps to reproduce: disconnect the internet during the middle of a run), it leaves lock files on the disk. <br>| > <br>| > Current OfflineImap always leaves the lock file existing, it just<br>| > request an exclusive lock via fcntl when it wants to take it. This way<br>| > the lock is going away when the process has gone away even if the lock<br>| > file is still there. [...]<br>| > If it doesn't work
 after a "crash" I would be very surprised (I believe<br>| > you though :-)). The locking code hasn't changed in ages.<br>| <br>| If I understand correctly, the lock is a secondary artifact of the actual exclusive lock object, which exists inside fcntl, and when our offlineimap python process crashes, the lock object is unlocked, and the file is also unlocked.<br><br>Sounds like it.<br><br>| To me, this implies that, on startup of offlineimap, the initialization procedure should be updated to try to delete the lock file(s),<br><br>Why? All that seems to be needed is no lock, not no lock file.<br><br>| and if successful, this proves that there is no currently running offlineimap process, so by deleting the lock file(s) we just effectively did some garbage collection.  <br>| But if deleting those lock file(s) fails, then this proves there is another offlineimap process currently running, so the code should exit with this info message.<br><br>I
 would expect deletion of the lock file to work even if it is locked;<br>delete only needs write on the directory, not write on the file itself.<br><br>Does offlineimap actually not run for you after a crash? Of do you just<br>not believe it will run? It sounds like more detail of the actual<br>failing behaviour is needed, since the lock file _should_ not be the cause of<br>the trouble.<br><br>Disclaimer: I am not an offlineimap developer.,<br><br>Cheers,<br>============================================<br><br><br>After the crash: offlineimap failed to start again. It gave an error like "Another offlineimap is currently running, so this one will exit now, so as to avoid corrupting any data..."  <br><br>The problem was: offlineimap was stalled for far too long (hours) and was going to stay stuck waiting forever for a response from the remote server, because the net connection dropped while sync'ing 10 or 20 messages, then the net reconnected, and it
 still was waiting on the old connection but would never see it.  It had to be made to restart.<br><br>I see this on the File Locking page on wikipedia: "<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: sans-serif; font-size: 16px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><span class="Apple-style-span" style="font-size: 13px; line-height: 19px; "><i>All</i><span class="Apple-converted-space"> </span><tt style="font-family: monospace, 'Courier New'; ">fcntl</tt><span class="Apple-converted-space"> </span>locks
 associated with a file for a given process are removed when<span class="Apple-converted-space"> </span><i>any</i><span class="Apple-converted-space"> </span>file descriptor for that file is closed by that process, even if a lock was never requested for that file descriptor. Also,<span class="Apple-converted-space"> </span><tt style="font-family: monospace, 'Courier New'; ">fcntl</tt><span class="Apple-converted-space"> </span>locks are not inherited by a child process. The<span class="Apple-converted-space"> </span><tt style="font-family: monospace, 'Courier New'; ">fcntl</tt><span class="Apple-converted-space"> </span>close semantics are particularly troublesome for applications that call subroutine libraries that may access files."</span></span><br><br>By default, file locking on unix/linux is ADVISORY.  "<span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: sans-serif;
 font-size: 16px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; "><span class="Apple-style-span" style="font-size: 13px; line-height: 19px; ">Mandatory locks have no effect on the unlink function. As a result, certain programs may, effectively, circumvent mandatory locking. The authors of Advanced Programming in the UNIX Environment (Second Edition) observed that the ed editor did so (page 456)."<br><br>Is it possible the implementation of locking in offlineimap might be based on mandatory locking, not advisory locking ?<br><br><br></span></span></div></div></div></body></html>