osd: check for is_stretch_pool() in stretch_set_can_peer()

This makes it easy and cheap to call from non-stretch contexts.

Signed-off-by: Greg Farnum <gfarnum@redhat.com>

diff --git a/src/osd/osd_types.cc b/src/osd/osd_types.cc
index 82bc693dfe6c99817e0c6b2d1b24db041ac6a3ae..07bdf025a45fe91f7b693f66e3b3de43eab0ee59 100644 (file)
--- a/src/osd/osd_types.cc
+++ b/src/osd/osd_types.cc
@@ -2239,6 +2239,7 @@ void pg_pool_t::decode(ceph::buffer::list::const_iterator& bl)
 bool pg_pool_t::stretch_set_can_peer(const set<int>& want, const OSDMap& osdmap,
                                     std::ostream * out) const
 {
+  if (!is_stretch_pool()) return true;
   const uint32_t barrier_id = peering_crush_bucket_barrier;
   const uint32_t barrier_count = peering_crush_bucket_count;
   set<int> ancestors;

diff --git a/src/osd/osd_types.h b/src/osd/osd_types.h
index 31195adae84e4742ebd9904f2e066c503d4c5724..c0fe8f781c21ea6da4c04107c4d44ef21dfbb830 100644 (file)
--- a/src/osd/osd_types.h
+++ b/src/osd/osd_types.h
@@ -1521,6 +1521,7 @@ public:
                            std::ostream *out) const;
   bool stretch_set_can_peer(const vector<int>& want, const OSDMap& osdmap,
                            std::ostream *out) const {
+    if (!is_stretch_pool()) return true;
     set<int> swant;
     for (auto i : want) swant.insert(i);
     return stretch_set_can_peer(swant, osdmap, out);