From: Pavan Rallabhandi <pavan.rallabhandi@sandisk.com>
Date: Tue, 5 Aug 2014 11:51:35 +0000 (+0530)
Subject: TrackedOp: Removed redundant lock in OpTracker::_mark_event()
X-Git-Tag: v0.86~236^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=f680a24b4ddd5fe08b1c7ee81ac771925598bca3;p=ceph.git

TrackedOp: Removed redundant lock in OpTracker::_mark_event()

ops_in_flight_lock seems redundant in OpTracker::_mark_event()
and this lock is highly contended for. Removing the same
is giving a significant performance boost.

Signed-off-by: Pavan Rallabhandi <pavan.rallabhandi@sandisk.com>
---

diff --git a/src/common/TrackedOp.cc b/src/common/TrackedOp.cc
index 013a66456feb..9ad8e447b4c9 100644
--- a/src/common/TrackedOp.cc
+++ b/src/common/TrackedOp.cc
@@ -231,7 +231,6 @@ void OpTracker::mark_event(TrackedOp *op, const string &dest, utime_t time)
 void OpTracker::_mark_event(TrackedOp *op, const string &evt,
 			    utime_t time)
 {
-  Mutex::Locker locker(ops_in_flight_lock);
   stringstream ss;
   op->_dump_op_descriptor(ss);
   dout(5) << //"reqid: " << op->get_reqid() <<