PG: clear missing_loc only during async_recovery

Signed-off-by: Neha Ojha <nojha@redhat.com>

diff --git a/src/osd/PrimaryLogPG.cc b/src/osd/PrimaryLogPG.cc
index 7f9bd5cdb08fa3edeff450d46308ec10929a8bb9..a568320e396d82e0e19c1f35a7f24864181c8e6c 100644 (file)
--- a/src/osd/PrimaryLogPG.cc
+++ b/src/osd/PrimaryLogPG.cc
@@ -10107,16 +10107,16 @@ void PrimaryLogPG::issue_repop(RepGather *repop, OpContext *ctx)
   }
 
   dout(10) << __func__ << " missing_loc before: " << missing_loc.get_locations(soid) << dendl;
-  // clear out missing_loc
-  set<pg_shard_t> peers(missing_loc.get_locations(soid));
-  for (set<pg_shard_t>::iterator r = peers.begin();
-       r != peers.end();
-       ++r) {
-    pg_shard_t peer(*r);
-    missing_loc.remove_location(soid, peer);
-  }
 
   if (requires_missing_loc) {
+    // clear out missing_loc
+    set<pg_shard_t> peers(missing_loc.get_locations(soid));
+    for (set<pg_shard_t>::iterator r = peers.begin();
+         r != peers.end();
+         ++r) {
+      pg_shard_t peer(*r);
+      missing_loc.remove_location(soid, peer);
+    }
     for (set<pg_shard_t>::const_iterator i = actingset.begin();
          i != actingset.end();
          ++i) {