Fixed BakerGC diagnostics.

rubinius · Jul 7, 2015 · a292fce · a292fce
1 parent c660bba
commit a292fce
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/vm/gc/baker.cpp b/vm/gc/baker.cpp
@@ -60,8 +60,8 @@ namespace rubinius {
     diagnostics::Diagnostics::log();
 
     utilities::logger::write("baker: diagnostics: " \
-        "10%: %ld, 20%: %ld, 30%: %ld, 40%: %ld, 50%: %ld, " \
-        "60%: %ld, 70%: %ld, 80%: %ld, 90%: %ld",
+        "10%%: %ld, 20%%: %ld, 30%%: %ld, 40%%: %ld, 50%%: %ld, " \
+        "60%%: %ld, 70%%: %ld, 80%%: %ld, 90%%: %ld",
         occupancy_histo_[0], occupancy_histo_[1], occupancy_histo_[2],
         occupancy_histo_[3], occupancy_histo_[4], occupancy_histo_[5],
         occupancy_histo_[6], occupancy_histo_[7], occupancy_histo_[8]);

diff --git a/vm/gc/baker.hpp b/vm/gc/baker.hpp
@@ -62,20 +62,21 @@ namespace rubinius {
     public:
       const static int cPercentTiles = 10;
 
-      double occupancy_histo_[cPercentTiles];
+      int64_t occupancy_histo_[cPercentTiles];
 
       Diagnostics()
         : diagnostics::Diagnostics()
       {
         for(int i = 0; i < cPercentTiles; i++) {
-          occupancy_histo_[i] = 0.0;
+          occupancy_histo_[i] = 0;
         }
       }
 
       void record_occupancy(double percentage) {
         if(percentage < 0.0 || percentage > 100.0) return;
 
         occupancy_histo_[int(percentage / cPercentTiles)]++;
+        modify();
       }
 
       void log();