]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
osd: disable potentially dangerous dead code
authorSage Weil <sage@newdream.net>
Thu, 19 Aug 2010 23:06:24 +0000 (16:06 -0700)
committerSage Weil <sage@newdream.net>
Thu, 19 Aug 2010 23:06:24 +0000 (16:06 -0700)
Greg got a segfault accessing the OSDOP& first reference. Not clear why,
but this whole block is dead code.

src/osd/ReplicatedPG.cc

index e10e2254f810ac682a3b7d883cee4a1c1f8108dc..3416ca104497e5fc70f48e438fb6ae640e2156ed 100644 (file)
@@ -2135,13 +2135,13 @@ void ReplicatedPG::op_applied(RepGather *repop)
 
   update_stats();
 
+#if 0
   // any completion stuff to do here?
   if (repop->ctx->ops.size()) {
     const sobject_t& soid = repop->ctx->obs->oi.soid;
     OSDOp& first = repop->ctx->ops[0];
 
     switch (first.op.op) { 
-#if 0
     case CEPH_OSD_OP_UNBALANCEREADS:
       dout(-10) << "op_applied  completed unbalance-reads on " << oid << dendl;
       unbalancing_reads.erase(oid);
@@ -2160,7 +2160,6 @@ void ReplicatedPG::op_applied(RepGather *repop)
        }
       */
       break;
-#endif
 
     case CEPH_OSD_OP_WRUNLOCK:
       dout(-10) << "op_applied  completed wrunlock on " << soid << dendl;
@@ -2171,6 +2170,7 @@ void ReplicatedPG::op_applied(RepGather *repop)
       break;
     }   
   }
+#endif
 
   if (!repop->aborted)
     eval_repop(repop);