From: Sage Weil <sage@inktank.com>
Date: Thu, 29 Aug 2013 21:27:46 +0000 (-0700)
Subject: osd/ReplicatedPG: remove debug lines from snapset_context get/put
X-Git-Tag: v0.69~32^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=00b6a94c2d5b0a62a518ec14d9f3b962983dea1d;p=ceph.git

osd/ReplicatedPG: remove debug lines from snapset_context get/put

The dout() prefix does get_osdmap(), which requires (and asserts) that we
hold the pg lock, but in some cases we do not, notably
ReplicatedPG::object_context_destructor_callback.

Signed-off-by: Sage Weil <sage@inktank.com>
---

diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc
index 86d2db51cdb9..f2bf36e82d6a 100644
--- a/src/osd/ReplicatedPG.cc
+++ b/src/osd/ReplicatedPG.cc
@@ -4765,7 +4765,6 @@ SnapSetContext *ReplicatedPG::create_snapset_context(const object_t& oid)
 {
   Mutex::Locker l(snapset_contexts_lock);
   SnapSetContext *ssc = new SnapSetContext(oid);
-  dout(10) << "create_snapset_context " << ssc << " " << ssc->oid << dendl;
   _register_snapset_context(ssc);
   ssc->ref++;
   return ssc;
@@ -4803,8 +4802,6 @@ SnapSetContext *ReplicatedPG::get_snapset_context(const object_t& oid,
     }
   }
   assert(ssc);
-  dout(10) << "get_snapset_context " << ssc->oid << " "
-	   << ssc->ref << " -> " << (ssc->ref+1) << dendl;
   ssc->ref++;
   return ssc;
 }
@@ -4812,8 +4809,6 @@ SnapSetContext *ReplicatedPG::get_snapset_context(const object_t& oid,
 void ReplicatedPG::put_snapset_context(SnapSetContext *ssc)
 {
   Mutex::Locker l(snapset_contexts_lock);
-  dout(10) << "put_snapset_context " << ssc->oid << " "
-	   << ssc->ref << " -> " << (ssc->ref-1) << dendl;
   --ssc->ref;
   if (ssc->ref == 0) {
     if (ssc->registered)