[Tick processor] Add an option to the tick-processor to print the summary.

tools/tickprocessor.js

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 138 matching lines @@
     cppEntriesProvider,
     separateIc,
     callGraphSize,
     ignoreUnknown,
     stateFilter,
     snapshotLogProcessor,
     distortion,
     range,
     sourceMap,
     timedRange,
-    pairwiseTimedRange) {
+    pairwiseTimedRange,
+    printSummary) {
   LogReader.call(this, {
       'shared-library': { parsers: [null, parseInt, parseInt],
           processor: this.processSharedLibrary },
       'code-creation': {
           parsers: [null, parseInt, parseInt, parseInt, null, 'var-args'],
           processor: this.processCodeCreation },
       'code-move': { parsers: [parseInt, parseInt],
           processor: this.processCodeMove },
       'code-delete': { parsers: [parseInt],
           processor: this.processCodeDelete },
@@ skipping 70 matching lines @@
   };
 
   this.profile_ = new V8Profile(separateIc);
   this.codeTypes_ = {};
   // Count each tick as a time unit.
   this.viewBuilder_ = new ViewBuilder(1);
   this.lastLogFileName_ = null;
 
   this.generation_ = 1;
   this.currentProducerProfile_ = null;
+  this.printSummary_ = printSummary;
 };
 inherits(TickProcessor, LogReader);
 
 
 TickProcessor.VmStates = {
   JS: 0,
   GC: 1,
   COMPILER: 2,
   OTHER: 3,
   EXTERNAL: 4,
@@ skipping 189 matching lines @@
   var flatProfile = this.profile_.getFlatProfile();
   var flatView = this.viewBuilder_.buildView(flatProfile);
   // Sort by self time, desc, then by name, desc.
   flatView.sort(function(rec1, rec2) {
       return rec2.selfTime - rec1.selfTime ||
           (rec2.internalFuncName < rec1.internalFuncName ? -1 : 1); });
   var totalTicks = this.ticks_.total;
   if (this.ignoreUnknown_) {
     totalTicks -= this.ticks_.unaccounted;
   }
+  var printAllTicks = this.printSummary_ == undefined ||

[Jakob Kummerow, 2015/09/03 08:37:08]

How about simply:

var printAllTicks = !this.onlySummary;

[gdeepti1, 2015/09/03 17:50:21]

Done.

+      this.printSummary_ == false;
 
   // Count library ticks
   var flatViewNodes = flatView.head.children;
   var self = this;
 
   var libraryTicks = 0;
-  this.printHeader('Shared libraries');
+  if(printAllTicks) this.printHeader('Shared libraries');
   this.printEntries(flatViewNodes, totalTicks, null,
       function(name) { return self.isSharedLibrary(name); },
-      function(rec) { libraryTicks += rec.selfTime; });
+      function(rec) { libraryTicks += rec.selfTime; }, printAllTicks);
   var nonLibraryTicks = totalTicks - libraryTicks;
 
   var jsTicks = 0;
-  this.printHeader('JavaScript');
+  if(printAllTicks) this.printHeader('JavaScript');
   this.printEntries(flatViewNodes, totalTicks, nonLibraryTicks,
       function(name) { return self.isJsCode(name); },
-      function(rec) { jsTicks += rec.selfTime; });
+      function(rec) { jsTicks += rec.selfTime; }, printAllTicks);
 
   var cppTicks = 0;
-  this.printHeader('C++');
+  if(printAllTicks) this.printHeader('C++');
   this.printEntries(flatViewNodes, totalTicks, nonLibraryTicks,
       function(name) { return self.isCppCode(name); },
-      function(rec) { cppTicks += rec.selfTime; });
+      function(rec) { cppTicks += rec.selfTime; }, printAllTicks);
 
   this.printHeader('Summary');
   this.printLine('JavaScript', jsTicks, totalTicks, nonLibraryTicks);
   this.printLine('C++', cppTicks, totalTicks, nonLibraryTicks);
   this.printLine('GC', this.ticks_.gc, totalTicks, nonLibraryTicks);
   this.printLine('Shared libraries', libraryTicks, totalTicks, null);
   if (!this.ignoreUnknown_ && this.ticks_.unaccounted > 0) {
     this.printLine('Unaccounted', this.ticks_.unaccounted,
                    this.ticks_.total, null);
   }
 
-  print('\n [C++ entry points]:');
-  print('   ticks    cpp   total   name');
-  var c_entry_functions = this.profile_.getCEntryProfile();
-  var total_c_entry = c_entry_functions[0].ticks;
-  for (var i = 1; i < c_entry_functions.length; i++) {
-    c = c_entry_functions[i];
-    this.printLine(c.name, c.ticks, total_c_entry, totalTicks);
-  }
+  if(printAllTicks) {
+    print('\n [C++ entry points]:');
+    print('   ticks    cpp   total   name');
+    var c_entry_functions = this.profile_.getCEntryProfile();
+    var total_c_entry = c_entry_functions[0].ticks;
+    for (var i = 1; i < c_entry_functions.length; i++) {
+      c = c_entry_functions[i];
+      this.printLine(c.name, c.ticks, total_c_entry, totalTicks);
+    }
 
-  this.printHeavyProfHeader();
-  var heavyProfile = this.profile_.getBottomUpProfile();
-  var heavyView = this.viewBuilder_.buildView(heavyProfile);
-  // To show the same percentages as in the flat profile.
-  heavyView.head.totalTime = totalTicks;
-  // Sort by total time, desc, then by name, desc.
-  heavyView.sort(function(rec1, rec2) {
-      return rec2.totalTime - rec1.totalTime ||
-          (rec2.internalFuncName < rec1.internalFuncName ? -1 : 1); });
-  this.printHeavyProfile(heavyView.head.children);
-};
+    this.printHeavyProfHeader();
+    var heavyProfile = this.profile_.getBottomUpProfile();
+    var heavyView = this.viewBuilder_.buildView(heavyProfile);
+    // To show the same percentages as in the flat profile.
+    heavyView.head.totalTime = totalTicks;
+    // Sort by total time, desc, then by name, desc.
+    heavyView.sort(function(rec1, rec2) {
+        return rec2.totalTime - rec1.totalTime ||
+            (rec2.internalFuncName < rec1.internalFuncName ? -1 : 1); });
+    this.printHeavyProfile(heavyView.head.children);
+    }

