]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mon/PGMap: use pool's crush_rule directly
authorSage Weil <sage@newdream.net>
Fri, 25 Jun 2021 15:08:39 +0000 (11:08 -0400)
committerSage Weil <sage@newdream.net>
Mon, 28 Jun 2021 22:34:30 +0000 (18:34 -0400)
Signed-off-by: Sage Weil <sage@newdream.net>
src/mon/PGMap.cc

index c8ea4eabdcfc5874d172f7ace12e965d3ae2902d..0bce41af4b478846ac83b3a21692d27f8afe1723 100644 (file)
@@ -771,9 +771,7 @@ void PGMapDigest::dump_pool_stats_full(
     const pool_stat_t &stat = pg_pool_sum.at(pool_id);
 
     const pg_pool_t *pool = osd_map.get_pg_pool(pool_id);
-    int ruleno = osd_map.crush->find_rule(pool->get_crush_rule(),
-                                         pool->get_type(),
-                                         pool->get_size());
+    int ruleno = pool->get_crush_rule();
     int64_t avail;
     if (avail_by_rule.count(ruleno) == 0) {
       // FIXME: we don't guarantee avail_space_by_rule is up-to-date before this function is invoked
@@ -975,9 +973,7 @@ int64_t PGMapDigest::get_pool_free_space(const OSDMap &osd_map,
                                         int64_t poolid) const
 {
   const pg_pool_t *pool = osd_map.get_pg_pool(poolid);
-  int ruleno = osd_map.crush->find_rule(pool->get_crush_rule(),
-                                       pool->get_type(),
-                                       pool->get_size());
+  int ruleno = pool->get_crush_rule();
   int64_t avail;
   avail = get_rule_avail(ruleno);
   if (avail < 0)
@@ -1039,9 +1035,7 @@ void PGMap::get_rules_avail(const OSDMap& osdmap,
     if ((pool_id < 0) || (pg_pool_sum.count(pool_id) == 0))
       continue;
     const pg_pool_t *pool = osdmap.get_pg_pool(pool_id);
-    int ruleno = osdmap.crush->find_rule(pool->get_crush_rule(),
-                                        pool->get_type(),
-                                        pool->get_size());
+    int ruleno = pool->get_crush_rule();
     if (avail_map->count(ruleno) == 0)
       (*avail_map)[ruleno] = get_rule_avail(osdmap, ruleno);
   }