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

[Simon Goldschmidt]

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

Hm, 64 bit is what I would favour, also. But 64-bit calculating in on a
16-bit processor is not what you can call fast, I think! Plus we need that
timestamp rather often...

But I would favour a portable solution (like a sys_time_t) which allows for
64bit or 32bit time (16bit is too short). I think Dmitrys patch can do that.

Regarding the SNMP sysUpTime value: if it is defined to be in 100th of
seconds (not 1000th), we should implement it that way. Of course, (also with
the current solution?) if you count the time in ms and the nconvert to 10ms,
you get an overflow after 49,7 days instead of 497 days. I don't think that's
a real problem. As you say, "not infinite"...

I would definitively not change SNMP time implementation until that timestamp
thing is solved. (sysuptime+=inc would work but is a suboptimal solution, I
think.)

This leads us to bugs #19167 and #1902:

Should we implement this / check it in to CVS HEAD or not???? It will break
current ports, but I think it's worth it!

Kieran?

    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/