]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ReplicatedPG: For MAPEXT, using bl.length() instead of op.extent.length as num_rd_kb. 2530/head
authorJianpeng Ma <jianpeng.ma@intel.com>
Fri, 19 Sep 2014 03:27:57 +0000 (11:27 +0800)
committerJianpeng Ma <jianpeng.ma@intel.com>
Fri, 12 Dec 2014 06:38:30 +0000 (14:38 +0800)
For fiemap, it don't real read the data(offset,length). It only fetech the infos of
extents. So don't use length as num_rd_kb.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
src/osd/ReplicatedPG.cc

index 9addff11125ab4d52841ad966c6a5738da3675c4..579535bbe66105591919a5be20745b47163b0427 100644 (file)
@@ -3367,7 +3367,8 @@ int ReplicatedPG::do_osd_ops(OpContext *ctx, vector<OSDOp>& ops)
        osd_op.outdata.claim(bl);
        if (r < 0)
          result = r;
-       ctx->delta_stats.num_rd_kb += SHIFT_ROUND_UP(op.extent.length, 10);
+       else
+         ctx->delta_stats.num_rd_kb += SHIFT_ROUND_UP(bl.length(), 10);
        ctx->delta_stats.num_rd++;
        dout(10) << " map_extents done on object " << soid << dendl;
       }