From a9a370be2d8155b696ebe2866febb0571da5740f Mon Sep 17 00:00:00 2001 From: Samuel Just Date: Fri, 2 Aug 2013 11:58:52 -0700 Subject: [PATCH] PG: set !flushed in Reset() Otherwise, we might serve a pull before we start_flush in the ReplicaActive constructor. Fixes: #5799 Signed-off-by: Samuel Just Reviewed-by: Sage Weil (cherry picked from commit 9e7d6d547e0e8a6db6ba611882afa9bf74ea0195) --- src/osd/PG.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/osd/PG.cc b/src/osd/PG.cc index b547840a0f2..3b06d89be4b 100644 --- a/src/osd/PG.cc +++ b/src/osd/PG.cc @@ -5004,7 +5004,7 @@ void PG::start_flush(ObjectStore::Transaction *t, FlushStateRef flush_trigger( new FlushState(this, get_osdmap()->get_epoch())); t->nop(); - flushed = false; + assert(!flushed); on_applied->push_back(new ContainerContext(flush_trigger)); on_safe->push_back(new ContainerContext(flush_trigger)); DeletingStateRef del = osd->deleting_pgs.lookup(info.pgid); @@ -5953,6 +5953,7 @@ PG::RecoveryState::Reset::Reset(my_context ctx) state_name = "Reset"; context< RecoveryMachine >().log_enter(state_name); PG *pg = context< RecoveryMachine >().pg; + pg->flushed = false; pg->set_last_peering_reset(); } -- 2.47.3