<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 15 June 2018 at 08:38, Chris Johns <span dir="ltr"><<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 14/06/2018 03:12, Gedare Bloom wrote:<br>
> On Wed, Jun 13, 2018 at 12:23 PM, Vijay Kumar Banerjee<br>
> <<a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@gmail.com</a>> wrote:<br>
>> On Wed, 13 Jun 2018, 21:39 Gedare Bloom, <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br>
>>> On Wed, Jun 13, 2018 at 11:58 AM, Vijay Kumar Banerjee<br>
>>> <<a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@gmail.com</a>> wrote:<br>
>>>> On 13 June 2018 at 10:29, Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br>
>>>>> On Thu, Jun 7, 2018 at 7:08 AM, Vijay Kumar Banerjee<br>
>>>>> <<a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@gmail.com</a>> wrote:<br>
</span><span class="">>>>>>>          bsp = opts.find_arg('--rtems-bsp')<br>
>>>>>> +        if 'cov' in bsp[1].split('-'):<br>
>>>>><br>
>>>>> I'm not sure if this use of the 'cov' field in the bsp config filename<br>
>>>>> itself is the proper way to go about accomplishing the activation of<br>
>>>>> coverage. What are other possible ways to get this done? Is the use of<br>
>>>>> a portion of the bsp config filename done elsewhere in tester?<br>
>>>><br>
>>>> This patch was made following Chris' comments in another thread<br>
>>>><br>
>>>> <a href="https://lists.rtems.org/pipermail/devel/2018-June/021931.html" rel="noreferrer" target="_blank">https://lists.rtems.org/<wbr>pipermail/devel/2018-June/<wbr>021931.html</a><br>
>>>><br>
>>><br>
>>> I can't be sure, but I don't think his intent was to infer the<br>
>>> coverage from the ini file name.<br>
<br>
</span>Correct.<br>
<span class=""><br>
>>> For example, does the tester parse<br>
>>> the ini file name to check for 'qemu' to decide if that target is<br>
>>> being used? Instead, it should look in to the config file to read the<br>
>>> option somehow.<br>
>><br>
>> In leon3-qemu.ini the bsp option inside the<br>
>> config file is set to leon3-qemu.<br>
>><br>
>> There's no such special thing added to bsp for coverage.<br>
>> Only difference we have is that,<br>
>> the option 'bsp_qemu_cov_opts' is added in the coverage supported file. we<br>
>> can<br>
>> read the config file to see if this option is present.<br>
>><br>
>> Shall I do it this way?<br>
> <br>
> Yes, I suspect you should.<br>
> <br>
<br>
</span>Can we have 'coverage = true' in the INI file to indicate this BSP supports<br>
coverage?<br>
<br>
>>><br>
>>>>><br>
>>>>>> +            coverage_enabled = True<br>
<br>
And that values maps to here.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>As the covoar patches are merged, shall I send a patch with the coverage=true option ? </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Chris<br>
</font></span></blockquote></div><br></div></div>