]> git.sesse.net Git - cubemap/blobdiff - main.cpp
Revert "Rewrite the entire internal signal handling/wakeup."
[cubemap] / main.cpp
index 0564e27e8a2a865e1ea9e0f62cf7bc622c27c5d4..46eddaf834ce7aacfa3c3e518d70618747731246 100644 (file)
--- a/main.cpp
+++ b/main.cpp
+#include <assert.h>
+#include <errno.h>
+#include <getopt.h>
+#include <limits.h>
+#include <signal.h>
+#include <stddef.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
-#include <stdint.h>
-#include <assert.h>
-#include <arpa/inet.h>
-#include <sys/socket.h>
-#include <pthread.h>
-#include <sys/types.h>
-#include <sys/ioctl.h>
-#include <sys/poll.h>
 #include <sys/time.h>
-#include <signal.h>
-#include <errno.h>
-#include <ctype.h>
-#include <fcntl.h>
-#include <vector>
-#include <string>
+#include <sys/wait.h>
+#include <unistd.h>
 #include <map>
 #include <set>
+#include <string>
+#include <utility>
+#include <vector>
 
-#include "metacube.h"
-#include "parse.h"
-#include "server.h"
-#include "serverpool.h"
+#include "acceptor.h"
+#include "accesslog.h"
+#include "config.h"
 #include "input.h"
+#include "log.h"
+#include "markpool.h"
+#include "serverpool.h"
 #include "state.pb.h"
-
-#define STREAM_ID "stream"
-#define STREAM_URL "http://gruessi.zrh.sesse.net:4013/"
+#include "stats.h"
+#include "stream.h"
+#include "util.h"
+#include "version.h"
 
 using namespace std;
 
+AccessLogThread *access_log = NULL;
 ServerPool *servers = NULL;
+vector<MarkPool *> mark_pools;
 volatile bool hupped = false;
+volatile bool stopped = false;
 
-void hup(int ignored)
+struct InputWithRefcount {
+       Input *input;
+       int refcount;
+};
+
+void hup(int signum)
 {
        hupped = true;
+       if (signum == SIGINT) {
+               stopped = true;
+       }
 }
 
