From: Josh Durgin <josh.durgin@inktank.com>
Date: Fri, 7 Feb 2014 02:17:06 +0000 (-0800)
Subject: ReplicatedPG: don't copy the ops vector into OpContext
X-Git-Tag: v0.78~154^2~4
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=77436989078fa6261fe50ed80ff2df8f12fcaca7;p=ceph.git

ReplicatedPG: don't copy the ops vector into OpContext

There's no need to copy it, and if we refer to the same vector from
the original MOSDOp, we can modify rvals for individual ops, since the
MOSDOp ops vector is copied to the MOSDOpReply when it is constructed.

Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
---

diff --git a/src/osd/ReplicatedPG.h b/src/osd/ReplicatedPG.h
index b6a80e2594c..8e0876e58bb 100644
--- a/src/osd/ReplicatedPG.h
+++ b/src/osd/ReplicatedPG.h
@@ -409,7 +409,7 @@ public:
   struct OpContext {
     OpRequestRef op;
     osd_reqid_t reqid;
-    vector<OSDOp> ops;
+    vector<OSDOp> &ops;
 
     const ObjectState *obs; // Old objectstate
     const SnapSet *snapset; // Old snapset