Fixed BakerGC diagnostics.

brixen · brixen · commit a292fce8c9e0 · 2015-07-07T16:20:06.000-07:00
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();

Original file line number	Diff line number	Diff line change
`@@ -62,20 +62,21 @@ namespace rubinius {`
`62`	`62`	`public:`
`63`	`63`	`const static int cPercentTiles = 10;`
`64`	`64`
`65`		`- double occupancy_histo_[cPercentTiles];`
	`65`	`+ int64_t occupancy_histo_[cPercentTiles];`
`66`	`66`
`67`	`67`	`Diagnostics()`
`68`	`68`	`: diagnostics::Diagnostics()`
`69`	`69`	`{`
`70`	`70`	`for(int i = 0; i < cPercentTiles; i++) {`
`71`		`- occupancy_histo_[i] = 0.0;`
	`71`	`+ occupancy_histo_[i] = 0;`
`72`	`72`	`}`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`void record_occupancy(double percentage) {`
`76`	`76`	`if(percentage < 0.0 \|\| percentage > 100.0) return;`
`77`	`77`
`78`	`78`	`occupancy_histo_[int(percentage / cPercentTiles)]++;`
	`79`	`+ modify();`
`79`	`80`	`}`
`80`	`81`
`81`	`82`	`void log();`