On Fri, Dec 27, 2013, at 09:25 AM, Michael Haberler wrote: > this is mystery code, commented out in halcmd, still present in rtapi_app, > and obviously dead since not used elsewhere > > it seems it was intended to create a new instance of a running component (?) > - any good use case to complete the code/bring to action or OK to remove? > > - Michael
It was intended to solve an annoyance. Currently, instances of realtime components are created when the kernel module is loaded, typically by a "count=" argument. If you have say three sum2 components in your configuration and decide you want to add another one, you need to start over. That dead code was the beginnings of an attempt to separate the act of loading a kernel module from the act of creating an instance of a HAL component. The goal was that "loadrt" would ONLY load the module. You would then invoke "newinst" as many times as needed to create the actual components. It would move most arguments from "loadrt" commands to "newinst" commands, and since each "newinst" could concentrate on configuring that single instance, the arguments would get simpler. No more comma delimited lists of arguments that affect multiple instances. That was to be part of a big HAL refactoring that I had in mind 5-7 years ago, before some stuff in my personal life caused me to cut way back on LinuxCNC coding. I still think it would be a good thing to have, but I know I won't be coding it. The community should decide whether the general concept is useful. I'm almost certain that specific partial implementation makes no sense; time has moved on. The time/place to do the refactor is probably in the unified build world, and many of the hoops that "newinst" needs to jump through are either gone or very different in that world. John Kasunich ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk _______________________________________________ Emc-developers mailing list Emc-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/emc-developers
