common/TrackedOp: remove unused 'now' in _dump()

Greg pointed this out in PR #11985

Signed-off-by: John Spray <john.spray@redhat.com>

diff --git a/src/common/TrackedOp.cc b/src/common/TrackedOp.cc
index 9477da62a12ef86e695a18e01e07b22a98c1cf07..8826b720529065b35e820e5ed2071a5ba97f9a01 100644 (file)
--- a/src/common/TrackedOp.cc
+++ b/src/common/TrackedOp.cc
@@ -369,7 +369,7 @@ void TrackedOp::dump(utime_t now, Formatter *f) const
   f->dump_float("duration", get_duration());
   {
     f->open_array_section("type_data");
-    _dump(now, f);
+    _dump(f);
     f->close_section();
   }
 }

diff --git a/src/common/TrackedOp.h b/src/common/TrackedOp.h
index b19cc5e25cb0e3f24c7f6e000caee54b0d905e66..6424af42c094a22c6c1eee85a2ebc7a9041a63b2 100644 (file)
--- a/src/common/TrackedOp.h
+++ b/src/common/TrackedOp.h
@@ -150,7 +150,7 @@ protected:
   { }
 
   /// output any type-specific data you want to get when dump() is called
-  virtual void _dump(utime_t now, Formatter *f) const {}
+  virtual void _dump(Formatter *f) const {}
   /// if you want something else to happen when events are marked, implement
   virtual void _event_marked() {}
   /// return a unique descriptor of the Op; eg the message it's attached to

diff --git a/src/mds/Mutation.cc b/src/mds/Mutation.cc
index 2b39e5929497fbfe4b8db9ea3bc786a76c514dfb..e307920bd8bd83103fa2c9a400f6d6b6b370c50b 100644 (file)
--- a/src/mds/Mutation.cc
+++ b/src/mds/Mutation.cc
@@ -315,10 +315,10 @@ void MDRequestImpl::print(ostream &out) const
 
 void MDRequestImpl::dump(Formatter *f) const
 {
-  _dump(ceph_clock_now(g_ceph_context), f);
+  _dump(f);
 }
 
-void MDRequestImpl::_dump(utime_t now, Formatter *f) const
+void MDRequestImpl::_dump(Formatter *f) const
 {
   f->dump_string("flag_point", state_string());
   f->dump_stream("reqid") << reqid;

diff --git a/src/mds/Mutation.h b/src/mds/Mutation.h
index ed7d5af2028ad104274b7f0b606c9d0953d43c91..515bda8b29e75106e153e855a3bfc039aa24e005 100644 (file)
--- a/src/mds/Mutation.h
+++ b/src/mds/Mutation.h
@@ -333,7 +333,7 @@ struct MDRequestImpl : public MutationImpl, public TrackedOp {
   // TrackedOp stuff
   typedef ceph::shared_ptr<MDRequestImpl> Ref;
 protected:
-  void _dump(utime_t now, Formatter *f) const;
+  void _dump(Formatter *f) const;
   void _dump_op_descriptor_unlocked(ostream& stream) const;
 };
 

diff --git a/src/osd/OpRequest.cc b/src/osd/OpRequest.cc
index 746f8efc9b31a250bbf1c58b6736e6f813ccd20b..2b8d75d858d69e7b495347af78d26ca64e4abb2e 100644 (file)
--- a/src/osd/OpRequest.cc
+++ b/src/osd/OpRequest.cc
@@ -46,7 +46,7 @@ OpRequest::OpRequest(Message *req, OpTracker *tracker) :
   tracker->mark_event(this, "dispatched", request->get_dispatch_stamp());
 }
 
-void OpRequest::_dump(utime_t now, Formatter *f) const
+void OpRequest::_dump(Formatter *f) const
 {
   Message *m = request;
   f->dump_string("flag_point", state_string());

diff --git a/src/osd/OpRequest.h b/src/osd/OpRequest.h
index a5db48dd6a6442e3420ead210cece3698e2c0e1b..19320366a4152f16f404cef13f563d3c0357dde8 100644 (file)
--- a/src/osd/OpRequest.h
+++ b/src/osd/OpRequest.h
@@ -99,7 +99,7 @@ struct OpRequest : public TrackedOp {
     return classes_;
   }
 
-  void _dump(utime_t now, Formatter *f) const;
+  void _dump(Formatter *f) const;
 
   bool has_feature(uint64_t f) const {
     return request->get_connection()->has_feature(f);