osd/PeeringState: cleanup dead code in PeeringState::init

Argument 'backfill' in PeeringState::init is always set to
false. Commit log shows this is originally for _create_lock_pg
which is removed in d9dcaa79b779. So, just remove dead code.

Signed-off-by: Misono Tomohiro <misono.tomohiro@jp.fujitsu.com>

diff --git a/src/crimson/osd/osd.cc b/src/crimson/osd/osd.cc
index ab9862ae555658b9bf36dc61ab3a8ac72cb424d6..99854546dd72a58a48dae81422955223dc08c396 100644 (file)
--- a/src/crimson/osd/osd.cc
+++ b/src/crimson/osd/osd.cc
@@ -940,7 +940,6 @@ seastar::future<Ref<PG>> OSD::handle_pg_create_info(
           acting_primary,
           info->history,
           info->past_intervals,
-          false,
           rctx.transaction);
 
         return shard_services.start_operation<PGAdvanceMap>(

diff --git a/src/crimson/osd/pg.cc b/src/crimson/osd/pg.cc
index 35b41fbc2944b87df9fbe9f5dbf2b16887cffbcd..9693680154b9eef4d0e8dd3670fa1700db8be17e 100644 (file)
--- a/src/crimson/osd/pg.cc
+++ b/src/crimson/osd/pg.cc
@@ -405,12 +405,11 @@ void PG::init(
   const vector<int>& newacting, int new_acting_primary,
   const pg_history_t& history,
   const PastIntervals& pi,
-  bool backfill,
   ObjectStore::Transaction &t)
 {
   peering_state.init(
     role, newup, new_up_primary, newacting,
-    new_acting_primary, history, pi, backfill, t);
+    new_acting_primary, history, pi, t);
 }
 
 seastar::future<> PG::read_state(crimson::os::FuturizedStore* store)

diff --git a/src/crimson/osd/pg.h b/src/crimson/osd/pg.h
index 45958c4b8e2ffea1a9707effb55fa2c4cf89c91c..8ec5a06ebb65b00da564d7ea3c6d36a637460bb0 100644 (file)
--- a/src/crimson/osd/pg.h
+++ b/src/crimson/osd/pg.h
@@ -465,7 +465,6 @@ public:
     int acting_primary,
     const pg_history_t& history,
     const PastIntervals& pim,
-    bool backfill,
     ceph::os::Transaction &t);
 
   seastar::future<> read_state(crimson::os::FuturizedStore* store);

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 1cd3cf9fb20ff79adfd75920d56015ae5ffcf8d5..52101be2d0d9885d4b8fa533b61ac248e374a822 100644 (file)
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -4953,7 +4953,6 @@ PGRef OSD::handle_pg_create_info(const OSDMapRef& osdmap,
     acting_primary,
     info->history,
     info->past_intervals,
-    false,
     rctx.transaction);
 
   pg->init_collection_pool_opts();

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 82c2bf2bf8fd01f2d686f9a29c65938209efd063..d765995782cb6840958d32ddf401b026423765a5 100644 (file)
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -859,12 +859,11 @@ void PG::init(
   const vector<int>& newacting, int new_acting_primary,
   const pg_history_t& history,
   const PastIntervals& pi,
-  bool backfill,
   ObjectStore::Transaction &t)
 {
   recovery_state.init(
     role, newup, new_up_primary, newacting,
-    new_acting_primary, history, pi, backfill, t);
+    new_acting_primary, history, pi, t);
 }
 
 void PG::shutdown()

diff --git a/src/osd/PG.h b/src/osd/PG.h
index 3ac023c1f8469d0a459cd4f7aca55ea648780fe7..b977c98e056583a7796ceb5676aae565baec2afe 100644 (file)
--- a/src/osd/PG.h
+++ b/src/osd/PG.h
@@ -337,7 +337,6 @@ public:
     int acting_primary,
     const pg_history_t& history,
     const PastIntervals& pim,
-    bool backfill,
     ObjectStore::Transaction &t);
 
   /// read existing pg state off disk

diff --git a/src/osd/PeeringState.cc b/src/osd/PeeringState.cc
index a1e59c4bcd72e10165e11e8a9d7ce9fde765ee7e..5ae0884821282e435d91fdde309ac67aa78af680 100644 (file)
--- a/src/osd/PeeringState.cc
+++ b/src/osd/PeeringState.cc
@@ -3862,7 +3862,6 @@ void PeeringState::init(
   const vector<int>& newacting, int new_acting_primary,
   const pg_history_t& history,
   const PastIntervals& pi,
-  bool backfill,
   ObjectStore::Transaction &t)
 {
   psdout(10) << "init role " << role << " up "
@@ -3891,13 +3890,6 @@ void PeeringState::init(
     pg_log.set_missing_may_contain_deletes();
   }
 
-  if (backfill) {
-    psdout(10) << __func__ << ": Setting backfill" << dendl;
-    info.set_last_backfill(hobject_t());
-    info.last_complete = info.last_update;
-    pg_log.mark_log_for_rewrite();
-  }
-
   on_new_interval();
 
   dirty_info = true;

diff --git a/src/osd/PeeringState.h b/src/osd/PeeringState.h
index d8615ec5d07c327519be452f40a136a1a3a5a846..68237e4d8d09efc43b65fa00360a66c5b90afd0a 100644 (file)
--- a/src/osd/PeeringState.h
+++ b/src/osd/PeeringState.h
@@ -1719,7 +1719,6 @@ public:
     const std::vector<int>& newacting, int new_acting_primary,
     const pg_history_t& history,
     const PastIntervals& pi,
-    bool backfill,
     ObjectStore::Transaction &t);
 
   /// Init pg instance from disk state