search

OLSR / OONF

OLSR.org Network Framework - olsrd v2 / DLEP
55 stars 35 forks source link

systeminfo-time showing strangly high values #20

Closed pocki80 closed 2 years ago

pocki80 commented 5 years ago

using telnet plugin "systeminfo time" I see that the uptime second counter increases in an unexpected fast was. As there war "seconds since deamon start" in former versions, I looks like this right now:

time_system     time_internal
17:15:58.331     2.628k (a few seconds after daemon start)
17:16:11.643     7.065k (13.312sec later, increased by 4.437k that is 0.3333k per second)
17:16:21.668    10.407k
17:16:52.832    20.795k
17:19:47.271    78.941k
pocki80 commented 5 years ago

It was fine in v0.15.0-9-g2f559fb and maybe later. Is was not ok in v0.15.1-75-g312d09d or maybe earlier. So some commit between these two screwed it up.

HRogge commented 5 years ago

Hi,

we had a fix about this in v0.15.1-100-ga1a4c7e4, can you test the most recent "develop" branch?

Henning

On Mon, Nov 12, 2018 at 5:43 PM pocki80 notifications@github.com wrote:

It was fine in v0.15.0-9-g2f559fb and maybe later. Is was not ok in v0.15.1-75-g312d09d or maybe earlier. So some commit between these two screwed it up.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/OLSR/OONF/issues/20#issuecomment-437943756, or mute the thread https://github.com/notifications/unsubscribe-auth/AG8Ytx1lVcEPlrPuXpVL4Z95qKgk6gyuks5uuaF2gaJpZM4YZ_Cx .

pocki80 commented 5 years ago

Actually no, I was not able to test it until now. But looking at the code changes it seems to be fixed. Though, waiting for all these recent commits to get merged to master. BR

HRogge commented 5 years ago

Sorry,

I have been tracking a problem in the main netlink code for weeks... I don't want to merge before this is done because it tends to break the routing agent in my special test setup... I am lucky I found a way to trigger it reliable.

On Fri, Nov 30, 2018 at 9:45 AM pocki80 notifications@github.com wrote:

Actually no, I was not able to test it until now. But looking at the code changes it seems to be fixed. Though, waiting for all these recent commits to get merged to master. BR

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/OLSR/OONF/issues/20#issuecomment-443132953, or mute the thread https://github.com/notifications/unsubscribe-auth/AG8Yt_MtbFOQPEZkWBHLMke2hnAPJzojks5u0PAogaJpZM4YZ_Cx .