[lwip-devel] [patch #5785] Integrate SNMP initialization in tcpip.c

[Frédéric Bernon]

Follow-up Comment #2, patch #5785 (project lwip):

Ye,s agree with you, timeout timer are not a perfect feature, and, in this
case, I will prefer to be able to remove "snmp_inc_sysuptime()" and the
variable "sysuptime" and to be able to define my "own" function
"snmp_get_sysuptime" (based on platform timer). So, overhead will be reduce.

About lwip_init() function, in a lwip.c file? Perhaps it will be usefull to
add a define to tell is tcpip.c is used, to don't increase footprint if not
necessary (sockets.c, api_lib.c, api_msg.c, tcpip.c). With that, it will give
a pseudo-code like for tcpip.c users:

main()
{ //TODO: Target specific init

  lwip_init();

  //TODO: Add here network inferfaces with net_add

#if LWIP_SNMP
  snmp_set_sysdesr(...);
  snmp_set_syscontact(...);
  snmp_set_sysname(...);
  snmp_set_syslocation(...);
  //etc...
#endif 


  tcpip_init();

  //TODO: Application
}  

For RAW API users, only tcpip_init will have to be replace...


    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?5785>

_______________________________________________
  Message posté via/par Savannah
  http://savannah.nongnu.org/