Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- diff --git a/memtest.py b/memtest.py
- index 991912c..8ff4e6a 100644
- --- a/memtest.py
- +++ b/memtest.py
- @@ -135,47 +135,3 @@ class StackTestJerry:
- return -1, 0
- -
- -class RSSTestJerry:
- - def __init__(self, log_file, verbose=False, test_dir='', script=''):
- - self.log_file = log_file
- - self.verbose = verbose
- - self.test_dir = test_dir
- - self.script = script
- -
- - def run(self, runnable, test_name='', test_dir=''):
- - if test_dir:
- - test_path = os.path.join(test_dir, test_name)
- - else:
- - test_path = os.path.join(self.test_dir, test_name)
- -
- - self.log_file.write('Start RSS test on ' + test_name + '\n')
- - if self.verbose:
- - out_file = self.log_file
- - else:
- - out_file = open('/dev/null', 'w')
- -
- - #out_file.write('test_path: ' + test_path + '\n')
- - #out_file.write('stdout:\n')
- -
- - rss_value = 0
- - num_of_measure = 7
- - for i in range(0, num_of_measure):
- - try:
- - return_value = subprocess.check_output([self.script, runnable, test_path], stderr=self.log_file)
- - except subprocess.CalledProcessError as e:
- - out_file.write(e.output)
- - self.log_file.write('Process finished with ' + str(e.returncode) + ' return code.\n')
- - return -1, e.returncode
- -
- - # Strip the newline from the script's output
- - return_value = return_value.rstrip()
- - # Change the unit from kilobytes to bytes
- - return_value = int(return_value) * 1024
- -
- - rss_value += return_value
- -
- - rss_value = int(rss_value / num_of_measure)
- - out_file.write(str(rss_value))
- -
- - return rss_value, 0
- diff --git a/tape_measure.py b/tape_measure.py
- index 0b43abb..fcf9fab 100755
- --- a/tape_measure.py
- +++ b/tape_measure.py
- @@ -11,7 +11,6 @@ from perftest import PerfTest
- from memtest import MemTest
- from memtest import MemTestJerry
- from memtest import StackTestJerry
- -from memtest import RSSTestJerry
- from readelf import ReadElf
- def collect_tests_from_directory(path):
- @@ -63,14 +62,12 @@ def fancy_output(result, options, log_file=sys.stdout):
- log_file.write('{type:20}|{size:10,}\n'.format(type='DATA', size=size_data))
- log_file.write('{type:20}|{size:10,}\n'.format(type='BSS', size=size_bss))
- - if options.memory or options.stack or options.rss or options.performance or options.run_all:
- + if options.memory or options.stack or options.performance or options.run_all:
- line = '{name:30}'.format(name='test name')
- if options.memory or options.run_all:
- line += '|{peak:10}'.format(peak='heap')
- if options.stack or options.run_all:
- line += '|{peak:10}'.format(peak='stack')
- - if options.rss or options.run_all:
- - line += '|{peak:10}'.format(peak='rss')
- if options.performance or options.run_all:
- line += '|{avg:10}|{std_dev:10}'.format(avg='avg', std_dev='std_dev')
- log_file.write(line + '\n')
- @@ -81,8 +78,6 @@ def fancy_output(result, options, log_file=sys.stdout):
- tst_res = result['test_results_mem']
- elif options.stack:
- tst_res = result['test_results_stack']
- - elif options.rss:
- - tst_res = result['test_results_rss']
- for index, test in enumerate(tst_res):
- line = '{name:30}'.format(name=test['test'])
- @@ -90,8 +85,6 @@ def fancy_output(result, options, log_file=sys.stdout):
- line += '|{peak:10}'.format(peak=result['test_results_mem'][index]['memory_peak'])
- if options.stack or options.run_all:
- line += '|{peak:10}'.format(peak=result['test_results_stack'][index]['max_stack'])
- - if options.rss or options.run_all:
- - line += '|{peak:10}'.format(peak=result['test_results_rss'][index]['max_rss'])
- if options.performance or options.run_all:
- line += '|{avg:10}|{std_dev:10.4}'.format(avg=test['avg_time'], std_dev=float(test['std_deviation']))
- line += '\n'
- @@ -140,8 +133,6 @@ if __name__ == '__main__':
- help="comment for the measurements")
- parser.add_option("--remote", action="store", dest="remote", default='',
- help="information about the remote")
- - parser.add_option("--rss", action="store", dest="rss", default='',
- - help="path of the RSS script")
- (options, args) = parser.parse_args()
- @@ -191,17 +182,12 @@ if __name__ == '__main__':
- else:
- stacktest = None
- - if options.rss or options.run_all:
- - rsstest = RSSTestJerry(log_file, options.verbose, options.test_dir, options.rss)
- - else:
- - rsstest = None
- -
- if options.performance or options.run_all:
- perftest = PerfTest(log_file, options.verbose, options.test_dir)
- else:
- perftest = None
- - if memtest or stacktest or rsstest or perftest:
- + if memtest or stacktest or perftest:
- if options.test_list:
- test_list = collect_tests(options.test_list.split())
- else:
- @@ -216,7 +202,6 @@ if __name__ == '__main__':
- test_results_mem = []
- test_results_stack = []
- - test_results_rss = []
- test_results_perf = []
- for test in test_list:
- @@ -232,12 +217,6 @@ if __name__ == '__main__':
- stack_result['max_stack'] = peak
- stack_result['exit_code'] = exit_code
- test_results_stack.append(stack_result)
- - if rsstest:
- - rss_result = { 'test' : test}
- - peak, exit_code = rsstest.run(runnable, test)
- - rss_result['max_rss'] = peak
- - rss_result['exit_code'] = exit_code
- - test_results_rss.append(rss_result)
- if perftest:
- perf_result = { 'test' : test}
- time, std_deviation, exit_code = perftest.run(runnable, test)
- @@ -252,9 +231,6 @@ if __name__ == '__main__':
- if stacktest:
- result['test_results_stack'] = test_results_stack
- - if rsstest:
- - result['test_results_rss'] = test_results_rss
- -
- if perftest:
- result['test_results_perf'] = test_results_perf
- diff --git a/web_chart/compare_view.html b/web_chart/compare_view.html
- index e9b81d7..348a5fb 100644
- --- a/web_chart/compare_view.html
- +++ b/web_chart/compare_view.html
- @@ -37,7 +37,6 @@ textarea {
- <div>
- <input type="checkbox" id="check_heap" value="heap"> Memory peak
- <input type="checkbox" id="check_stack" value="stack" checked> Max stack
- - <input type="checkbox" id="check_rss" value="rss" checked> Max RSS
- <input type="checkbox" id="check_perf" value="perf" checked> Runtime <br>
- </div>
- <input type="submit" value="Generate" onclick="generate_markdown()">
- diff --git a/web_chart/index.html b/web_chart/index.html
- index 8a91bb6..aea6624 100644
- --- a/web_chart/index.html
- +++ b/web_chart/index.html
- @@ -183,10 +183,6 @@
- <td>Max stack</td>
- <td><span id="commit_stack"></span></td>
- </tr>
- - <tr>
- - <td>Max RSS</td>
- - <td><span id="commit_rss"></span></td>
- - </tr>
- <tr>
- <td>Runtime</td>
- <td><span id="commit_runtime"></span></td>
- diff --git a/web_chart/js/compare.js b/web_chart/js/compare.js
- index f026e05..632abe2 100644
- --- a/web_chart/js/compare.js
- +++ b/web_chart/js/compare.js
- @@ -38,7 +38,6 @@ function create_compare_table(data) {
- insert_cell(row, baseline.test_results[i].test, 'compare-table');
- insert_cell(row, baseline.test_results[i].memory_peak, 'compare-table compare-table-data left-border');
- insert_cell(row, baseline.test_results[i].max_stack, 'compare-table compare-table-data left-border');
- - insert_cell(row, baseline.test_results[i].max_rss, 'compare-table compare-table-data left-border');
- insert_cell(row, baseline.test_results[i].avg_time.toFixed(3), 'compare-table compare-table-data right-border');
- }
- @@ -51,7 +50,6 @@ function create_compare_table(data) {
- var row = rows[j];
- insert_cell(row, test_result.memory_peak, 'compare-table compare-table-data left-border');
- insert_cell(row, test_result.max_stack, 'compare-table compare-table-data left-border');
- - insert_cell(row, test_result.max_rss, 'compare-table compare-table-data left-border');
- insert_cell(row, test_result.avg_time.toFixed(3), 'compare-table compare-table-data');
- var percentage_class = 'compare-table compare-table-data ';
- @@ -60,8 +58,6 @@ function create_compare_table(data) {
- insert_cell(row, format_percentage(test_result.memory_gain_percentage), percentage_class + color);
- color = percentage_color(test_result.stack_gain_percentage);
- insert_cell(row, format_percentage(test_result.stack_gain_percentage), percentage_class + color);
- - color = percentage_color(test_result.rss_gain_percentage);
- - insert_cell(row, format_percentage(test_result.rss_gain_percentage), percentage_class + color);
- color = percentage_color(test_result.avg_time_gain_percentage, test_result.avg_time, test_result.std_deviation);
- insert_cell(row, format_percentage(test_result.avg_time_gain_percentage), percentage_class + color + 'right-border');
- }
- @@ -73,7 +69,6 @@ function create_compare_table(data) {
- function generate_markdown() {
- var need_heap = document.getElementById("check_heap").checked;
- var need_stack = document.getElementById("check_stack").checked;
- - var need_rss = document.getElementById("check_rss").checked;
- var need_perf = document.getElementById("check_perf").checked;
- var columns = [];
- var markdown_result = "Benchmark |";
- @@ -89,11 +84,6 @@ function generate_markdown() {
- count++;
- columns.push('max_stack');
- }
- - if (need_rss) {
- - markdown_result += " RSS(bytes) |";
- - count++;
- - columns.push('max_rss');
- - }
- if (need_perf) {
- markdown_result += " Perf(sec) |";
- count++;
- @@ -176,7 +166,6 @@ function create_header(data, table) {
- insert_cell(second_row, 'Sunspider test name', 'compare-table-header');
- insert_cell(second_row, 'Memory peak<br>(bytes)', 'compare-table-header left-border compare-table-header-unit');
- insert_cell(second_row, 'Max stack<br>(bytes)', 'compare-table-header left-border compare-table-header-unit');
- - insert_cell(second_row, 'Max RSS<br>(bytes)', 'compare-table-header left-border compare-table-header-unit');
- insert_cell(second_row, 'Runtime<br>(sec)', 'compare-table-header right-border compare-table-header-unit');
- measurements = data.compared_measurements;
- @@ -185,11 +174,9 @@ function create_header(data, table) {
- insert_cell(second_row, 'Memory peak<br>(bytes)', 'compare-table-header left-border compare-table-header-unit');
- insert_cell(second_row, 'Max stack<br>(bytes)', 'compare-table-header left-border compare-table-header-unit');
- - insert_cell(second_row, 'Max RSS<br>(bytes)', 'compare-table-header left-border compare-table-header-unit');
- insert_cell(second_row, 'Runtime<br>(sec)', 'compare-table-header compare-table-header-unit');
- insert_cell(second_row, 'Memory gain<br>(percentage)', 'compare-table-header compare-table-header-unit');
- insert_cell(second_row, 'Stack gain<br>(percentage)', 'compare-table-header compare-table-header-unit');
- - insert_cell(second_row, 'RSS gain<br>(percentage)', 'compare-table-header compare-table-header-unit');
- insert_cell(second_row, 'Runtime gain<br>(percentage)', 'compare-table-header right-border compare-table-header-unit');
- }
- }
- diff --git a/web_chart/js/utils.js b/web_chart/js/utils.js
- index 20b1172..e8556ab 100644
- --- a/web_chart/js/utils.js
- +++ b/web_chart/js/utils.js
- @@ -60,9 +60,6 @@ function current_test() {
- case 'aggregated stack':
- test = 'stack';
- break;
- - case 'aggregated rss':
- - test = 'rss';
- - break;
- case 'aggregated runtime':
- test = 'runtime';
- break;
- @@ -85,10 +82,6 @@ function test_is_stack(test) {
- return test == 'stack';
- }
- -function test_is_rss(test) {
- - return test == 'rss';
- -}
- -
- function test_is_runtime(test) {
- return test == 'runtime';
- }
- diff --git a/web_chart/js/web_chart.js b/web_chart/js/web_chart.js
- index c265f49..548a3e0 100644
- --- a/web_chart/js/web_chart.js
- +++ b/web_chart/js/web_chart.js
- @@ -56,7 +56,6 @@ String.prototype.lpad = function(padString, length) {
- function draw_chart(data, selected_test) {
- var memory_peak = [];
- var max_stack = [];
- - var max_rss = [];
- var runtime_avg = [];
- var ticks = [];
- var section_size = [];
- @@ -68,7 +67,6 @@ function draw_chart(data, selected_test) {
- if (result.test == selected_test) {
- memory_peak.push(result.memory_peak);
- max_stack.push(result.max_stack);
- - max_rss.push(result.max_rss);
- runtime_avg.push(result.avg_time);
- break;
- }
- @@ -135,8 +133,8 @@ function draw_chart(data, selected_test) {
- if (plot1)
- plot1.destroy();
- - if (memory_peak.length || max_stack.length || max_rss.length || runtime_avg.length)
- - plot1 = $.jqplot ('chart1', [memory_peak, max_stack, max_rss, runtime_avg], plot_options);
- + if (memory_peak.length || max_stack.length || runtime_avg.length)
- + plot1 = $.jqplot ('chart1', [memory_peak, max_stack, runtime_avg], plot_options);
- $('#chart1').unbind('jqplotDataClick');
- $('#chart1').bind('jqplotDataClick',
- @@ -167,10 +165,8 @@ function draw_chart_aggregated(data, type) {
- test_results[current_test.test].push(current_test.avg_time);
- else if (type == 'memory')
- test_results[current_test.test].push(current_test.memory_peak);
- - else if (type == 'stack')
- + else // type == 'stack'
- test_results[current_test.test].push(current_test.max_stack);
- - else // type == 'rss'
- - test_results[current_test.test].push(current_test.max_rss);
- }
- size = 0;
- for (j in measurement.sections) {
- @@ -325,11 +321,10 @@ function show_commit_info(ev, seriesIndex, pointIndex, value, data, aggregated)
- commit_message = 'Commit message is missing!';
- var measurement = data.results[pointIndex];
- - var sum = { memory: 0, stack: 0, rss: 0, runtime: 0 };
- + var sum = { memory: 0, stack: 0, runtime: 0 };
- $.each(measurement.test_results, function(key, test) {
- sum.memory += test.memory_peak;
- sum.stack += test.max_stack;
- - sum.rss += test.max_rss;
- sum.runtime += test.avg_time;
- });
- @@ -338,7 +333,6 @@ function show_commit_info(ev, seriesIndex, pointIndex, value, data, aggregated)
- $('#commit_date').html(nice_date(measurement.commit.timestamp));
- $('#commit_memory').html(sum.memory.toLocaleString() + ' bytes');
- $('#commit_stack').html(sum.stack.toLocaleString() + ' bytes');
- - $('#commit_rss').html(sum.rss.toLocaleString() + ' bytes');
- $('#commit_runtime').html(sum.runtime.toFixed(3) + ' sec');
- $('#commit_size').html(data.section_size[pointIndex].toLocaleString() + ' bytes');
- $('#commit_message').html(format_commit_message(commit_message));
- @@ -394,7 +388,7 @@ function async_draw_chart(data, test) {
- if (test_is_codesize(test)) {
- draw_size_chart(data);
- }
- - else if (test_is_runtime(test) || test_is_memory(test) || test_is_stack(test) || test_is_rss(test)) {
- + else if (test_is_runtime(test) || test_is_memory(test) || test_is_stack(test)) {
- draw_chart_aggregated(data, test);
- }
- else {
- @@ -513,7 +507,7 @@ function update_navbar() {
- update_selector('remote-selector', remote_list)
- bind_selector('remote-selector');
- - var test_selector_list = ['aggregated memory', 'aggregated stack', 'aggregated rss', 'aggregated runtime', 'code size'].concat(test_list);
- + var test_selector_list = ['aggregated memory', 'aggregated stack', 'aggregated runtime', 'code size'].concat(test_list);
- update_selector('test-selector', test_selector_list);
- bind_selector('test-selector');
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement