common:dotlock: Escalate a warning message up to INFO from DEBUG.HEADmaster

* common/dotlock.c (dotlock_take): Use my_info_1 when already locked.
(dotlock_release): Likewise.

--

Since recursive lock is not intended with dotlock (yet), it's a
programming error when detecting already locked state on the call of
dotlock_take.  Also, it's a programming error when detecting
non-locked state on the call of dotlock_release.  Use of my_debug_1 is
not relevant here.

Signed-off-by: NIIBE Yutaka <[email protected]>

1 files changed, 8 insertions, 5 deletions

diff --git a/common/dotlock.c b/common/dotlock.c
index c3a4af855..b96854afb 100644
--- a/common/dotlock.c
+++ b/common/dotlock.c
@@ -1548,7 +1548,7 @@ dotlock_take (dotlock_t h, long timeout)
 
   if ( h->locked )
     {
-      my_debug_1 ("Oops, '%s' is already locked\n", h->lockname);
+      my_info_1 ("Oops, '%s' is already locked\n", h->lockname);
       return 0;
     }
 
@@ -1659,10 +1659,13 @@ dotlock_release (dotlock_t h)
 
   if ( !h->locked )
     {
-      my_debug_1 ("Oops, '%s' is not locked\n", h->lockname);
-      if (h->info_cb)
-        h->info_cb (h, h->info_cb_value, DOTLOCK_NOT_LOCKED,
-                    "Oops, '%s' is not locked\n", h->lockname);
+      if (!never_lock)
+        {
+          my_info_1 ("Oops, '%s' is not locked\n", h->lockname);
+          if (h->info_cb)
+            h->info_cb (h, h->info_cb_value, DOTLOCK_NOT_LOCKED,
+                        "Oops, '%s' is not locked\n", h->lockname);
+        }
       return 0;
     }