<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 14, 2018, 1:07 AM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank" rel="noreferrer">chrisj@rtems.org</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 13/12/2018 20:30, Sebastian Huber wrote:<br>
> On 13/12/2018 09:16, Jiri Gaisler wrote:<br>
>>> I use the SIS every day for RTEMS development and test suite runs. The<br>
>>> SMP support would be a very valuable for RTEMS development. If we can<br>
>>> get also code coverage from it, then it would be perfect.<br>
>> I can look at the coverage support. <br>
<br>
This would be fantastic. The support for detailed analysis of trace data in<br>
RTEMS is growing. Covoar has had a lot of work done on it this year. An<br>
efficient cross-host way to get good trace data will improve the quality of RTEMS.<br>
<br>
I, like Sebastian, use the SIS for my development work<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Me too. And we still use it as the first bsp and simulator we recommend in the getting started. It is simple to use, fast and trustworthy.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>> Is there a standardized format for coverage data that I should use?<br>
> <br>
> I am not sure if there exists a standardized format. but Joel is the expert in<br>
> this area.<br>
<br>
The covoar tool currently support the couverture qemu trace format. I am not<br>
sure if it is something Adacore invented.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I think that format is something AdaCore invented. It's not bad but I wish they would push harder to get the trace support into upstream qemu.</div><div dir="auto"><br></div><div dir="auto">But couverture also supports the tsim bitmap format. It can support anything since that's a factory class and you can implement it yourself.</div><div dir="auto"><br></div><div dir="auto">But Jiri should know the tsim format so that would be my recommendation. He might remember adding a bit for branch take long ago at my request. :)</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The covoar tool is run from within the rtems-test tool when the BSP supports<br>
coverage. It has the ability to support a number of formats so we can add what<br>
is created. I suggest any format that is compact and can be parsed in a single<br>
pass should be fine.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">The tsim bitmap format is easy. If it produces that there will be no modifications.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The ability to control the trace file name would be nice. The rtems-test tool<br>
runs the sis simulator in parallel on the available cores and we now have a<br>
number of executables in the same directory.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">+1</div><div dir="auto"><br></div><div dir="auto">The riscv news and move of the simulator is great news.</div><div dir="auto"><br></div><div dir="auto">Moving grlib is also useful. We are working on an arm bsp.</div><div dir="auto"><br></div><div dir="auto">This is awesome!</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Chris<br>
</blockquote></div></div></div>