LX3JL / xlxd

HAM radio multiprotocol dstar reflector server
GNU General Public License v3.0
163 stars 111 forks source link

Improve shutdown behavior (enable clean shutdowns) #232

Open anonymouspage opened 1 year ago

anonymouspage commented 1 year ago

The two commits in this PR have their own commentary describing the changes that can be referred to, but to quickly summarize, this PR enables clean shutdowns of the reflector. The current code has no way of performing a clean shutdown for a couple reasons: (1) infinite loops in main() and (2) long blocking sleeps in various worker threads.

The general approach was to swap the blocking sleeps with condition variables that are signaled during shutdown, and to add signal handling in the main thread for termination signals. With these changes the reflector shuts down cleanly in less than a second with run as a daemon or in the foreground (when in the foreground, CTRL-C will initiate the shutdown).