]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
common: eradicate buffer_track_c_str in bufferlist. 25424/head
authorRadoslaw Zarzynski <rzarzyns@redhat.com>
Thu, 6 Dec 2018 18:04:37 +0000 (19:04 +0100)
committerRadoslaw Zarzynski <rzarzyns@redhat.com>
Thu, 6 Dec 2018 21:07:16 +0000 (22:07 +0100)
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
src/common/buffer.cc
src/include/buffer.h

index f48f865028a17526b778baeadb26b5eb604d830b..2012c8d272452f80a8166c98fbf588523dcf5174 100644 (file)
@@ -107,17 +107,6 @@ using namespace ceph;
     return buffer_missed_crc;
   }
 
-  static std::atomic<unsigned> buffer_c_str_accesses { 0 };
-
-  static bool buffer_track_c_str = get_env_bool("CEPH_BUFFER_TRACK");
-
-  void buffer::track_c_str(bool b) {
-    buffer_track_c_str = b;
-  }
-  int buffer::get_c_str_accesses() {
-    return buffer_c_str_accesses;
-  }
-
 #ifdef CEPH_HAVE_SETPIPE_SZ
   static std::atomic<unsigned> buffer_max_pipe_size { 0 };
   int update_max_pipe_size() {
@@ -676,26 +665,18 @@ using namespace ceph;
 
   const char *buffer::ptr::c_str() const {
     ceph_assert(_raw);
-    if (unlikely(buffer_track_c_str))
-      buffer_c_str_accesses++;
     return _raw->get_data() + _off;
   }
   char *buffer::ptr::c_str() {
     ceph_assert(_raw);
-    if (unlikely(buffer_track_c_str))
-      buffer_c_str_accesses++;
     return _raw->get_data() + _off;
   }
   const char *buffer::ptr::end_c_str() const {
     ceph_assert(_raw);
-    if (unlikely(buffer_track_c_str))
-      buffer_c_str_accesses++;
     return _raw->get_data() + _off + _len;
   }
   char *buffer::ptr::end_c_str() {
     ceph_assert(_raw);
-    if (unlikely(buffer_track_c_str))
-      buffer_c_str_accesses++;
     return _raw->get_data() + _off + _len;
   }
 
index 56c5e8154e2866e42d63a2582aa836b2172fe8ae..c563aff5ff37d465f2dfa792c36f803aadb04905 100644 (file)
@@ -134,11 +134,6 @@ namespace buffer CEPH_BUFFER_API {
   /// enable/disable tracking of cached crcs
   void track_cached_crc(bool b);
 
-  /// count of calls to buffer::ptr::c_str()
-  int get_c_str_accesses();
-  /// enable/disable tracking of buffer::ptr::c_str() calls
-  void track_c_str(bool b);
-
   /*
    * an abstract raw buffer.  with a reference count.
    */