From: Samuel Just Date: Wed, 19 Nov 2014 17:47:30 +0000 (-0800) Subject: PGBackend: rename clear_state to clear_recovery_state X-Git-Tag: v0.93~128^2~12 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=46212362cc32e782e4021f3b191f1a34a1bf3770;p=ceph.git PGBackend: rename clear_state to clear_recovery_state This is called when we complete recovery, and does not clear all state. clear_recovery_state is clearer. Signed-off-by: Samuel Just --- diff --git a/src/osd/ECBackend.cc b/src/osd/ECBackend.cc index ee994b666846..a8c75c5de1cb 100644 --- a/src/osd/ECBackend.cc +++ b/src/osd/ECBackend.cc @@ -1156,10 +1156,10 @@ void ECBackend::on_change() } in_progress_client_reads.clear(); shard_to_read_map.clear(); - clear_state(); + clear_recovery_state(); } -void ECBackend::clear_state() +void ECBackend::clear_recovery_state() { recovery_ops.clear(); } diff --git a/src/osd/ECBackend.h b/src/osd/ECBackend.h index 04e9beb6124c..d7e6fbdac590 100644 --- a/src/osd/ECBackend.h +++ b/src/osd/ECBackend.h @@ -83,7 +83,7 @@ public: void check_recovery_sources(const OSDMapRef osdmap); void on_change(); - void clear_state(); + void clear_recovery_state(); void on_flushed(); diff --git a/src/osd/PGBackend.h b/src/osd/PGBackend.h index 528a59e127f2..91b4d105fca6 100644 --- a/src/osd/PGBackend.h +++ b/src/osd/PGBackend.h @@ -314,7 +314,7 @@ * won't be called after on_change() */ virtual void on_change() = 0; - virtual void clear_state() = 0; + virtual void clear_recovery_state() = 0; virtual void on_flushed() = 0; diff --git a/src/osd/ReplicatedBackend.cc b/src/osd/ReplicatedBackend.cc index 5e29c6e4bc6e..486f07994a4a 100644 --- a/src/osd/ReplicatedBackend.cc +++ b/src/osd/ReplicatedBackend.cc @@ -199,7 +199,7 @@ bool ReplicatedBackend::handle_message( return false; } -void ReplicatedBackend::clear_state() +void ReplicatedBackend::clear_recovery_state() { // clear pushing/pulling maps pushing.clear(); @@ -218,7 +218,7 @@ void ReplicatedBackend::on_change() if (i->second.on_applied) delete i->second.on_applied; } - clear_state(); + clear_recovery_state(); } void ReplicatedBackend::on_flushed() diff --git a/src/osd/ReplicatedBackend.h b/src/osd/ReplicatedBackend.h index 7ce90da1e500..5ad22bfc0b22 100644 --- a/src/osd/ReplicatedBackend.h +++ b/src/osd/ReplicatedBackend.h @@ -70,7 +70,7 @@ public: ); void on_change(); - void clear_state(); + void clear_recovery_state(); void on_flushed(); class RPCRecPred : public IsRecoverablePredicate { diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc index ae2339abd31b..f0860e6608d5 100644 --- a/src/osd/ReplicatedPG.cc +++ b/src/osd/ReplicatedPG.cc @@ -10393,7 +10393,7 @@ void ReplicatedPG::_clear_recovery_state() assert(backfills_in_flight.empty()); pending_backfill_updates.clear(); recovering.clear(); - pgbackend->clear_state(); + pgbackend->clear_recovery_state(); } void ReplicatedPG::cancel_pull(const hobject_t &soid)