<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 28, 2024 at 7:30 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@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 Fri, Jun 28, 2024 at 10:41 AM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
><br>
> On Thu, Jun 27, 2024 at 11:39 PM Sebastian Huber<br>
> <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a>> wrote:<br>
> ><br>
> > On 28.06.24 00:42, Gedare Bloom wrote:<br>
> > > I've been doing back through my third-party source code attribution.<br>
> > > I've found that there is a chunk of header files that are apparently<br>
> > > not included anywhere in rtems, and that are third-party headers. Is<br>
> > > there any reason we can't get rid of them?<br>
> ><br>
> > These files are included from Newlib header files and are used by the<br>
> > legacy network stack and libbsd.<br>
> ><br>
> Ok, thanks. The legacy network stack is maintenance-only. Any updates<br>
> to these files will result in divergence and breakage?<br>
><br>
> I think we need to split these files out of rtems.git. Likely the<br>
> current versions need to be copied into legacy network stack, and the<br>
> libbsd should provide its own versions?<br>
><br>
> -Gedare<br>
<br>
This got me to thinking there may be other header files that are<br>
potentially not used in rtems.git, and of questionable value to keep<br>
there:<br>
<br>
cpukit/include/rtems/vmeintr.h<br>
cpukit/include/rtems/spurious.h<br>
cpukit/include/rtems/score/smplockseq.h<br>
cpukit/include/rtems/score/smplockmcs.h<br>
cpukit/include/rtems/rtemsdialer.h<br>
cpukit/include/rtems/mptables.h<br>
<br>
Anyone have insight on whether these headers are unused, or are used<br>
in other dependent repos and should be considered for migration?<br></blockquote><div><br></div><div>mptables. is mentioned in a comment in spsize. The file mptables.h has</div><div>been empty since 2003. As best I can recall, it contained the declaration</div><div>for the distributed multiprocessing configuration.<br><br>spsize may itself be hopelessly out of date. It prints the size of various</div><div>data structures allocated by RTEMS during initialization and shows the</div><div>formula for computing memory usage. I don't remember anyone touching</div><div>the math in ages. Probably a candidate for removal as well.</div><div><br></div><div>spurious.h isn't included but the method it prototypes is defined by</div><div>one BSP.<br><br>$ grep -r Spurious_Initialize<br><br>cpukit/include/rtems/spurious.h:  { Spurious_Initialize, NULL, NULL, NULL, NULL, NULL }<br>cpukit/include/rtems/spurious.h:rtems_device_driver Spurious_Initialize(<br>bsps/m68k/mrm332/include/bsp.h:void Spurious_Initialize(void);<br>bsps/m68k/mrm332/start/bspstart.c:extern void Spurious_Initialize(void);<br>bsps/m68k/mrm332/start/bspstart.c:  Spurious_Initialize,<br>bsps/m68k/mrm332/start/spinit.c:void Spurious_Initialize(void)<br><br>Perhaps a rename in the BSP and remove the header file since bsp.h </div><div>prototypes it. </div><div><br></div><div>vmeintr.h is not included and uses the namespace VME_interrupt.</div><div>No implementation exists of anything in that namespace. Remove it.<br><br>That is three that can go. I think doing them one commit at a time </div><div>would be preferred to me at least.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
_______________________________________________<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></blockquote></div></div>