Opened 8 years ago

Closed 4 weeks ago

#6991 closed enhancement (fixed)

Update registrar to use start_watching_system() API

Reported by: anevilyak Owned by: bonefish
Priority: low Milestone: R1
Component: Servers/registrar Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

Currently the registrar does a polling sanity check in order to detect teams which have gone away without unregistering properly (i.e. in the case where a team is killed/crashes rather than quitting normally). This could be eliminated by instead using the new start_watching_system() call to monitor for team creation/destruction.

Change History (4)

comment:1 Changed 8 years ago by anevilyak

If no one beats me to it, I'll try to deal with this over the weekend sometime, just wanted to file the ticket so it's not forgotten.

comment:2 Changed 4 months ago by waddlesplash

That's a long weekend... :)

I might take a stab at this. Anything else that might have changed in the past 8 years that I'd need to know?

comment:3 Changed 6 weeks ago by tqh

I'll see if I can take a stab at this, I was checking what is using cpu at idle and 2% seems to be registrar timer_thread, and that seems to be this. Changing it from 1s to 5s takes that down to 0.1% cpu. Checked with top.

Some reference for how to do it (from waddlesplash): http://xref.plausible.coop/source/xref/haiku/headers/private/system/system_info.h#56 http://xref.plausible.coop/source/xref/haiku/src/servers/launch/LaunchDaemon.cpp#488

comment:4 Changed 4 weeks ago by waddlesplash

Resolution: fixed
Status: newclosed

Fix merged in hrev52796.

Note: See TracTickets for help on using tickets.