<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 2, 2021 at 4:48 PM 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 3/3/21 3:54 am, Gedare Bloom wrote:<br>
> On Tue, Mar 2, 2021 at 8:52 AM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
>> On Tue, Mar 2, 2021 at 9:46 AM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
>>><br>
>>> On Mon, Mar 1, 2021 at 1:01 PM Alex White <<a href="mailto:alexanderjwhite@gmail.com" target="_blank">alexanderjwhite@gmail.com</a>> wrote:<br>
>>>><br>
>>>> The tester configurations had not been updated to match the paths and<br>
>>>> conventions used by the new build system. These have been updated,<br>
>>>> and a few more libraries have been enabled in symbol-sets.ini.<br>
>>>> ---<br>
>>>>  tester/rtems/testing/bsps/griscv-sis-cov.ini  |  4 +-<br>
>>>>  tester/rtems/testing/bsps/leon3-qemu-cov.ini  |  4 +-<br>
>>>>  tester/rtems/testing/bsps/leon3-sis-cov.ini   |  4 +-<br>
>>>>  tester/rtems/testing/coverage/symbol-sets.ini | 91 ++++++++++---------<br>
>>>>  4 files changed, 52 insertions(+), 51 deletions(-)<br>
>>>><br>
>>>> diff --git a/tester/rtems/testing/bsps/griscv-sis-cov.ini b/tester/rtems/testing/bsps/griscv-sis-cov.ini<br>
>>>> index 7249af6..e8de95f 100644<br>
>>>> --- a/tester/rtems/testing/bsps/griscv-sis-cov.ini<br>
>>>> +++ b/tester/rtems/testing/bsps/griscv-sis-cov.ini<br>
>>>> @@ -35,7 +35,7 @@<br>
>>>>  bsp            = griscv-sis<br>
>>>>  arch           = riscv<br>
>>>>  tester         = %{_rtscripts}/run.cfg<br>
>>>> -target         = riscv-rtems5<br>
>>>> +target         = riscv<br>
<br>
What is this being used for? I can only it present in the cov scripts.<br></blockquote><div><br></div><div>See below. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>>> These are the only ini files that define this target? With this<br>
>>> change, target is same as arch. Is target needed? I can't immediately<br>
>>> tell where it gets used later?  'target' is usually understood as the<br>
>>> binutil/gcc target triplet, so if the triplet is not being used, the<br>
>>> variable should be less misleading. covoar_target might be better if<br>
>>> this variable is needed in covar? I don't see it in any other<br>
>>> non-coverage ini files though, so maybe it is not needed at all?<br>
>><br>
>><br>
>> covoar originally used nm, addr2line, and objdump. This has been reduced to<br>
>> objdump but it needs the target name to be able to invoke that. At this point,<br>
>> the target has to be provided or covoar has to have rtems6 hard-coded and<br>
>> add rtems6 to the architecture.<br>
>><br>
>> Is there a variable in the tester to set this to?<br>
>><br>
>>><br>
>>>>  bsp_run_cmd    = %{rtems_tools}/%{bsp_arch}-rtems%{rtems_version}-sis<br>
>>>>  bsp_run_opts   = -nouartrx -r -tlim 300 s -m 4 -cov<br>
>>>> -bsp_covoar_cmd = -S %{bsp_symbol_path} -E %{cov_explanations} -f TSIM<br>
>>>> +bsp_covoar_cmd = -T riscv-rtems6 -S %{bsp_symbol_path} -E %{cov_explanations} -f TSIM<br>
>>><br>
>>> what about:<br>
>>> -T %{arch}-rtems%{rtems_version}<br>
>>><br>
>>> We should be limiting hard-coding version numbers.<br>
>><br>
>><br>
>> If that works, I would be happy.<br>
> <br>
> I t hink it might be %{bsp_arch} --- see bsp_run_cmd above ;)<br>
<br>
Do you need the option? There are other tools in rtems-tool that determine the<br>
exec-prefix. Could the toolkit's support in `rtemstoolkit/rld-cc.h` help?<br>
<br>
We knows the arch from the machine type in the ELF file and the toolkit should<br>
know the version. The nice thing about covoar is it keyed into the version.<br></blockquote><div><br></div><div>Can you get the GNU target? Knowing the architecture is only a bit of the problem?</div><div><br></div><div>I didn't spot anything in rtemstoolkit which  would give you CPU-rtems[567]...</div><div><br></div><div>If you have an example, great. </div><div><br></div><div>--joel</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Chris<br>
<br>
</blockquote></div></div>