Removed perfmons: serializer_data_blocks_written, serializer_data_extents_reclaimed.

Daniel Mewes · Daniel Mewes · commit 2770e9e6555b · 2014-01-22T15:03:08.000-08:00
diff --git a/src/serializer/log/data_block_manager.cc b/src/serializer/log/data_block_manager.cc
@@ -724,8 +724,6 @@ data_block_manager_t::many_writes(const std::vector<buf_write_info_t> &writes,
         }
     }
 
-    stats->pm_serializer_data_blocks_written += writes.size();
-
     struct intermediate_cb_t : public iocallback_t {
         virtual void on_io_complete() {
             --ops_remaining;
@@ -795,7 +793,6 @@ data_block_manager_t::many_writes(const std::vector<buf_write_info_t> &writes,
 
 void data_block_manager_t::destroy_entry(gc_entry_t *entry) {
     rassert(entry != NULL);
-    ++stats->pm_serializer_data_extents_reclaimed;
     entry->destroy();
 }
 
diff --git a/src/serializer/log/log_serializer.cc b/src/serializer/log/log_serializer.cc
@@ -123,9 +123,7 @@ log_serializer_stats_t::log_serializer_stats_t(perfmon_collection_t *parent)
       pm_serializer_lba_extents(),
       pm_serializer_data_extents(),
       pm_serializer_data_extents_allocated(),
-      pm_serializer_data_extents_reclaimed(),
       pm_serializer_data_extents_gced(),
-      pm_serializer_data_blocks_written(),
       pm_serializer_old_garbage_block_bytes(),
       pm_serializer_old_total_block_bytes(),
       pm_serializer_lba_gcs(),
@@ -141,9 +139,7 @@ log_serializer_stats_t::log_serializer_stats_t(perfmon_collection_t *parent)
           &pm_serializer_lba_extents, "serializer_lba_extents",
           &pm_serializer_data_extents, "serializer_data_extents",
           &pm_serializer_data_extents_allocated, "serializer_data_extents_allocated",
-          &pm_serializer_data_extents_reclaimed, "serializer_data_extents_reclaimed",
           &pm_serializer_data_extents_gced, "serializer_data_extents_gced",
-          &pm_serializer_data_blocks_written, "serializer_data_blocks_written",
           &pm_serializer_old_garbage_block_bytes, "serializer_old_garbage_block_bytes",
           &pm_serializer_old_total_block_bytes, "serializer_old_total_block_bytes",
           &pm_serializer_lba_gcs, "serializer_lba_gcs",
diff --git a/src/serializer/log/stats.hpp b/src/serializer/log/stats.hpp
@@ -24,9 +24,7 @@ struct log_serializer_stats_t {
     /* used in serializer/log/data_block_manager.cc */
     perfmon_counter_t pm_serializer_data_extents;
     perfmon_counter_t pm_serializer_data_extents_allocated;
-    perfmon_counter_t pm_serializer_data_extents_reclaimed;
     perfmon_counter_t pm_serializer_data_extents_gced;
-    perfmon_counter_t pm_serializer_data_blocks_written;
     perfmon_counter_t pm_serializer_old_garbage_block_bytes;
     perfmon_counter_t pm_serializer_old_total_block_bytes;
 

Original file line number	Diff line number	Diff line change
`@@ -724,8 +724,6 @@ data_block_manager_t::many_writes(const std::vector<buf_write_info_t> &writes,`
`724`	`724`	`}`
`725`	`725`	`}`
`726`	`726`
`727`		`- stats->pm_serializer_data_blocks_written += writes.size();`
`728`		`-`
`729`	`727`	`struct intermediate_cb_t : public iocallback_t {`
`730`	`728`	`virtual void on_io_complete() {`
`731`	`729`	`--ops_remaining;`
`@@ -795,7 +793,6 @@ data_block_manager_t::many_writes(const std::vector<buf_write_info_t> &writes,`
`795`	`793`
`796`	`794`	`void data_block_manager_t::destroy_entry(gc_entry_t *entry) {`
`797`	`795`	`rassert(entry != NULL);`
`798`		`- ++stats->pm_serializer_data_extents_reclaimed;`
`799`	`796`	`entry->destroy();`
`800`	`797`	`}`
`801`	`798`