Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(411)

Unified Diff: tools/plot-timer-events

Issue 12040075: Log event start and event end separately when using --log-timer-events. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/vm-state-inl.h ('k') | tools/plot-timer-events.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/plot-timer-events
diff --git a/tools/plot-timer-events b/tools/plot-timer-events
index 581e0ae333081dc421b976ca4b70d220f148980a..0f85699764942469773ec04dae229fbce92d3510 100755
--- a/tools/plot-timer-events
+++ b/tools/plot-timer-events
@@ -41,18 +41,25 @@ else
$d8_exec --nocrankshaft --prof --logfile $calibration_log \
--log-timer-events -e "$calibration_script" > /dev/null
- t_1=`grep "V8.Execute" $calibration_log | tail -n1 | awk -F, '{print $4}'`
- n_1=`grep "timer-event" $calibration_log | wc -l`
+ t_1_start=`grep "timer-event-start,\"V8.Execute\"" $calibration_log \
+ | tail -n1 | awk -F, '{print $3}'`
+ t_1_end=`grep "timer-event-end,\"V8.Execute\"" $calibration_log \
+ | tail -n1 | awk -F, '{print $3}'`
+ n_1=`grep "timer-event\|tick" $calibration_log | wc -l`
$d8_exec --nocrankshaft --prof --logfile $calibration_log \
--log-internal-timer-events -e "$calibration_script" > /dev/null
- t_2=`grep "V8.Execute" $calibration_log | tail -n1 | awk -F, '{print $4}'`
- n_2=`grep "timer-event" $calibration_log | wc -l`
+ t_2_start=`grep "timer-event-start,\"V8.Execute\"" $calibration_log \
+ | tail -n1 | awk -F, '{print $3}'`
+ t_2_end=`grep "timer-event-end,\"V8.Execute\"" $calibration_log \
+ | tail -n1 | awk -F, '{print $3}'`
+ n_2=`grep "timer-event\|tick" $calibration_log | wc -l`
rm $calibration_log
# Overhead in picoseconds.
- distortion=`echo "1000*($t_1 - $t_2)/($n_1 - $n_2)" | bc`
+ distortion=`echo "1000*(($t_1_end - $t_1_start) - ($t_2_end - $t_2_start)) \
+ / ($n_1 - $n_2)" | bc`
fi
if [ -n "$PLOT_RANGE" ]; then
« no previous file with comments | « src/vm-state-inl.h ('k') | tools/plot-timer-events.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698