<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 31 May 2018 at 02:02, Joel Sherrill <span dir="ltr"><<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Wed, May 30, 2018 at 3:29 PM, Vijay Kumar Banerjee <span dir="ltr"><<a href="mailto:vijaykumar9597@gmail.com" target="_blank">vijaykumar9597@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On 31 May 2018 at 00:28, Joel Sherrill <span dir="ltr"><<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><div><div class="m_8226677743700300841m_-208057441006636333h5"><div><br></div></div></div><div>I may not understand correctly but there is test_run and coverage_run. Someone</div><div>suggested making coverage_running an option to test_run. If that's what's being</div><div>asked for, then I think doing it in a follow up patch is OK.</div><div><br></div><div>If that's the intended request, perhaps a ticket should be filed.</div><div><div class="m_8226677743700300841m_-208057441006636333h5"><div> </div></div></div></div></div></div></blockquote></span><div>Sorry for all the confusion.</div><div>This patch doesn't change the way test works. It only adds an option to run </div><div><span style="color:rgb(80,0,80)">the coverage script. coverage_run just runs the coverage.coverage_run</span></div></div></div></div></blockquote><div><br></div></span><div>:) And I am saying if we want to have one test_run with an argument, do it as</div><div>a future work iteration. File a ticket. </div><div><br>We need to get the code working on the master.</div><div><div class="h5"><div><br></div></div></div></div></div></div></blockquote><div>Okay, we can keep that as a future work (I haven't thought about it though). :)</div><div>Getting it to work on master is our primary objective. </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5"><div></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="m_8226677743700300841h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="m_8226677743700300841m_-208057441006636333h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="m_8226677743700300841m_-208057441006636333m_825675599414950786h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="m_8226677743700300841m_-208057441006636333m_825675599414950786m_7284862638170494635h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="m_8226677743700300841m_-208057441006636333m_825675599414950786m_7284862638170494635m_727216205359767578h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="m_8226677743700300841m_-208057441006636333m_825675599414950786m_7284862638170494635m_727216205359767578m_7921129128105952456m_6803257989446512162h5">
><br>
>      except error.general as gerr:<br>
>          print(gerr)<br>
> diff --git a/tester/rtems/testing/bsps/le<wbr>on3-qemu-cov.ini b/tester/rtems/testing/bsps/le<wbr>on3-qemu-cov.ini<br>
> index 6b5e7e6..2f89117 100644<br>
> --- a/tester/rtems/testing/bsps/le<wbr>on3-qemu-cov.ini<br>
> +++ b/tester/rtems/testing/bsps/le<wbr>on3-qemu-cov.ini<br>
> @@ -31,9 +31,10 @@<br>
>  #<br>
>  # The Leon 3 QEMU BSP<br>
>  #<br>
> -[leon3-qemu]<br>
> +[leon3-qemu-cov]<br>
>  bsp               = leon3-qemu<br>
>  arch              = sparc<br>
> +target            = sparc-rtems5<br>
>  tester            = %{_rtscripts}/qemu.cfg<br>
>  bsp_qemu_opts     = %{qemu_opts_base} -M leon3_generic<br>
>  bsp_qemu_cov_opts = -exec-trace %{test_executable}.cov<br>
> diff --git a/tester/rtems/testing/coverag<wbr>e/symbol-sets.ini b/tester/rtems/testing/coverag<wbr>e/symbol-sets.ini<br>
> new file mode 100644<br>
> index 0000000..a2ec7bc<br>
> --- /dev/null<br>
> +++ b/tester/rtems/testing/coverag<wbr>e/symbol-sets.ini<br>
> @@ -0,0 +1,36 @@<br>
> +#<br>
> +# RTEMS Tools Project (<a href="http://www.rtems.org/" rel="noreferrer" target="_blank">http://www.rtems.org/</a>)<br>
> +# Copyright 2018 Chris Johns (<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>)<br>
> +# All rights reserved.<br>
> +#<br>
> +# This file is part of the RTEMS Tools package in 'rtems-tools'.<br>
> +#<br>
> +# Redistribution and use in source and binary forms, with or without<br>
> +# modification, are permitted provided that the following conditions are met:<br>
> +#<br>
> +# 1. Redistributions of source code must retain the above copyright notice,<br>
> +# this list of conditions and the following disclaimer.<br>
> +#<br>
> +# 2. Redistributions in binary form must reproduce the above copyright notice,<br>
> +# this list of conditions and the following disclaimer in the documentation<br>
> +# and/or other materials provided with the distribution.<br>
> +#<br>
> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"<br>
> +# AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE<br>
> +# IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE<br>
> +# ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE<br>
> +# LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR<br>
> +# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF<br>
> +# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS<br>
> +# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN<br>
> +# CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)<br>
> +# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE<br>
> +# POSSIBILITY OF SUCH DAMAGE.<br>
> +#<br>
> +<br>
> +[symbol-sets]<br>
> +sets = score,rtems<br>
> +<br>
> +[libraries]<br>
> +score = @BUILD-TARGET@/c/@BSP@/cpukit/<wbr>score/libscore.a<br>
> +rtems = @BUILD-TARGET@/c/@BSP@/cpukit/<wbr>rtems/librtems.a<br>
> diff --git a/tester/rtems/testing/<a href="http://qemu.cf" target="_blank">qemu.cf</a><wbr>g b/tester/rtems/testing/<a href="http://qemu.cf" target="_blank">qemu.cf</a><wbr>g<br>
> index bfcd2f5..52a3752 100644<br>
> --- a/tester/rtems/testing/<a href="http://qemu.cf" target="_blank">qemu.cf</a><wbr>g<br>
> +++ b/tester/rtems/testing/<a href="http://qemu.cf" target="_blank">qemu.cf</a><wbr>g<br>
> @@ -51,8 +51,8 @@<br>
>  #<br>
>  # Qemu common option patterns.<br>
>  #<br>
> -#%define qemu_opts_base   -no-reboot -monitor none -serial stdio -nographic<br>
> -%define qemu_opts_base   -no-reboot -serial null -serial mon:stdio -nographic<br>
> +%define qemu_opts_base   -no-reboot -monitor none -serial stdio -nographic<br>
> +#%define qemu_opts_base   -no-reboot -serial null -serial mon:stdio -nographic<br>
<br>
</div></div>Why changing the common options for qemu?<br>
<span><br></span></blockquote></div></div><div>actually it's a bit experimental on advice of Cillian. </div><div>It stayed in the patch. </div></div></div></div></blockquote><div><br></div></div></div><div>I know the impact of some of those options but maybe it would make sense</div><div>to add a comment block with the impact of each option? It would help future</div><div>readers.</div><div><br></div><div>And, from personal experience, qemu changes arguments from time to time. Knowing</div><div>what the old intent was helps mapping to different versions and target architectures.</div><span><div><br></div></span></div></div></div></blockquote></div></div><div>That's a good Idea. </div><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
>  %define qemu_opts_no_net -net none<br>
><br>
>  #<br>
> --<br>
> 2.14.3<br>
><br>
</span>> ______________________________<wbr>_________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman<wbr>/listinfo/devel</a><br>
</blockquote></span></div><br></div></div>
<br>______________________________<wbr>_________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman<wbr>/listinfo/devel</a><br></blockquote></span></div><br></div></div>
</blockquote></span></div><br></div></div>
</blockquote></div></div><br></div></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div><br></div></div>