common/finisher: use distinct lock names

Two Finishers should not be considered equivalent
for lockdep purposes: for example in mgr we have
our in-mgr Finisher, and then any finishers that
might be in libraries called by modules.

Signed-off-by: John Spray <john.spray@redhat.com>

diff --git a/src/common/Finisher.h b/src/common/Finisher.h
index 5947fa77df149f0700a10b0d5884c8a6050ad2aa..045b8e47a5a5c4e103304a783d1b9a98fd27c550 100644 (file)
--- a/src/common/Finisher.h
+++ b/src/common/Finisher.h
@@ -142,7 +142,7 @@ class Finisher {
 
   /// Construct a named Finisher that logs its queue length.
   Finisher(CephContext *cct_, string name, string tn) :
-    cct(cct_), finisher_lock("Finisher::finisher_lock"),
+    cct(cct_), finisher_lock("Finisher::" + name),
     finisher_stop(false), finisher_running(false),
     thread_name(tn), logger(0),
     finisher_thread(this) {