[Michael Achenbach, 2015/09/03 08:19:11]

nit: indentation of } and };

[gdeepti1, 2015/09/03 17:50:21]

Done.

+  };
 
 
 function padLeft(s, len) {
   s = s.toString();
   if (s.length < len) {
     var padLength = len - s.length;
     if (!(padLength in padLeft)) {
       padLeft[padLength] = new Array(padLength + 1).join(' ');
     }
     s = padLeft[padLength] + s;
@@ skipping 70 matching lines @@
   var column = lc.column - 1;
   var entry = this.sourceMap.findEntry(lineNumber, column);
   var sourceFile = entry[2];
   var sourceLine = entry[3] + 1;
   var sourceColumn = entry[4] + 1;
 
   return sourceFile + ':' + sourceLine + ':' + sourceColumn + ' -> ' + funcName;
 };
 
 TickProcessor.prototype.printEntries = function(
-    profile, totalTicks, nonLibTicks, filterP, callback) {
+    profile, totalTicks, nonLibTicks, filterP, callback, printAllTicks) {
   var that = this;
   this.processProfile(profile, filterP, function (rec) {
     if (rec.selfTime == 0) return;
     callback(rec);
     var funcName = that.formatFunctionName(rec.internalFuncName);
-    that.printLine(funcName, rec.selfTime, totalTicks, nonLibTicks);
+    if(printAllTicks) {
+      that.printLine(funcName, rec.selfTime, totalTicks, nonLibTicks);
+    }
   });
 };
 
 
 TickProcessor.prototype.printHeavyProfile = function(profile, opt_indent) {
   var self = this;
   var indent = opt_indent || 0;
   var indentStr = padLeft('', indent);
   this.processProfile(profile, function() { return true; }, function (rec) {
     // Cut off too infrequent callers.
@@ skipping 257 matching lines @@
         'Specify snapshot log file to use (e.g. --snapshot-log=snapshot.log)'],
     '--range': ['range', 'auto,auto',
         'Specify the range limit as [start],[end]'],
     '--distortion': ['distortion', 0,
         'Specify the logging overhead in picoseconds'],
     '--source-map': ['sourceMap', null,
         'Specify the source map that should be used for output'],
     '--timed-range': ['timedRange', true,
         'Ignore ticks before first and after last Date.now() call'],
     '--pairwise-timed-range': ['pairwiseTimedRange', true,
-        'Ignore ticks outside pairs of Date.now() calls']
+        'Ignore ticks outside pairs of Date.now() calls'],
+    '--print-summary': ['printSummary', true,

[Jakob Kummerow, 2015/09/03 08:37:08]

I'd call the flag --only-summary (and the corresponding variables "onlySummary")
to make its effect more obvious.

[gdeepti1, 2015/09/03 17:50:21]

Done.

+        'Print only tick summary, exclude other information']
   };
   this.argsDispatch_['--js'] = this.argsDispatch_['-j'];
   this.argsDispatch_['--gc'] = this.argsDispatch_['-g'];
   this.argsDispatch_['--compiler'] = this.argsDispatch_['-c'];
   this.argsDispatch_['--other'] = this.argsDispatch_['-o'];
   this.argsDispatch_['--external'] = this.argsDispatch_['-e'];
   this.argsDispatch_['--ptr'] = this.argsDispatch_['--pairwise-timed-range'];
 };
 
 
 ArgumentsProcessor.DEFAULTS = {
   logFileName: 'v8.log',
   snapshotLogFileName: null,
   platform: 'unix',
   stateFilter: null,
   callGraphSize: 5,
   ignoreUnknown: false,
   separateIc: false,
   targetRootFS: '',
   nm: 'nm',
   range: 'auto,auto',
   distortion: 0,
   timedRange: false,
-  pairwiseTimedRange: false
+  pairwiseTimedRange: false,
+  printSummary: false
 };
 
 
 ArgumentsProcessor.prototype.parse = function() {
   while (this.args_.length) {
     var arg = this.args_.shift();
     if (arg.charAt(0) != '-') {
       this.result_.logFileName = arg;
       continue;
     }
@@ skipping 39 matching lines @@
     for (var synArg in this.argsDispatch_) {
       if (arg !== synArg && dispatch === this.argsDispatch_[synArg]) {
         synonyms.push(synArg);
         delete this.argsDispatch_[synArg];
       }
     }
     print('  ' + padRight(synonyms.join(', '), 20) + " " + dispatch[2]);
   }
   quit(2);
 };