dsync: Add debug logging for .dovecot-sync.lock locking/unlocking

diff --git a/src/doveadm/dsync/dsync-brain.c b/src/doveadm/dsync/dsync-brain.c
index cb4a41158eff4d9d9f74629f9708d8595985daba..4fb92490d34656f9a39d70239af377c1c045ddaa 100644 (file)
--- a/src/doveadm/dsync/dsync-brain.c
+++ b/src/doveadm/dsync/dsync-brain.c
@@ -370,6 +370,11 @@ int dsync_brain_deinit(struct dsync_brain **_brain, enum mail_error *error_r)
        if (brain->lock_fd != -1) {
                /* unlink the lock file before it gets unlocked */
                i_unlink(brain->lock_path);
+               if (brain->debug) {
+                       i_debug("brain %c: Unlocked %s",
+                               brain->master_brain ? 'M' : 'S',
+                               brain->lock_path);
+               }
                file_lock_free(&brain->lock);
                i_close_fd(&brain->lock_fd);
        }
@@ -390,17 +395,29 @@ dsync_brain_lock(struct dsync_brain *brain, const char *remote_hostname)
                .lock_timeout_secs = brain->lock_timeout,
                .lock_method = FILE_LOCK_METHOD_FCNTL,
        };
-       const char *home, *error;
+       const char *home, *error, *local_hostname = my_hostdomain();
        bool created;
        int ret;
 
-       if ((ret = strcmp(remote_hostname, my_hostdomain())) < 0) {
+       if ((ret = strcmp(remote_hostname, local_hostname)) < 0) {
                /* locking done by remote */
+               if (brain->debug) {
+                       i_debug("brain %c: Locking done by remote "
+                               "(local hostname=%s, remote hostname=%s)",
+                               brain->master_brain ? 'M' : 'S',
+                               local_hostname, remote_hostname);
+               }
                return 0;
        }
        if (ret == 0 && !brain->master_brain) {
                /* running dsync within the same server.
                   locking done by master brain. */
+               if (brain->debug) {
+                       i_debug("brain %c: Locking done by local master-brain "
+                               "(local hostname=%s, remote hostname=%s)",
+                               brain->master_brain ? 'M' : 'S',
+                               local_hostname, remote_hostname);
+               }
                return 0;
        }
 
@@ -421,6 +438,12 @@ dsync_brain_lock(struct dsync_brain *brain, const char *remote_hostname)
                                            &brain->lock, &created, &error);
        if (brain->lock_fd == -1)
                i_error("Couldn't lock %s: %s", brain->lock_path, error);
+       else if (brain->debug) {
+               i_debug("brain %c: Locking done locally in %s "
+                       "(local hostname=%s, remote hostname=%s)",
+                       brain->master_brain ? 'M' : 'S',
+                       brain->lock_path, local_hostname, remote_hostname);
+       }
        if (brain->verbose_proctitle)
                process_title_set(dsync_brain_get_proctitle(brain));
        return brain->lock_fd == -1 ? -1 : 0;