<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 15, 2020 at 11:40 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 16/10/20 3:30 pm, Sebastian Huber wrote:<br>
> On 16/10/2020 03:09, Chris Johns wrote:<br>
>> On 16/10/20 4:57 am, Sebastian Huber wrote:<br>
>>> I suggest to remove this function and replace it with rtems_interrupt_catch()<br>
>>> and add the interrupt enable to rtems_interrupt_catch().<br>
>> Which BSPs will this effect?<br>
> <br>
> All SPARC BSPs.<br>
<br>
OK.<br></blockquote><div><br></div><div>Unless you make that all BSPs which have set_vector, I am not in favor of this.</div><div><br></div><div>set_vector() was intended many many years ago as a central place for a BSP </div><div>to touch an interrupt mask register if it had one. This was sometimes present</div><div>on architectures which we now call simple vectored. This was universally</div><div>present across all simple vectored architectures.</div><div><br></div><div>If you want to eliminate it, do so. Don't leave it partially present.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
>> What testing on real hardware will there be?<br>
> <br>
> I will test on GR712RC and GR740. I hope Gaisler tests the RTEMS master from<br>
> time to time on their hardware.<br>
<br>
Yes. Testing as we are working on changes is much more helpful than finding out<br>
long after the focus has shifted.<br></blockquote><div><br></div><div>+1 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks<br>
Chris<br>
_______________________________________________<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/listinfo/devel</a><br>
</blockquote></div></div>