On 1 June 2018 at 01:57, Cillian O'Donnell <cpodonne...@gmail.com> wrote:
> So is it checking whether it's --coverage or --coverage=set1,set2? Are > those the 2 possibilities your checking? > > Yes, right. :) > On Thu, 31 May 2018, 20:52 Vijay Kumar Banerjee, <vijaykumar9...@gmail.com> > wrote: > >> On 1 June 2018 at 01:19, Gedare Bloom <ged...@rtems.org> wrote: >> >>> On Thu, May 31, 2018 at 3:47 PM, Vijay Kumar Banerjee >>> <vijaykumar9...@gmail.com> wrote: >>> > On 1 June 2018 at 01:07, Cillian O'Donnell <cpodonne...@gmail.com> >>> wrote: >>> >> >>> >> >>> >> >>> >> On 31 May 2018 at 19:07, Vijay Kumar Banerjee < >>> vijaykumar9...@gmail.com> >>> >> wrote: >>> >>> >>> >>> Add support in tester to run covoar and generate an html report to >>> >>> display >>> >>> the summary of the coverage reports generated from covoar. >>> >>> >>> >>> Co-authored-by : Cillian O'Donnell <cpodonne...@gmail.com> >>> >>> --- >>> >>> tester/rt/coverage.py | 379 >>> >>> ++++++++++++++++++++++++++ >>> >>> tester/rt/test.py | 36 ++- >>> >>> tester/rtems/testing/bsps/leon3-qemu-cov.ini | 3 +- >>> >>> tester/rtems/testing/coverage/symbol-sets.ini | 36 +++ >>> >>> tester/rtems/testing/qemu.cfg | 4 +- >>> >>> 5 files changed, 446 insertions(+), 12 deletions(-) >>> >>> create mode 100644 tester/rt/coverage.py >>> >>> create mode 100644 tester/rtems/testing/coverage/symbol-sets.ini >>> >>> >>> >>> diff --git a/tester/rt/coverage.py b/tester/rt/coverage.py >>> >>> new file mode 100644 >>> >>> index 0000000..25fbb9d >>> >>> --- /dev/null >>> >>> +++ b/tester/rt/coverage.py >>> >>> @@ -0,0 +1,379 @@ >>> >>> +# >>> >>> +# RTEMS Tools Project (http://www.rtems.org/) >>> >>> +# Copyright 2014 Krzysztof Miesowicz (krzysztof.miesow...@gmail.com >>> ) >>> >>> +# All rights reserved. >>> >>> +# >>> >>> +# This file is part of the RTEMS Tools package in 'rtems-tools'. >>> >>> +# >>> >>> +# Redistribution and use in source and binary forms, with or without >>> >>> +# modification, are permitted provided that the following >>> conditions are >>> >>> met: >>> >>> +# >>> >>> +# 1. Redistributions of source code must retain the above copyright >>> >>> notice, >>> >>> +# this list of conditions and the following disclaimer. >>> >>> +# >>> >>> +# 2. 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 with the distribution. >>> >>> +# >>> >>> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND >>> CONTRIBUTORS >>> >>> 'AS IS' >>> >>> +# AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED >>> TO, >>> >>> THE >>> >>> +# IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >>> >>> PURPOSE >>> >>> +# ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR >>> CONTRIBUTORS >>> >>> BE >>> >>> +# LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, >>> OR >>> >>> +# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT >>> OF >>> >>> +# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR >>> >>> BUSINESS >>> >>> +# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, >>> WHETHER >>> >>> IN >>> >>> +# CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR >>> >>> OTHERWISE) >>> >>> +# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF >>> ADVISED OF >>> >>> THE >>> >>> +# POSSIBILITY OF SUCH DAMAGE. >>> >>> +# >>> >>> + >>> >>> +from rtemstoolkit import error >>> >>> +from rtemstoolkit import path >>> >>> +from rtemstoolkit import log >>> >>> +from rtemstoolkit import execute >>> >>> +from rtemstoolkit import macros >>> >>> + >>> >>> +from datetime import datetime >>> >>> + >>> >>> +from . import options >>> >>> + >>> >>> +import shutil >>> >>> +import os >>> >>> + >>> >>> +try: >>> >>> + import configparser >>> >>> +except: >>> >>> + import ConfigParser as configparser >>> >>> + >>> >>> +class summary: >>> >>> + def __init__(self, p_summary_dir): >>> >>> + self.summary_file_path = path.join(p_summary_dir, >>> 'summary.txt') >>> >>> + self.index_file_path = path.join(p_summary_dir, >>> 'index.html') >>> >>> + self.bytes_analyzed = 0 >>> >>> + self.bytes_not_executed = 0 >>> >>> + self.percentage_executed = 0.0 >>> >>> + self.percentage_not_executed = 100.0 >>> >>> + self.ranges_uncovered = 0 >>> >>> + self.branches_uncovered = 0 >>> >>> + self.branches_total = 0 >>> >>> + self.branches_always_taken = 0 >>> >>> + self.branches_never_taken = 0 >>> >>> + self.percentage_branches_covered = 0.0 >>> >>> + self.is_failure = False >>> >>> + >>> >>> + def parse(self): >>> >>> + if(not path.exists(self.summary_file_path)): >>> >>> + log.notice('summary file %s does not exist!' % >>> >>> (self.summary_file_path)) >>> >>> + self.is_failure = True >>> >>> + return >>> >>> + >>> >>> + with open(self.summary_file_path,'r') as summary_file: >>> >>> + self.bytes_analyzed = self._get_next_with_colon( >>> summary_file) >>> >>> + self.bytes_not_executed = >>> >>> self._get_next_with_colon(summary_file) >>> >>> + self.percentage_executed = >>> >>> self._get_next_with_colon(summary_file) >>> >>> + self.percentage_not_executed = >>> >>> self._get_next_with_colon(summary_file) >>> >>> + self.ranges_uncovered = >>> >>> self._get_next_with_colon(summary_file) >>> >>> + self.branches_total = self._get_next_with_colon( >>> summary_file) >>> >>> + self.branches_uncovered = >>> >>> self._get_next_with_colon(summary_file) >>> >>> + self.branches_always_taken = >>> >>> self._get_next_without_colon(summary_file) >>> >>> + self.branches_never_taken = >>> >>> self._get_next_without_colon(summary_file) >>> >>> + if len(self.branches_uncovered) > 0 and >>> len(self.branches_total) >>> >>> > 0: >>> >>> + self.percentage_branches_covered = \ >>> >>> + 1 - (float(self.branches_uncovered) / >>> >>> float(self.branches_total)) >>> >>> + else: >>> >>> + self.percentage_branches_covered = 0.0 >>> >>> + return >>> >>> + >>> >>> + def _get_next_with_colon(self, summary_file): >>> >>> + line = summary_file.readline() >>> >>> + if ':' in line: >>> >>> + return line.split(':')[1].strip() >>> >>> + else: >>> >>> + return '' >>> >>> + >>> >>> + def _get_next_without_colon(self, summary_file): >>> >>> + line = summary_file.readline() >>> >>> + return line.strip().split(' ')[0] >>> >>> + >>> >>> +class report_gen_html: >>> >>> + def __init__(self, p_symbol_sets_list, build_dir, rtdir): >>> >>> + self.symbol_sets_list = ['score'] >>> >>> + self.build_dir = build_dir >>> >>> + self.partial_reports_files = list(["index.html", >>> "summary.txt"]) >>> >>> + self.number_of_columns = 1 >>> >>> + self.covoar_src_path = path.join(rtdir, 'covoar') >>> >>> + >>> >>> + def _find_partial_reports(self): >>> >>> + partial_reports = {} >>> >>> + for symbol_set in self.symbol_sets_list: >>> >>> + set_summary = summary(path.join(self.build_dir, >>> "coverage", >>> >>> + symbol_set)) >>> >>> + set_summary.parse() >>> >>> + partial_reports[symbol_set] = set_summary >>> >>> + return partial_reports >>> >>> + >>> >>> + def _prepare_head_section(self): >>> >>> + head_section = ''' >>> >>> + <head> >>> >>> + <title>RTEMS coverage report</title> >>> >>> + <style type="text/css"> >>> >>> + progress[value] { >>> >>> + -webkit-appearance: none; >>> >>> + appearance: none; >>> >>> + >>> >>> + width: 150px; >>> >>> + height: 15px; >>> >>> + } >>> >>> + </style> >>> >>> + </head>''' >>> >>> + return head_section >>> >>> + >>> >>> + def _prepare_index_content(self, partial_reports): >>> >>> + header = "<h1> RTEMS coverage analysis report </h1>" >>> >>> + header += "<h3>Coverage reports by symbols sets:</h3>" >>> >>> + table = "<table>" >>> >>> + table += self._header_row() >>> >>> + for symbol_set in partial_reports: >>> >>> + table += self._row(symbol_set, >>> partial_reports[symbol_set]) >>> >>> + table += "</table> </br>" >>> >>> + timestamp = "Analysis performed on " + >>> datetime.now().ctime() >>> >>> + return "<body>\n" + header + table + timestamp + "\n</body>" >>> >>> + >>> >>> + def _row(self, symbol_set, summary): >>> >>> + row = "<tr>" >>> >>> + row += "<td>" + symbol_set + "</td>" >>> >>> + if summary.is_failure: >>> >>> + row += ' <td colspan="' + str(self.number_of_columns-1) >>> \ >>> >>> + + '" style="background-color:red">FAILURE</td>' >>> >>> + else: >>> >>> + row += " <td>" + self._link(summary.index_file_ >>> path,"Index") >>> >>> \ >>> >>> + + "</td>" >>> >>> + row += " <td>" + >>> >>> self._link(summary.summary_file_path,"Summary") \ >>> >>> + + "</td>" >>> >>> + row += " <td>" + summary.bytes_analyzed + "</td>" >>> >>> + row += " <td>" + summary.bytes_not_executed + "</td>" >>> >>> + row += " <td>" + summary.ranges_uncovered + "</td>" >>> >>> + row += " <td>" + summary.percentage_executed + "%</td>" >>> >>> + row += " <td>" + summary.percentage_not_executed + >>> "%</td>" >>> >>> + row += ' <td><progress value="' + >>> >>> summary.percentage_executed \ >>> >>> + + '" max="100"></progress></td>' >>> >>> + row += " <td>" + summary.branches_uncovered + "</td>" >>> >>> + row += " <td>" + summary.branches_total + "</td>" >>> >>> + row += " <td> {:.3%} >>> >>> </td>".format(summary.percentage_branches_covered) >>> >>> + row += ' <td><progress value="{:.3}" >>> >>> max="100"></progress></td>'.format(100*summary.percentage_ >>> branches_covered) >>> >>> + row += "</tr>\n" >>> >>> + return row >>> >>> + >>> >>> + def _header_row(self): >>> >>> + row = "<tr>" >>> >>> + row += "<th> Symbols set name </th>" >>> >>> + row += "<th> Index file </th>" >>> >>> + row += "<th> Summary file </th>" >>> >>> + row += "<th> Bytes analyzed </th>" >>> >>> + row += "<th> Bytes not executed </th>" >>> >>> + row += "<th> Uncovered ranges </th>" >>> >>> + row += "<th> Percentage covered </th>" >>> >>> + row += "<th> Percentage uncovered </th>" >>> >>> + row += "<th> Instruction coverage </th>" >>> >>> + row += "<th> Branches uncovered </th>" >>> >>> + row += "<th> Branches total </th>" >>> >>> + row += "<th> Branches covered percentage </th>" >>> >>> + row += "<th> Branches coverage </th>" >>> >>> + row += "</tr>\n" >>> >>> + self.number_of_columns = row.count('<th>') >>> >>> + return row >>> >>> + >>> >>> + def _link(self, address, text): >>> >>> + return '<a href="' + address + '">' + text + '</a>' >>> >>> + >>> >>> + def _create_index_file(self, head_section, content): >>> >>> + with open(path.join(self.build_dir,"report.html"),'w') as >>> f: >>> >>> + f.write(head_section) >>> >>> + f.write(content) >>> >>> + >>> >>> + def generate(self): >>> >>> + partial_reports = self._find_partial_reports() >>> >>> + head_section = self._prepare_head_section() >>> >>> + index_content = self._prepare_index_content( >>> partial_reports) >>> >>> + self._create_index_file(head_section,index_content) >>> >>> + >>> >>> + def add_covoar_src_path(self): >>> >>> + table_js_path = path.join(self.covoar_src_path, 'table.js') >>> >>> + covoar_css_path = path.join(self.covoar_src_path, >>> 'covoar.css') >>> >>> + for symbol_set in self.symbol_sets_list: >>> >>> + symbol_set_dir = path.join(self.build_dir, "coverage", >>> >>> symbol_set) >>> >>> + html_files = os.listdir(symbol_set_dir) >>> >>> + for html_file in html_files: >>> >>> + html_file = path.join(symbol_set_dir, html_file) >>> >>> + if path.exists(html_file) and 'html' in html_file: >>> >>> + with open(html_file, 'r') as f: >>> >>> + file_data = f.read() >>> >>> + file_data = file_data.replace('table.js', >>> >>> table_js_path) >>> >>> + file_data = file_data.replace('covoar.css', >>> >>> + covoar_css_path) >>> >>> + with open(html_file, 'w') as f: >>> >>> + f.write(file_data) >>> >>> + >>> >>> +class build_path_generator(object): >>> >>> + ''' >>> >>> + Generates the build path from the path to executables >>> >>> + ''' >>> >>> + def __init__(self, executables, target): >>> >>> + self.executables = executables >>> >>> + self.target = target >>> >>> + def run(self): >>> >>> + build_path = '/' >>> >>> + Path = self.executables[0].split('/') >>> >>> + for P in Path: >>> >>> + if P == self.target: >>> >>> + break; >>> >>> + else: >>> >>> + build_path = path.join(build_path, P) >>> >>> + return build_path >>> >>> + >>> >>> +class symbol_parser(object): >>> >>> + ''' >>> >>> + Parse the symbol sets ini and create custom ini file for covoar >>> >>> + ''' >>> >>> + def __init__(self, symbol_config_path, >>> >>> + symbol_select_path, coverage_arg, build_dir): >>> >>> + self.symbol_select_file = symbol_select_path >>> >>> + self.symbol_file = symbol_config_path >>> >>> + self.build_dir = build_dir >>> >>> + self.symbol_sets = {} >>> >>> + self.cov_arg = coverage_arg >>> >>> + self.ssets = [] >>> >>> + >>> >>> + def parse(self): >>> >>> + config = configparser.ConfigParser() >>> >>> + try: >>> >>> + config.read(self.symbol_file) >>> >>> + if self.cov_arg: >>> >>> + self.ssets = self.cov_arg.split(',') >>> >>> + else: >>> >>> + self.ssets = config.get('symbol-sets', >>> >>> 'sets').split(',') >>> >>> + self.ssets = [ sset.encode('utf-8') for sset in >>> >>> self.ssets] >>> >>> + for sset in self.ssets: >>> >>> + lib = path.join(self.build_dir, >>> >>> + config.get('libraries', sset)) >>> >>> + self.symbol_sets[sset] = lib.encode('utf-8') >>> >>> + except: >>> >>> + raise error.general('Symbol set parsing failed') >>> >>> + >>> >>> + def _write_ini(self): >>> >>> + config = configparser.ConfigParser() >>> >>> + try: >>> >>> + sets = ', '.join(self.symbol_sets.keys()) >>> >>> + config.add_section('symbol-sets') >>> >>> + config.set('symbol-sets', 'sets', sets) >>> >>> + for key in self.symbol_sets.keys(): >>> >>> + config.add_section(key) >>> >>> + config.set(key, 'libraries', self.symbol_sets[key]) >>> >>> + with open(self.symbol_select_file, 'w') as conf: >>> >>> + config.write(conf) >>> >>> + except: >>> >>> + raise error.general('write failed') >>> >>> + >>> >>> + def run(self): >>> >>> + self.parse() >>> >>> + self._write_ini() >>> >>> + >>> >>> +class covoar(object): >>> >>> + ''' >>> >>> + Covoar runner >>> >>> + ''' >>> >>> + def __init__(self, base_result_dir, config_dir, executables, >>> >>> explanations_txt): >>> >>> + self.base_result_dir = base_result_dir >>> >>> + self.config_dir = config_dir >>> >>> + self.executables = ' '.join(executables) >>> >>> + self.explanations_txt = explanations_txt >>> >>> + self.project_name = 'RTEMS-5' >>> >>> + >>> >>> + def run(self, set_name, symbol_file): >>> >>> + covoar_result_dir = path.join(self.base_result_dir, >>> set_name) >>> >>> + if (not path.exists(covoar_result_dir)): >>> >>> + path.mkdir(covoar_result_dir) >>> >>> + if (not path.exists(symbol_file)): >>> >>> + raise error.general('symbol set file: coverage %s was >>> not >>> >>> created for covoar, skipping %s'% (symbol_file, set_name)) >>> >>> + command = ('covoar -S ' + symbol_file >>> >>> + + ' -O ' + covoar_result_dir >>> >>> + + ' -E ' + self.explanations_txt >>> >>> + + ' -p ' + self.project_name + ' ' + >>> self.executables) >>> >>> + log.notice('Running covoar for %s' % (set_name)) >>> >>> + print( 'covoar results directory:\n' + covoar_result_dir ) >>> >>> + executor = execute.execute(verbose = True, output = >>> >>> self.output_handler) >>> >>> + exit_code = executor.shell(command, cwd=os.getcwd()) >>> >>> + if (exit_code[0] != 0): >>> >>> + raise error.general('covoar failure exit code: %d' % >>> >>> (exit_code[0])) >>> >>> + log.notice('Coverage run for %s finished successfully.' % >>> >>> (set_name)) >>> >>> + log.notice('------------------ >>> -----------------------------') >>> >>> + >>> >>> + def output_handler(self, text): >>> >>> + log.notice('%s' % (text)) >>> >>> + >>> >>> +class coverage_run(object): >>> >>> + ''' >>> >>> + Coverage analysis support for rtems-test >>> >>> + ''' >>> >>> + def __init__(self, p_macros, coverage_arg, target): >>> >>> + ''' >>> >>> + Constructor >>> >>> + ''' >>> >>> + self.macros = p_macros >>> >>> + self.build_dir = self.macros['_cwd'] >>> >>> + self.explanations_txt = >>> >>> self.macros.expand(self.macros['cov_explanations']) >>> >>> + self.test_dir = path.join(self.build_dir, 'coverage') >>> >>> + if (not path.exists(self.test_dir)): >>> >>> + path.mkdir(self.test_dir) >>> >>> + self.rtdir = path.abspath(self.macros['_rtdir']) >>> >>> + self.rtscripts = self.macros.expand(self. >>> macros['_rtscripts']) >>> >>> + self.coverage_config_path = path.join(self.rtscripts, >>> >>> 'coverage') >>> >>> + self.symbol_config_path = path.join(self.coverage_ >>> config_path, >>> >>> + 'symbol-sets.ini') >>> >>> + self.symbol_select_path = path.join(self.coverage_ >>> config_path, >>> >>> + 'symbol-select.ini') >>> >>> + self.executables = None >>> >>> + self.symbol_sets = [] >>> >>> + self.no_clean = int(self.macros['_no_clean']) >>> >>> + self.report_format = self.macros['cov_report_format'] >>> >>> + self.coverage_arg = coverage_arg >>> >>> + self.target = target >>> >>> + >>> >>> + def run(self): >>> >>> + try: >>> >>> + if self.executables is None: >>> >>> + raise error.general('no test executables provided.') >>> >>> + build_dir = build_path_generator(self.executables, >>> >>> self.target).run() >>> >>> + parser = symbol_parser(self.symbol_config_path, >>> >>> + self.symbol_select_path, >>> >>> + self.coverage_arg, >>> >>> + build_dir) >>> >>> + parser.run() >>> >>> + covoar_runner = covoar(self.test_dir, >>> >>> self.symbol_select_path, >>> >>> + self.executables, >>> >>> self.explanations_txt) >>> >>> + covoar_runner.run('score', self.symbol_select_path) >>> >>> + self._generate_reports(); >>> >>> + self._summarize(); >>> >>> + finally: >>> >>> + self._cleanup(); >>> >>> + >>> >>> + def _generate_reports(self): >>> >>> + log.notice('Generating reports') >>> >>> + if self.report_format == 'html': >>> >>> + report = report_gen_html(self.symbol_sets, >>> >>> + self.build_dir, >>> >>> + self.rtdir) >>> >>> + report.generate() >>> >>> + report.add_covoar_src_path() >>> >>> + >>> >>> + def _cleanup(self): >>> >>> + if not self.no_clean: >>> >>> + log.notice('***Cleaning tempfiles***') >>> >>> + for exe in self.executables: >>> >>> + trace_file = exe + '.cov' >>> >>> + if path.exists(trace_file): >>> >>> + os.remove(trace_file) >>> >>> + >>> >>> + def _summarize(self): >>> >>> + log.notice('Coverage analysis finished. You can find >>> results in >>> >>> %s' % (self.build_dir)) >>> >>> diff --git a/tester/rt/test.py b/tester/rt/test.py >>> >>> index f4d9b5c..84dff01 100644 >>> >>> --- a/tester/rt/test.py >>> >>> +++ b/tester/rt/test.py >>> >>> @@ -48,12 +48,14 @@ from rtemstoolkit import mailer >>> >>> from rtemstoolkit import reraise >>> >>> from rtemstoolkit import stacktraces >>> >>> from rtemstoolkit import version >>> >>> +from rtemstoolkit import check >>> >>> >>> >>> from . import bsps >>> >>> from . import config >>> >>> from . import console >>> >>> from . import options >>> >>> from . import report >>> >>> +from . import coverage >>> >>> >>> >>> class log_capture(object): >>> >>> def __init__(self): >>> >>> @@ -147,7 +149,7 @@ class test_run(object): >>> >>> >>> >>> def run(self): >>> >>> self.thread = threading.Thread(target = self.runner, >>> >>> - name = 'test[%s]' % >>> >>> path.basename(self.executable)) >>> >>> + name = 'test[%s]' % >>> >>> path.basename(self.executable)) >>> >>> self.thread.start() >>> >>> >>> >>> def is_alive(self): >>> >>> @@ -214,6 +216,10 @@ def killall(tests): >>> >>> for test in tests: >>> >>> test.kill() >>> >>> >>> >>> +def coverage_run(opts, coverage, executables): >>> >>> + coverage.executables = executables >>> >>> + coverage.run() >>> >>> + >>> >>> def run(command_path = None): >>> >>> import sys >>> >>> tests = [] >>> >>> @@ -221,15 +227,16 @@ def run(command_path = None): >>> >>> opts = None >>> >>> default_exefilter = '*.exe' >>> >>> try: >>> >>> - optargs = { '--rtems-tools': 'The path to the RTEMS tools', >>> >>> - '--rtems-bsp': 'The RTEMS BSP to run the test >>> on', >>> >>> - '--user-config': 'Path to your local user >>> >>> configuration INI file', >>> >>> - '--report-mode': 'Reporting modes, failures >>> >>> (default),all,none', >>> >>> - '--list-bsps': 'List the supported BSPs', >>> >>> - '--debug-trace': 'Debug trace based on specific >>> >>> flags', >>> >>> - '--filter': 'Glob that executables must >>> match >>> >>> to run (default: ' + >>> >>> + optargs = { '--rtems-tools': 'The path to the RTEMS >>> tools', >>> >>> + '--rtems-bsp': 'The RTEMS BSP to run the >>> test >>> >>> on', >>> >>> + '--user-config': 'Path to your local user >>> >>> configuration INI file', >>> >>> + '--report-mode': 'Reporting modes, failures >>> >>> (default),all,none', >>> >>> + '--list-bsps': 'List the supported BSPs', >>> >>> + '--debug-trace': 'Debug trace based on >>> specific >>> >>> flags', >>> >>> + '--filter': 'Glob that executables must >>> >>> match to run (default: ' + >>> >>> default_exefilter + ')', >>> >>> - '--stacktrace': 'Dump a stack trace on a user >>> >>> termination (^C)' } >>> >>> + '--stacktrace': 'Dump a stack trace on a >>> user >>> >>> termination (^C)', >>> >>> + '--coverage': 'Perform coverage analysis >>> of >>> >>> test executables.'} >>> >>> mailer.append_options(optargs) >>> >>> opts = options.load(sys.argv, >>> >>> optargs = optargs, >>> >>> @@ -279,6 +286,15 @@ def run(command_path = None): >>> >>> raise error.general('RTEMS BSP not provided or an >>> invalid >>> >>> option') >>> >>> bsp = config.load(bsp[1], opts) >>> >>> bsp_config = opts.defaults.expand(opts.defaults['tester']) >>> >>> + coverage_enabled = opts.find_arg('--coverage') >>> >>> + if coverage_enabled: >>> >>> + if len(coverage_enabled) == 2: >>> >> >>> >> >>> >> Is this to check if sets have been provided to --coverage=score,sapi, >>> that >>> >> kind of thing? If so the variable name coverage_enabled doesn't make >>> sense >>> >> any more. Probably if symbol_sets and if len(symbol_sets) will make >>> more >>> >> sense for both cases. >>> > >>> > That's a good point. >>> > How about changing it to just 'coverage' instead ? >>> > Because it's still checking whether the option is given or >>> > not. symbol_sets might create confusion. >>> > Will that be OK ? >>> >>> That name might collide with your class object (coverage)? >>> >> That's actually why I kept it as coverage_enabled. >> I think we can go for coverage_option (?) >> >>> >>> >> >>> >> >>> >>> + coverage_runner = coverage.coverage_run(opts. >>> defaults, >>> >>> + coverage_enabled[1], >>> >>> + >>> opts.defaults['target']) >>> >>> + else: >>> >>> + coverage_runner = coverage.coverage_run(opts. >>> defaults, >>> >>> 0, >>> >>> + >>> >>> opts.defaults['target']) >>> >>> report_mode = opts.find_arg('--report-mode') >>> >>> if report_mode: >>> >>> if report_mode[1] != 'failures' and \ >>> >>> @@ -365,6 +381,8 @@ def run(command_path = None): >>> >>> reports.failures(), >>> >>> 'Log', '===', ''] + output.get() >>> >>> mail.send(to_addr, subject, os.linesep.join(body)) >>> >>> + if coverage_enabled: >>> >>> + coverage_run(opts, coverage_runner, executables) >>> >>> >>> >>> except error.general as gerr: >>> >>> print(gerr) >>> >>> diff --git a/tester/rtems/testing/bsps/leon3-qemu-cov.ini >>> >>> b/tester/rtems/testing/bsps/leon3-qemu-cov.ini >>> >>> index 6b5e7e6..2f89117 100644 >>> >>> --- a/tester/rtems/testing/bsps/leon3-qemu-cov.ini >>> >>> +++ b/tester/rtems/testing/bsps/leon3-qemu-cov.ini >>> >>> @@ -31,9 +31,10 @@ >>> >>> # >>> >>> # The Leon 3 QEMU BSP >>> >>> # >>> >>> -[leon3-qemu] >>> >>> +[leon3-qemu-cov] >>> >>> bsp = leon3-qemu >>> >>> arch = sparc >>> >>> +target = sparc-rtems5 >>> >>> tester = %{_rtscripts}/qemu.cfg >>> >>> bsp_qemu_opts = %{qemu_opts_base} -M leon3_generic >>> >>> bsp_qemu_cov_opts = -exec-trace %{test_executable}.cov >>> >>> diff --git a/tester/rtems/testing/coverage/symbol-sets.ini >>> >>> b/tester/rtems/testing/coverage/symbol-sets.ini >>> >>> new file mode 100644 >>> >>> index 0000000..a2ec7bc >>> >>> --- /dev/null >>> >>> +++ b/tester/rtems/testing/coverage/symbol-sets.ini >>> >>> @@ -0,0 +1,36 @@ >>> >>> +# >>> >>> +# RTEMS Tools Project (http://www.rtems.org/) >>> >>> +# Copyright 2018 Chris Johns (chr...@rtems.org) >>> >>> +# All rights reserved. >>> >>> +# >>> >>> +# This file is part of the RTEMS Tools package in 'rtems-tools'. >>> >>> +# >>> >>> +# Redistribution and use in source and binary forms, with or without >>> >>> +# modification, are permitted provided that the following >>> conditions are >>> >>> met: >>> >>> +# >>> >>> +# 1. Redistributions of source code must retain the above copyright >>> >>> notice, >>> >>> +# this list of conditions and the following disclaimer. >>> >>> +# >>> >>> +# 2. 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 with the distribution. >>> >>> +# >>> >>> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND >>> CONTRIBUTORS >>> >>> "AS IS" >>> >>> +# AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED >>> TO, >>> >>> THE >>> >>> +# IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR >>> >>> PURPOSE >>> >>> +# ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR >>> CONTRIBUTORS >>> >>> BE >>> >>> +# LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, >>> OR >>> >>> +# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT >>> OF >>> >>> +# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR >>> >>> BUSINESS >>> >>> +# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, >>> WHETHER >>> >>> IN >>> >>> +# CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR >>> >>> OTHERWISE) >>> >>> +# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF >>> ADVISED OF >>> >>> THE >>> >>> +# POSSIBILITY OF SUCH DAMAGE. >>> >>> +# >>> >>> + >>> >>> +[symbol-sets] >>> >>> +sets = score,rtems >>> >>> + >>> >>> +[libraries] >>> >>> +score = @BUILD-TARGET@/c/@BSP@/cpukit/score/libscore.a >>> >>> +rtems = @BUILD-TARGET@/c/@BSP@/cpukit/rtems/librtems.a >>> >>> diff --git a/tester/rtems/testing/qemu.cfg >>> >>> b/tester/rtems/testing/qemu.cfg >>> >>> index bfcd2f5..52a3752 100644 >>> >>> --- a/tester/rtems/testing/qemu.cfg >>> >>> +++ b/tester/rtems/testing/qemu.cfg >>> >>> @@ -51,8 +51,8 @@ >>> >>> # >>> >>> # Qemu common option patterns. >>> >>> # >>> >>> -#%define qemu_opts_base -no-reboot -monitor none -serial stdio >>> >>> -nographic >>> >>> -%define qemu_opts_base -no-reboot -serial null -serial mon:stdio >>> >>> -nographic >>> >>> +%define qemu_opts_base -no-reboot -monitor none -serial stdio >>> >>> -nographic >>> >>> +#%define qemu_opts_base -no-reboot -serial null -serial mon:stdio >>> >>> -nographic >>> >>> %define qemu_opts_no_net -net none >>> >>> >>> >>> # >>> >>> -- >>> >>> 2.14.3 >>> >>> >>> >>> _______________________________________________ >>> >>> devel mailing list >>> >>> devel@rtems.org >>> >>> http://lists.rtems.org/mailman/listinfo/devel >>> >> >>> >> >>> > >>> > >>> > _______________________________________________ >>> > devel mailing list >>> > devel@rtems.org >>> > http://lists.rtems.org/mailman/listinfo/devel >>> >> >>
_______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel