PG: it's not necessary to call build_inc_scrub_map in build_scrub_map

Because we have called osr.flush(), it's safe to tag map.valid_through
as last_update.   We will still have to catch up once we have stopped
writes and allowed the filestore to catch up anyway.

Signed-off-by: Samuel Just <samuel.just@dreamhost.com>

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index d8548e029f047ef1e992c8d98742c800dd8f33ad..aecffccc82c4a85898d74e567aa075d4ca5917f8 100644 (file)
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -2765,11 +2765,6 @@ void PG::build_scrub_map(ScrubMap &map)
 
   dout(10) << "PG relocked, finalizing" << dendl;
 
-  // Catch up
-  ScrubMap incr;
-  build_inc_scrub_map(incr, map.valid_through);
-  map.merge_incr(incr);
-
   // pg attrs
   osd->store->collection_getattrs(coll, map.attrs);