diff options
| author | Sage Weil <sage@inktank.com> | 2013-08-29 14:27:46 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-08-29 14:27:46 -0700 |
| commit | 00b6a94c2d5b0a62a518ec14d9f3b962983dea1d (patch) | |
| tree | bf4bdffd55e2c362a079044e59fb02762d6e2e65 | |
| parent | 9cc40a52f8d9213c24c83263ca0eae85bd0da6ee (diff) | |
| download | ceph-00b6a94c2d5b0a62a518ec14d9f3b962983dea1d.tar.gz | |
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>
| -rw-r--r-- | src/osd/ReplicatedPG.cc | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc index 86d2db51cdb..f2bf36e82d6 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) |