<div dir="ltr"><div dir="ltr"><div dir="ltr"><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div dir="ltr"><br></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 8 Feb 2019 at 08:34, Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 8/2/19 9:13 am, Vijay Kumar Banerjee wrote:<br>> On Fri, 8 Feb 2019 at 00:15, Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a><br>> <mailto:<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>>> wrote:<br>> <br>>     On 7/2/19 7:17 pm, Vijay Kumar Banerjee wrote:<br>>     > I'm trying to do this ...<br>>     ><br>>     > ===========<br>>     > diff --git a/tester/rtems/testing/bsps/leon3-sis-cov.ini<br>>     > b/tester/rtems/testing/bsps/leon3-sis-cov.ini<br>>     > index d60257c..e450b60 100644<br>>     > --- a/tester/rtems/testing/bsps/leon3-sis-cov.ini<br>>     > +++ b/tester/rtems/testing/bsps/leon3-sis-cov.ini<br>>     > @@ -38,4 +38,4 @@ tester         = %{_rtscripts}/run.cfg<br>>     >  target         = sparc-rtems5<br>>     >  bsp_run_cmd    = %{rtems_tools}/%{bsp_arch}-rtems%{rtems_version}-sis<br>>     >  bsp_run_opts   = -leon3 -nouartrx -r -tlim 200 s -cov<br>>     > -bsp_covoar_cmd = -E %{cov_explanations} -f TSIM<br>>     > +bsp_covoar_cmd = -S %{bsp_symbol_path} -p %{project_name} -E<br>>     > %{cov_explanations} -f TSIM<br>> <br>>     Can the common options be moved to coverage part of qemu.cfg?<br>> <br>> Since I'm trying with the sis. I added it to the run.cfg like this <br>> <br>> #<br>> # Coverage<br>> #<br>> %if %{defined coverage}<br>>   %define covoar_cmd -S %{bsp_symbol_path} -p RTEMS-%{rtems_version} -E<br>> %{cov_explanations}<br>> %endif<br>><br>
<br>Ahh, I wonder if this should be in a coverage.cfg file you include and this way<br>it can be shared?<br>
<br></blockquote><div>That's a good idea. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">  <br>> When I print the macros from the config.py I see this macro created like this...<br>> <br>>  coverage:             'none'     'none'      '1'<br>>  covoar_cmd:           'none'     'none'      '-S<br>> /home/lunatic/development/rtems/rtems-tools/tester/rtems/testing/coverage/leo \<br>>                                               n3-sis-symbols.ini -p RTEMS-5 -E<br>> /home/lunatic/development/rtems/rtems-tools/tes \<br>>                                              <br>> ter/rtems/testing/coverage/Explanations.txt'<br>> <br>> The only problem now is to expand it from coverage.py which is giving error.<br>> If I print the macros list from the coverage.py , I see that the list is missing<br>> this and some other macros. How to get the 'complete' list of macros? :/<br>
<br>Sorry I am not sute, can you see the config object from the coverage code?<br>
<br>I would need to review the code and figure it out and I do not have the time.<br>
<br></blockquote><div>I can attempt to figure it out with some help :)</div><div>Meanwhile, do we want to push this patch in the current working state, with the options</div><div>added to the BSP file? I will send a v3 of this patch with the changes.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Chris<br>
</blockquote></div></div></div>