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 013a66456feb208336c446b8381eec3f411054e1..9ad8e447b4c980ca1fca082a19e47d0a17a05df2 100644 (file)
--- 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() <<