-int create_server_socket(int port)
+CubemapStateProto collect_state(const timeval &serialize_start,
+                                const vector<Acceptor *> acceptors,
+                                const multimap<string, InputWithRefcount> inputs,
+                                ServerPool *servers)
 {
-       int server_sock = socket(PF_INET6, SOCK_STREAM, IPPROTO_TCP);
-       if (server_sock == -1) {
-               perror("socket");
-               exit(1);
-       }
-
-       int one = 1;
-       if (setsockopt(server_sock, SOL_SOCKET, SO_REUSEADDR, &one, sizeof(one)) == -1) {
-               perror("setsockopt(SO_REUSEADDR)");
-               exit(1);
-       }
-
-       // We want dual-stack sockets. (Sorry, OpenBSD and Windows XP...)
-       int zero = 0;
-       if (setsockopt(server_sock, IPPROTO_IPV6, IPV6_V6ONLY, &zero, sizeof(zero)) == -1) {
-               perror("setsockopt(IPV6_V6ONLY)");
-               exit(1);
+       CubemapStateProto state = servers->serialize();  // Fills streams() and clients().
+       state.set_serialize_start_sec(serialize_start.tv_sec);
+       state.set_serialize_start_usec(serialize_start.tv_usec);
+       
+       for (size_t i = 0; i < acceptors.size(); ++i) {
+               state.add_acceptors()->MergeFrom(acceptors[i]->serialize());
        }
 
-       // Set as non-blocking, so the acceptor thread can notice that we want to shut it down.
-       if (ioctl(server_sock, FIONBIO, &one) == -1) {
-               perror("ioctl(FIONBIO)");
-               exit(1);
+       for (multimap<string, InputWithRefcount>::const_iterator input_it = inputs.begin();
+            input_it != inputs.end();
+            ++input_it) {
+               state.add_inputs()->MergeFrom(input_it->second.input->serialize());
        }
 
-       sockaddr_in6 addr;
-       memset(&addr, 0, sizeof(addr));
-       addr.sin6_family = AF_INET6;
-       addr.sin6_port = htons(port);
+       return state;
+}
 
-       if (bind(server_sock, reinterpret_cast<sockaddr *>(&addr), sizeof(addr)) == -1) {
-               perror("bind");
-               exit(1);
+// Find all port statements in the configuration file, and create acceptors for htem.
+vector<Acceptor *> create_acceptors(
+       const Config &config,
+       map<int, Acceptor *> *deserialized_acceptors)
+{
+       vector<Acceptor *> acceptors;
+       for (unsigned i = 0; i < config.acceptors.size(); ++i) {
+               const AcceptorConfig &acceptor_config = config.acceptors[i];
+               Acceptor *acceptor = NULL;
+               map<int, Acceptor *>::iterator deserialized_acceptor_it =
+                       deserialized_acceptors->find(acceptor_config.port);
+               if (deserialized_acceptor_it != deserialized_acceptors->end()) {
+                       acceptor = deserialized_acceptor_it->second;
+                       deserialized_acceptors->erase(deserialized_acceptor_it);
+               } else {
+                       int server_sock = create_server_socket(acceptor_config.port, TCP_SOCKET);
+                       acceptor = new Acceptor(server_sock, acceptor_config.port);
+               }
+               acceptor->run();
+               acceptors.push_back(acceptor);
        }
 
-       if (listen(server_sock, 128) == -1) {
-               perror("listen");
-               exit(1);
+       // Close all acceptors that are no longer in the configuration file.
+       for (map<int, Acceptor *>::iterator acceptor_it = deserialized_acceptors->begin();
+            acceptor_it != deserialized_acceptors->end();
+            ++acceptor_it) {
+               acceptor_it->second->close_socket();
+               delete acceptor_it->second;
        }
 
-       return server_sock;
+       return acceptors;
 }
 
-void *acceptor_thread_run(void *arg)
+// Find all streams in the configuration file, and create inputs for them.
+void create_config_inputs(const Config &config, multimap<string, InputWithRefcount> *inputs)
 {
-       int server_sock = int(intptr_t(arg));
-       while (!hupped) {
-               // Since we are non-blocking, we need to wait for the right state first.
-               // Wait up to 50 ms, then check hupped.
-               pollfd pfd;
-               pfd.fd = server_sock;
-               pfd.events = POLLIN;
-
-               int nfds = poll(&pfd, 1, 50);
-               if (nfds == 0 || (nfds == -1 && errno == EINTR)) {
-                       continue;
-               }
-               if (nfds == -1) {
-                       perror("poll");
-                       usleep(100000);
+       for (unsigned i = 0; i < config.streams.size(); ++i) {
+               const StreamConfig &stream_config = config.streams[i];
+               if (stream_config.src.empty()) {
                        continue;
                }
 
-               sockaddr_in6 addr;
-               socklen_t addrlen = sizeof(addr);
-
-               // Get a new socket.
-               int sock = accept(server_sock, reinterpret_cast<sockaddr *>(&addr), &addrlen);
-               if (sock == -1 && errno == EINTR) {
-                       continue;
-               }
-               if (sock == -1) {
-                       perror("accept");
-                       usleep(100000);
+               string src = stream_config.src;
+               if (inputs->count(src) != 0) {
                        continue;
                }
 
-               // Set the socket as nonblocking.
-               int one = 1;
-               if (ioctl(sock, FIONBIO, &one) == -1) {
-                       perror("FIONBIO");
-                       exit(1);
+               InputWithRefcount iwr;
+               iwr.input = create_input(src);
+               if (iwr.input == NULL) {
+                       log(ERROR, "did not understand URL '%s', clients will not get any data.",
+                               src.c_str());
+                       continue;
                }
-
-               // Pick a server, round-robin, and hand over the socket to it.
-               servers->add_client(sock);
+               iwr.refcount = 0;
+               inputs->insert(make_pair(src, iwr));
        }
-       return NULL;
 }
 
-struct StatsThreadParameters {
-       string stats_file;
-       int stats_interval;
-};
-               
-void *stats_thread_run(void *arg)
+void create_streams(const Config &config,
+                    const set<string> &deserialized_stream_ids,
+                    multimap<string, InputWithRefcount> *inputs)
 {
-       const StatsThreadParameters *parms = reinterpret_cast<StatsThreadParameters *>(arg);
-       while (!hupped) {
-               int fd;
-               FILE *fp;
-               time_t now;
-               vector<ClientStats> client_stats;
-
-               // Open a new, temporary file.
-               char *filename = strdup((parms->stats_file + ".new.XXXXXX").c_str());
-               fd = mkostemp(filename, O_WRONLY);
-               if (fd == -1) {
-                       perror(filename);
-                       free(filename);
-                       goto sleep;
-               }
+       for (unsigned i = 0; i < config.mark_pools.size(); ++i) {
+               const MarkPoolConfig &mp_config = config.mark_pools[i];
+               mark_pools.push_back(new MarkPool(mp_config.from, mp_config.to));
+       }
 
-               fp = fdopen(fd, "w");
-               if (fp == NULL) {
-                       perror("fdopen");
-                       close(fd);
-                       unlink(filename);
-                       free(filename);
-                       goto sleep;
+       set<string> expecting_stream_ids = deserialized_stream_ids;
+       for (unsigned i = 0; i < config.streams.size(); ++i) {
+               const StreamConfig &stream_config = config.streams[i];
+               if (deserialized_stream_ids.count(stream_config.stream_id) == 0) {
+                       servers->add_stream(stream_config.stream_id,
+                                           stream_config.backlog_size,
+                                           Stream::Encoding(stream_config.encoding));
+               } else {
+                       servers->set_backlog_size(stream_config.stream_id, stream_config.backlog_size);
+                       servers->set_encoding(stream_config.stream_id,
+                                             Stream::Encoding(stream_config.encoding));
                }
+               expecting_stream_ids.erase(stream_config.stream_id);
 
-               now = time(NULL);
-               client_stats = servers->get_client_stats();
-               for (size_t i = 0; i < client_stats.size(); ++i) {
-                       fprintf(fp, "%s %s %d %llu\n",
-                               client_stats[i].remote_addr.c_str(),
-                               client_stats[i].stream_id.c_str(),
-                               int(now - client_stats[i].connect_time),
-                               (long long unsigned)(client_stats[i].bytes_sent));
-               }
-               if (fclose(fp) == EOF) {
-                       perror("fclose");
-                       unlink(filename);
-                       free(filename);
-                       goto sleep;
+               if (stream_config.mark_pool != -1) {
+                       servers->set_mark_pool(stream_config.stream_id,
+                                              mark_pools[stream_config.mark_pool]);
                }
-               
-               if (rename(filename, parms->stats_file.c_str()) == -1) {
-                       perror("rename");
-                       unlink(filename);
+
+               string src = stream_config.src;
+               if (!src.empty()) {
+                       multimap<string, InputWithRefcount>::iterator input_it = inputs->find(src);
+                       assert(input_it != inputs->end());
+                       input_it->second.input->add_destination(stream_config.stream_id);
+                       ++input_it->second.refcount;
                }
+       }
 
-sleep:
-               int left_to_sleep = parms->stats_interval;
-               do {
-                       left_to_sleep = sleep(left_to_sleep);
-               } while (left_to_sleep > 0 && !hupped);
+       // Warn about any servers we've lost.
+       // TODO: Make an option (delete=yes?) to actually shut down streams.
+       for (set<string>::const_iterator stream_it = expecting_stream_ids.begin();
+            stream_it != expecting_stream_ids.end();
+            ++stream_it) {
+               string stream_id = *stream_it;
+               log(WARNING, "stream '%s' disappeared from the configuration file. "
+                            "It will not be deleted, but clients will not get any new inputs.",
+                            stream_id.c_str());
        }
-       return NULL;
 }
-
-// Serialize the given state to a file descriptor, and return the (still open)
-// descriptor.
-int make_tempfile(const CubemapStateProto &state)
+       
+void open_logs(const vector<LogConfig> &log_destinations)
 {
-       char tmpl[] = "/tmp/cubemapstate.XXXXXX";
-       int state_fd = mkstemp(tmpl);
-       if (state_fd == -1) {
-               perror("mkstemp");
-               exit(1);
+       for (size_t i = 0; i < log_destinations.size(); ++i) {
+               if (log_destinations[i].type == LogConfig::LOG_TYPE_FILE) {
+                       add_log_destination_file(log_destinations[i].filename);
+               } else if (log_destinations[i].type == LogConfig::LOG_TYPE_CONSOLE) {
+                       add_log_destination_console();
+               } else if (log_destinations[i].type == LogConfig::LOG_TYPE_SYSLOG) {
+                       add_log_destination_syslog();
+               } else {
+                       assert(false);
+               }
        }
+       start_logging();
+}
+       
+bool dry_run_config(const std::string &argv0, const std::string &config_filename)
+{
+       char *argv0_copy = strdup(argv0.c_str());
+       char *config_filename_copy = strdup(config_filename.c_str());
+
+       pid_t pid = fork();
+       switch (pid) {
+       case -1:
+               log_perror("fork()");
+               free(argv0_copy);
+               free(config_filename_copy);
+               return false;
+       case 0:
+               // Child.
+               execlp(argv0_copy, argv0_copy, "--test-config", config_filename_copy, NULL);
+               log_perror(argv0_copy);
+               _exit(1);
+       default:
+               // Parent.
+               break;
+       }
+               
+       free(argv0_copy);
+       free(config_filename_copy);
 
-       string serialized;
-       state.SerializeToString(&serialized);
-
-       const char *ptr = serialized.data();
-       size_t to_write = serialized.size();
-       while (to_write > 0) {
-               ssize_t ret = write(state_fd, ptr, to_write);
-               if (ret == -1) {
-                       perror("write");
-                       exit(1);
-               }
+       int status;
+       pid_t err;
+       do {
+               err = waitpid(pid, &status, 0);
+       } while (err == -1 && errno == EINTR);
 
-               ptr += ret;
-               to_write -= ret;
-       }
+       if (err == -1) {
+               log_perror("waitpid()");
+               return false;
+       }       
 
-       return state_fd;
+       return (WIFEXITED(status) && WEXITSTATUS(status) == 0);
 }
 
-// Read the state back from the file descriptor made by make_tempfile,
-// and close it.
-CubemapStateProto read_tempfile(int state_fd)
+int main(int argc, char **argv)
 {
-       if (lseek(state_fd, 0, SEEK_SET) == -1) {
-               perror("lseek");
-               exit(1);
-       }
-
-       string serialized;
-       char buf[4096];
+       signal(SIGHUP, hup);
+       signal(SIGINT, hup);
+       signal(SIGPIPE, SIG_IGN);
+       
+       // Parse options.
+       int state_fd = -1;
+       bool test_config = false;
        for ( ;; ) {
-               ssize_t ret = read(state_fd, buf, sizeof(buf));
-               if (ret == -1) {
-                       perror("read");
-                       exit(1);
+               static const option long_options[] = {
+                       { "state", required_argument, 0, 's' },
+                       { "test-config", no_argument, 0, 't' },
+                       { 0, 0, 0, 0 }
+               };
+               int option_index = 0;
+               int c = getopt_long(argc, argv, "s:t", long_options, &option_index);
+     
+               if (c == -1) {
+                       break;
                }
-               if (ret == 0) {
-                       // EOF.
+               switch (c) {
+               case 's':
+                       state_fd = atoi(optarg);
                        break;
+               case 't':
+                       test_config = true;
+                       break;
+               default:
+                       fprintf(stderr, "Unknown option '%s'\n", argv[option_index]);
+                       exit(1);
                }
+       }
 
-               serialized.append(string(buf, buf + ret));
+       string config_filename = "cubemap.config";
+       if (optind < argc) {
+               config_filename = argv[optind++];
        }
 
-       close(state_fd);  // Implicitly deletes the file.
+       // Canonicalize argv[0] and config_filename.
+       char argv0_canon[PATH_MAX];
+       char config_filename_canon[PATH_MAX];
 
-       CubemapStateProto state;
-       if (!state.ParseFromString(serialized)) {
-               fprintf(stderr, "PANIC: Failed deserialization of state.\n");
+       if (realpath(argv[0], argv0_canon) == NULL) {
+               log_perror(argv[0]);
+               exit(1);
+       }
+       if (realpath(config_filename.c_str(), config_filename_canon) == NULL) {
+               log_perror(config_filename.c_str());
                exit(1);
        }
 
-       return state;
-}
-
-int main(int argc, char **argv)
-{
-       fprintf(stderr, "\nCubemap starting.\n");
-
-       struct timeval serialize_start;
-       bool is_reexec = false;
+       // Now parse the configuration file.
+       Config config;
+       if (!parse_config(config_filename_canon, &config)) {
+               exit(1);
+       }
+       if (test_config) {
+               exit(0);
+       }
+       
+       // Ideally we'd like to daemonize only when we've started up all threads etc.,
+       // but daemon() forks, which is not good in multithreaded software, so we'll
+       // have to do it here.
+       if (config.daemonize) {
+               if (daemon(0, 0) == -1) {
+                       log_perror("daemon");
+                       exit(1);
+               }
+       }
 
-       string config_filename = (argc == 1) ? "cubemap.config" : argv[1];
-       vector<ConfigLine> config = parse_config(config_filename);
+start:
+       // Open logs as soon as possible.
+       open_logs(config.log_destinations);
 
-       int port = fetch_config_int(config, "port", 1, 65535, PARAMATER_MANDATORY);
-       int num_servers = fetch_config_int(config, "num_servers", 1, 20000, PARAMATER_MANDATORY);  // Insanely high max limit.
+       log(INFO, "Cubemap " SERVER_VERSION " starting.");
+       if (config.access_log_file.empty()) {
+               // Create a dummy logger.
+               access_log = new AccessLogThread();
+       } else {
+               access_log = new AccessLogThread(config.access_log_file);
+       }
+       access_log->run();
 
-       servers = new ServerPool(num_servers);
+       servers = new ServerPool(config.num_servers);
 
-       int server_sock = -1, old_port = -1;
+       CubemapStateProto loaded_state;
+       struct timeval serialize_start;
        set<string> deserialized_stream_ids;
-       map<string, Input *> deserialized_inputs;
-       if (argc == 4 && strcmp(argv[2], "-state") == 0) {
-               is_reexec = true;
-
-               fprintf(stderr, "Deserializing state from previous process... ");
-               int state_fd = atoi(argv[3]);
-               CubemapStateProto loaded_state = read_tempfile(state_fd);
+       map<int, Acceptor *> deserialized_acceptors;
+       multimap<string, InputWithRefcount> inputs;  // multimap due to older versions without deduplication.
+       if (state_fd != -1) {
+               log(INFO, "Deserializing state from previous process...");
+               string serialized;
+               if (!read_tempfile(state_fd, &serialized)) {
+                       exit(1);
+               }
+               if (!loaded_state.ParseFromString(serialized)) {
+                       log(ERROR, "Failed deserialization of state.");
+                       exit(1);
+               }
 
                serialize_start.tv_sec = loaded_state.serialize_start_sec();
                serialize_start.tv_usec = loaded_state.serialize_start_usec();
 
                // Deserialize the streams.
                for (int i = 0; i < loaded_state.streams_size(); ++i) {
-                       servers->add_stream_from_serialized(loaded_state.streams(i));
-                       deserialized_stream_ids.insert(loaded_state.streams(i).stream_id());
-               }
+                       const StreamProto &stream = loaded_state.streams(i);
 
-               // Put back the existing clients. It doesn't matter which server we
-               // allocate them to, so just do round-robin.
-               for (int i = 0; i < loaded_state.clients_size(); ++i) {
-                       servers->add_client_from_serialized(loaded_state.clients(i));
+                       vector<int> data_fds;
+                       for (int j = 0; j < stream.data_fds_size(); ++j) {
+                               data_fds.push_back(stream.data_fds(j));
+                       }
+
+                       // Older versions stored the data once in the protobuf instead of
+                       // sending around file descriptors.
+                       if (data_fds.empty() && stream.has_data()) {
+                               data_fds.push_back(make_tempfile(stream.data()));
+                       }
+
+                       servers->add_stream_from_serialized(stream, data_fds);
+                       deserialized_stream_ids.insert(stream.stream_id());
                }
 
-               // Deserialize the inputs. Note that we don't actually add them to any state yet.
+               // Deserialize the inputs. Note that we don't actually add them to any stream yet.
                for (int i = 0; i < loaded_state.inputs_size(); ++i) {
-                       deserialized_inputs.insert(make_pair(
-                               loaded_state.inputs(i).stream_id(),
-                               new Input(loaded_state.inputs(i))));
+                       InputWithRefcount iwr;
+                       iwr.input = create_input(loaded_state.inputs(i));
+                       iwr.refcount = 0;
+                       inputs.insert(make_pair(loaded_state.inputs(i).url(), iwr));
                } 
 
-               // Deserialize the server socket.
-               server_sock = loaded_state.server_sock();
-               old_port = loaded_state.port();
-
-               fprintf(stderr, "done.\n");
-       }
-
-       // Find all streams in the configuration file, and create them.
-       set<string> expecting_stream_ids = deserialized_stream_ids;
-       for (unsigned i = 0; i < config.size(); ++i) {
-               if (config[i].keyword != "stream") {
-                       continue;
-               }
-               if (config[i].arguments.size() != 1) {
-                       fprintf(stderr, "ERROR: 'stream' takes exactly one argument\n");
-                       exit(1);
-               }
-               string stream_id = config[i].arguments[0];
-               if (deserialized_stream_ids.count(stream_id) == 0) {
-                       servers->add_stream(stream_id);
+               // Deserialize the acceptors.
+               for (int i = 0; i < loaded_state.acceptors_size(); ++i) {
+                       deserialized_acceptors.insert(make_pair(
+                               loaded_state.acceptors(i).port(),
+                               new Acceptor(loaded_state.acceptors(i))));
                }
-               expecting_stream_ids.erase(stream_id);
-       }
 
-       // Warn about any servers we've lost.
-       // TODO: Make an option (delete=yes?) to actually shut down streams.
-       for (set<string>::const_iterator stream_it = expecting_stream_ids.begin();
-            stream_it != expecting_stream_ids.end();
-            ++stream_it) {
-               string stream_id = *stream_it;
-               fprintf(stderr, "WARNING: stream '%s' disappeared from the configuration file.\n",
-                       stream_id.c_str());
-               fprintf(stderr, "         It will not be deleted, but clients will not get any new inputs.\n");
-               if (deserialized_inputs.count(stream_id) != 0) {
-                       delete deserialized_inputs[stream_id];
-                       deserialized_inputs.erase(stream_id);
-               }
+               log(INFO, "Deserialization done.");
        }
 
-       // Open a new server socket if we do not already have one, or if we changed ports.
-       if (server_sock != -1 && port != old_port) {
-               fprintf(stderr, "NOTE: Port changed from %d to %d; opening new socket.\n", old_port, port);
-               close(server_sock);
-               server_sock = -1;
-       }
-       if (server_sock == -1) {
-               server_sock = create_server_socket(port);
-       }
-
-       // See if the user wants stats.
-       string stats_file = fetch_config_string(config, "stats_file", PARAMETER_OPTIONAL);
-       int stats_interval = fetch_config_int(config, "stats_interval", 1, INT_MAX, PARAMETER_OPTIONAL, -1);
-       if (stats_interval != -1 && stats_file.empty()) {
-               fprintf(stderr, "WARNING: 'stats_interval' given, but no 'stats_file'. No statistics will be written.\n");
+       // Add any new inputs coming from the config.
+       create_config_inputs(config, &inputs);
+       
+       // Find all streams in the configuration file, create them, and connect to the inputs.
+       create_streams(config, deserialized_stream_ids, &inputs);
+       vector<Acceptor *> acceptors = create_acceptors(config, &deserialized_acceptors);
+       
+       // Put back the existing clients. It doesn't matter which server we
+       // allocate them to, so just do round-robin. However, we need to add
+       // them after the mark pools have been set up.
+       for (int i = 0; i < loaded_state.clients_size(); ++i) {
+               servers->add_client_from_serialized(loaded_state.clients(i));
        }
-
+       
        servers->run();
 
-       pthread_t acceptor_thread;
-       pthread_create(&acceptor_thread, NULL, acceptor_thread_run, reinterpret_cast<void *>(server_sock));
-
-       // Find all streams in the configuration file, and create inputs for them.
-       vector<Input *> inputs;
-       for (unsigned i = 0; i < config.size(); ++i) {
-               if (config[i].keyword != "stream") {
-                       continue;
-               }
-               assert(config[i].arguments.size() == 1);
-               string stream_id = config[i].arguments[0];
-
-               if (config[i].parameters.count("src") == 0) {
-                       fprintf(stderr, "WARNING: stream '%s' has no src= attribute, clients will not get any data.\n",
-                               stream_id.c_str());
-                       continue;
-               }
-
-               string src = config[i].parameters["src"];
-               Input *input = NULL;
-               if (deserialized_inputs.count(stream_id) != 0) {
-                       input = deserialized_inputs[stream_id];
-                       if (input->get_url() != src) {
-                               fprintf(stderr, "INFO: Stream '%s' has changed URL from '%s' to '%s', restarting input.\n",
-                                       stream_id.c_str(), input->get_url().c_str(), src.c_str());
-                               delete input;
-                               input = NULL;
-                       }
-                       deserialized_inputs.erase(stream_id);
+       // Now delete all inputs that are longer in use, and start the others.
+       for (multimap<string, InputWithRefcount>::iterator input_it = inputs.begin();
+            input_it != inputs.end(); ) {
+               if (input_it->second.refcount == 0) {
+                       log(WARNING, "Input '%s' no longer in use, closing.",
+                           input_it->first.c_str());
+                       input_it->second.input->close_socket();
+                       delete input_it->second.input;
+                       inputs.erase(input_it++);
+               } else {
+                       input_it->second.input->run();
+                       ++input_it;
                }
-               if (input == NULL) {
-                       input = new Input(stream_id, src);
-               }
-               input->run();
-               inputs.push_back(input);
        }
 
-       // All deserialized inputs should now have been taken care of, one way or the other.
-       assert(deserialized_inputs.empty());
-
        // Start writing statistics.
-       pthread_t stats_thread;
-       StatsThreadParameters stats_parameters;  // Must live for as long as the stats thread does.
-       if (!stats_file.empty()) {
-               stats_parameters.stats_file = stats_file;
-               stats_parameters.stats_interval = stats_interval;
-               pthread_create(&stats_thread, NULL, stats_thread_run, &stats_parameters);
+       StatsThread *stats_thread = NULL;
+       if (!config.stats_file.empty()) {
+               stats_thread = new StatsThread(config.stats_file, config.stats_interval);
+               stats_thread->run();
        }
 
-       signal(SIGHUP, hup);
-       
        struct timeval server_start;
        gettimeofday(&server_start, NULL);
-       if (is_reexec) {
+       if (state_fd != -1) {
                // Measure time from we started deserializing (below) to now, when basically everything
                // is up and running. This is, in other words, a conservative estimate of how long our
                // “glitch” period was, not counting of course reconnects if the configuration changed.
                double glitch_time = server_start.tv_sec - serialize_start.tv_sec +
                        1e-6 * (server_start.tv_usec - serialize_start.tv_usec);
-               fprintf(stderr, "Re-exec happened in approx. %.0f ms.\n", glitch_time * 1000.0);
+               log(INFO, "Re-exec happened in approx. %.0f ms.", glitch_time * 1000.0);
        }
 
        while (!hupped) {
@@ -432,55 +433,67 @@ int main(int argc, char **argv)
        // OK, we've been HUPed. Time to shut down everything, serialize, and re-exec.
        gettimeofday(&serialize_start, NULL);
 
-       if (!stats_file.empty()) {
-               pthread_kill(stats_thread, SIGHUP);
-               if (pthread_join(stats_thread, NULL) == -1) {
-                       perror("pthread_join");
-                       exit(1);
-               }
+       if (stats_thread != NULL) {
+               stats_thread->stop();
+               delete stats_thread;
        }
-       pthread_kill(acceptor_thread, SIGHUP);
-       if (pthread_join(acceptor_thread, NULL) == -1) {
-               perror("pthread_join");
-               exit(1);
+       for (size_t i = 0; i < acceptors.size(); ++i) {
+               acceptors[i]->stop();
+       }
+       for (multimap<string, InputWithRefcount>::iterator input_it = inputs.begin();
+            input_it != inputs.end();
+            ++input_it) {
+               input_it->second.input->stop();
        }
+       servers->stop();
 
        CubemapStateProto state;
-       state.set_serialize_start_sec(serialize_start.tv_sec);
-       state.set_serialize_start_usec(serialize_start.tv_usec);
-       state.set_server_sock(server_sock);
-       state.set_port(port);
-
-       for (size_t i = 0; i < inputs.size(); ++i) {
-               inputs[i]->stop();
-               state.add_inputs()->MergeFrom(inputs[i]->serialize());
+       if (stopped) {
+               log(INFO, "Shutting down.");
+       } else {
+               log(INFO, "Serializing state and re-execing...");
+               state = collect_state(
+                       serialize_start, acceptors, inputs, servers);
+               string serialized;
+               state.SerializeToString(&serialized);
+               state_fd = make_tempfile(serialized);
+               if (state_fd == -1) {
+                       exit(1);
+               }
        }
+       delete servers;
 
-       for (int i = 0; i < num_servers; ++i) { 
-               servers->get_server(i)->stop();
+       for (unsigned i = 0; i < mark_pools.size(); ++i) {
+               delete mark_pools[i];
+       }
+       mark_pools.clear();
 
-               CubemapStateProto local_state = servers->get_server(i)->serialize();
+       access_log->stop();
+       delete access_log;
+       shut_down_logging();
 
-               // The stream state should be identical between the servers, so we only store it once.
-               if (i == 0) {
-                       state.mutable_streams()->MergeFrom(local_state.streams());
-               }
-               for (int j = 0; j < local_state.clients_size(); ++j) {
-                       state.add_clients()->MergeFrom(local_state.clients(j));
-               }
+       if (stopped) {
+               exit(0);
        }
-       delete servers;
-
-       fprintf(stderr, "Serializing state and re-execing...\n");
-       int state_fd = make_tempfile(state);
 
+       // OK, so the signal was SIGHUP. Check that the new config is okay, then exec the new binary.
+       if (!dry_run_config(argv0_canon, config_filename_canon)) {
+               open_logs(config.log_destinations);
+               log(ERROR, "%s --test-config failed. Restarting old version instead of new.", argv[0]);
+               hupped = false;
+               shut_down_logging();
+               goto start;
+       }
+        
        char buf[16];
        sprintf(buf, "%d", state_fd);
 
        for ( ;; ) {
-               execlp(argv[0], argv[0], config_filename.c_str(), "-state", buf, NULL);
-               perror("execlp");
-               fprintf(stderr, "PANIC: re-exec of %s failed. Waiting 0.2 seconds and trying again...\n", argv[0]);
+               execlp(argv0_canon, argv0_canon, config_filename_canon, "--state", buf, NULL);
+               open_logs(config.log_destinations);
+               log_perror("execlp");
+               log(ERROR, "re-exec of %s failed. Waiting 0.2 seconds and trying again...", argv0_canon);
+               shut_down_logging();
                usleep(200000);
        }
 }