Hi Marek
- How do we design the algorithm registration. At the moment batman_init()
calls bat_iv_init() and possibly other algorithms in the future but this is less than ideal. I had hoped to find a mechanism which allows an init function to be declared and called inside of the respective routing algorithm files. These files can be compiled into the module or not.
I would suggest looking at some other modular system. Generally the core offers a _register() and a _unregister() function. The module implementing an algorithm would call the _register function in its module_init function and the _unregister in module_exit function. It also makes sense to have a naming scheme for these modules. It then becomes possible to use hotplug to load algorithms dynamically, when userspace requests an algorithm which is not already loaded. I guess you first need an algorithm when the soft interface is taken up. If you don't have an algorithm loaded at that point, it makes sense to have a hard coded (or maybe menuconfig) default algorithm, which you try to load via hotplug.
- The API and its implementation. It is highly likely that this API won't
survive for very long as it has been designed to satisfy one routing algorithm only. However, changing it in the future should be easy.
It would be good to comment in the header file which functions can be NULL and which must be implemented. It would be best to assume that new methods added from now on will be NULL in older algorithms. There are a few options for handling this. Check the method is not NULL before calling it. Or the _register function can replace each NULL with a nop function and throw an BUG_ON() when a must have function is NULL.
Andrew