From: Sage Weil <sage@redhat.com>
Date: Tue, 13 Mar 2018 22:24:55 +0000 (-0500)
Subject: osd/SnapMapper: print debug line on check failure
X-Git-Tag: v13.1.0~101^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=1f92ca315af3bc7b75c08dd148f5e6477152e652;p=ceph-ci.git

osd/SnapMapper: print debug line on check failure

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

diff --git a/src/osd/SnapMapper.cc b/src/osd/SnapMapper.cc
index 6998ede3e24..348d7a3e224 100644
--- a/src/osd/SnapMapper.cc
+++ b/src/osd/SnapMapper.cc
@@ -134,6 +134,17 @@ void SnapMapper::object_snaps::decode(bufferlist::iterator &bl)
   DECODE_FINISH(bl);
 }
 
+bool SnapMapper::check(const hobject_t &hoid) const
+{
+  if (hoid.match(mask_bits, match)) {
+    return true;
+  }
+  derr << __func__ << " " << hoid << " mask_bits " << mask_bits
+       << " match 0x" << std::hex << match << std::dec << " is false"
+       << dendl;
+  return false;
+}
+
 int SnapMapper::get_snaps(
   const hobject_t &oid,
   object_snaps *out)
diff --git a/src/osd/SnapMapper.h b/src/osd/SnapMapper.h
index b4361ed7fa6..99a7c831018 100644
--- a/src/osd/SnapMapper.h
+++ b/src/osd/SnapMapper.h
@@ -144,9 +144,7 @@ private:
     MapCacher::Transaction<std::string, bufferlist> *t);
 
   // True if hoid belongs in this mapping based on mask_bits and match
-  bool check(const hobject_t &hoid) const {
-    return hoid.match(mask_bits, match);
-  }
+  bool check(const hobject_t &hoid) const;
 
   int _remove_oid(
     const hobject_t &oid,    ///< [in] oid to remove