X-Git-Url: https://git.sesse.net/?p=cubemap;a=blobdiff_plain;f=main.cpp;h=c4c5bd4187d3a74850a721fb2d8b90f77835405d;hp=c5640e0c6ecd6c70fadad8c748a1907dfbf565fe;hb=74cd48ffef90d7d0752e37a4515e4ecfb68f7c9d;hpb=2c6cd9718a5baf5a2ed5be73ec3c525d4873f45a

diff --git a/main.cpp b/main.cpp
index c5640e0..c4c5bd4 100644
--- a/main.cpp
+++ b/main.cpp
@@ -1,82 +1,66 @@
+#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 "acceptor.h"
+#include "accesslog.h"
 #include "config.h"
+#include "input.h"
+#include "input_stats.h"
+#include "log.h"
 #include "markpool.h"
-#include "metacube.h"
-#include "parse.h"
-#include "server.h"
 #include "serverpool.h"
-#include "input.h"
+#include "state.pb.h"
 #include "stats.h"
+#include "stream.h"
+#include "util.h"
 #include "version.h"
-#include "state.pb.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;
+	}
 }
 
-// Serialize the given state to a file descriptor, and return the (still open)
-// descriptor.
-int make_tempfile(const CubemapStateProto &state)
+void do_nothing(int signum)
 {
-	char tmpl[] = "/tmp/cubemapstate.XXXXXX";
-	int state_fd = mkstemp(tmpl);
-	if (state_fd == -1) {
-		perror("mkstemp");
-		exit(1);
-	}
-
-	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);
-		}
-
-		ptr += ret;
-		to_write -= ret;
-	}
-
-	return state_fd;
 }
 
 CubemapStateProto collect_state(const timeval &serialize_start,
                                 const vector<Acceptor *> acceptors,
-                                const vector<Input *> inputs,
-                                ServerPool *servers,
-                                int num_servers)
+                                const multimap<string, InputWithRefcount> inputs,
+                                ServerPool *servers)
 {
-	CubemapStateProto state;
+	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);
 	
@@ -84,61 +68,15 @@ CubemapStateProto collect_state(const timeval &serialize_start,
 		state.add_acceptors()->MergeFrom(acceptors[i]->serialize());
 	}
 
-	for (size_t i = 0; i < inputs.size(); ++i) {
-		state.add_inputs()->MergeFrom(inputs[i]->serialize());
-	}
-
-	for (int i = 0; i < num_servers; ++i) {	
-		CubemapStateProto local_state = servers->get_server(i)->serialize();
-
-		// 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));
-		}
+	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());
 	}
 
 	return state;
 }
 
-// Read the state back from the file descriptor made by make_tempfile,
-// and close it.
-CubemapStateProto read_tempfile(int state_fd)
-{
-	if (lseek(state_fd, 0, SEEK_SET) == -1) {
-		perror("lseek");
-		exit(1);
-	}
-
-	string serialized;
-	char buf[4096];
-	for ( ;; ) {
-		ssize_t ret = read(state_fd, buf, sizeof(buf));
-		if (ret == -1) {
-			perror("read");
-			exit(1);
-		}
-		if (ret == 0) {
-			// EOF.
-			break;
-		}
-
-		serialized.append(string(buf, buf + ret));
-	}
-
-	close(state_fd);  // Implicitly deletes the file.
-
-	CubemapStateProto state;
-	if (!state.ParseFromString(serialized)) {
-		fprintf(stderr, "PANIC: Failed deserialization of state.\n");
-		exit(1);
-	}
-
-	return state;
-}
-	
 // Find all port statements in the configuration file, and create acceptors for htem.
 vector<Acceptor *> create_acceptors(
 	const Config &config,
@@ -172,136 +110,343 @@ vector<Acceptor *> create_acceptors(
 	return acceptors;
 }
 
+void create_config_input(const string &src, multimap<string, InputWithRefcount> *inputs)
+{
+	if (src.empty()) {
+		return;
+	}
+	if (inputs->count(src) != 0) {
+		return;
+	}
+
+	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());
+		return;
+	}
+	iwr.refcount = 0;
+	inputs->insert(make_pair(src, iwr));
+}
+
 // Find all streams in the configuration file, and create inputs for them.
-vector<Input *> create_inputs(const Config &config,
-                              map<string, Input *> *deserialized_inputs)
+void create_config_inputs(const Config &config, multimap<string, InputWithRefcount> *inputs)
 {
-	vector<Input *> inputs;
 	for (unsigned i = 0; i < config.streams.size(); ++i) {
 		const StreamConfig &stream_config = config.streams[i];
-		if (stream_config.src.empty()) {
-			continue;
-		}
-
-		string stream_id = stream_config.stream_id;
-		string src = stream_config.src;
-
-		Input *input = NULL;
-		map<string, Input *>::iterator deserialized_input_it =
-			deserialized_inputs->find(stream_id);
-		if (deserialized_input_it != deserialized_inputs->end()) {
-			input = deserialized_input_it->second;
-			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());
-				input->close_socket();
-				delete input;
-				input = NULL;
-			}
-			deserialized_inputs->erase(deserialized_input_it);
+		if (stream_config.src != "delete") {
+			create_config_input(stream_config.src, inputs);
 		}
-		if (input == NULL) {
-			input = create_input(stream_id, src);
-			if (input == NULL) {
-				fprintf(stderr, "ERROR: did not understand URL '%s', clients will not get any data.\n",
-				        src.c_str());
-				continue;
-			}
-		}
-		input->run();
-		inputs.push_back(input);
 	}
-	return inputs;
+	for (unsigned i = 0; i < config.udpstreams.size(); ++i) {
+		const UDPStreamConfig &udpstream_config = config.udpstreams[i];
+		create_config_input(udpstream_config.src, inputs);
+	}
 }
 
 void create_streams(const Config &config,
-                    const set<string> &deserialized_stream_ids,
-		    map<string, Input *> *deserialized_inputs)
+                    const set<string> &deserialized_urls,
+                    multimap<string, InputWithRefcount> *inputs)
 {
-	vector<MarkPool *> mark_pools;  // FIXME: leak
 	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));
 	}
 
-	set<string> expecting_stream_ids = deserialized_stream_ids;
+	// HTTP streams.
+	set<string> expecting_urls = deserialized_urls;
 	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);
+		int stream_index;
+
+		expecting_urls.erase(stream_config.url);
+
+		// Special-case deleted streams; they were never deserialized in the first place,
+		// so just ignore them.
+		if (stream_config.src == "delete") {
+			continue;
+		}
+
+		if (deserialized_urls.count(stream_config.url) == 0) {
+			stream_index = servers->add_stream(stream_config.url,
+			                                   stream_config.backlog_size,
+			                                   Stream::Encoding(stream_config.encoding));
+		} else {
+			stream_index = servers->lookup_stream_by_url(stream_config.url);
+			assert(stream_index != -1);
+			servers->set_backlog_size(stream_index, stream_config.backlog_size);
+			servers->set_encoding(stream_index,
+			                      Stream::Encoding(stream_config.encoding));
 		}
-		expecting_stream_ids.erase(stream_config.stream_id);
 
 		if (stream_config.mark_pool != -1) {
-			servers->set_mark_pool(stream_config.stream_id,
-			                       mark_pools[stream_config.mark_pool]);
+			servers->set_mark_pool(stream_index, mark_pools[stream_config.mark_pool]);
+		}
+
+		string src = stream_config.src;
+		if (!src.empty()) {
+			multimap<string, InputWithRefcount>::iterator input_it = inputs->find(src);
+			if (input_it != inputs->end()) {
+				input_it->second.input->add_destination(stream_index);
+				++input_it->second.refcount;
+			}
 		}
 	}
 
-	// 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();
+	// Warn about any streams servers we've lost.
+	for (set<string>::const_iterator stream_it = expecting_urls.begin();
+	     stream_it != expecting_urls.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);
+		string url = *stream_it;
+		log(WARNING, "stream '%s' disappeared from the configuration file. "
+		             "It will not be deleted, but clients will not get any new inputs. "
+		             "If you really meant to delete it, set src=delete and reload.",
+		             url.c_str());
+	}
+
+	// UDP streams.
+	for (unsigned i = 0; i < config.udpstreams.size(); ++i) {
+		const UDPStreamConfig &udpstream_config = config.udpstreams[i];
+		MarkPool *mark_pool = NULL;
+		if (udpstream_config.mark_pool != -1) {
+			mark_pool = mark_pools[udpstream_config.mark_pool];
+		}
+		int stream_index = servers->add_udpstream(udpstream_config.dst, mark_pool);
+
+		string src = udpstream_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_index);
+			++input_it->second.refcount;
+		}
+	}
+}
+	
+void open_logs(const vector<LogConfig> &log_destinations)
+{
+	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);
+
+	int status;
+	pid_t err;
+	do {
+		err = waitpid(pid, &status, 0);
+	} while (err == -1 && errno == EINTR);
+
+	if (err == -1) {
+		log_perror("waitpid()");
+		return false;
+	}	
+
+	return (WIFEXITED(status) && WEXITSTATUS(status) == 0);
+}
+
+void find_deleted_streams(const Config &config, set<string> *deleted_urls)
+{
+	for (unsigned i = 0; i < config.streams.size(); ++i) {
+		const StreamConfig &stream_config = config.streams[i];
+		if (stream_config.src == "delete") {
+			log(INFO, "Deleting stream '%s'.", stream_config.url.c_str());
+			deleted_urls->insert(stream_config.url);
 		}
 	}
 }
 
 int main(int argc, char **argv)
 {
-	fprintf(stderr, "\nCubemap " SERVER_VERSION " starting.\n");
+	signal(SIGHUP, hup);
+	signal(SIGINT, hup);
+	signal(SIGUSR1, do_nothing);  // Used in internal signalling.
+	signal(SIGPIPE, SIG_IGN);
+	
+	// Parse options.
+	int state_fd = -1;
+	bool test_config = false;
+	for ( ;; ) {
+		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;
+		}
+		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);
+		}
+	}
 
-	struct timeval serialize_start;
-	bool is_reexec = false;
+	string config_filename = "cubemap.config";
+	if (optind < argc) {
+		config_filename = argv[optind++];
+	}
+
+	// Canonicalize argv[0] and config_filename.
+	char argv0_canon[PATH_MAX];
+	char config_filename_canon[PATH_MAX];
+
+	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);
+	}
 
-	string config_filename = (argc == 1) ? "cubemap.config" : argv[1];
+	// Now parse the configuration file.
 	Config config;
-	if (!parse_config(config_filename, &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);
+		}
+	}
+
+start:
+	// Open logs as soon as possible.
+	open_logs(config.log_destinations);
+
+	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(config.num_servers);
 
+	// Find all the streams that are to be deleted.
+	set<string> deleted_urls;
+	find_deleted_streams(config, &deleted_urls);
+
 	CubemapStateProto loaded_state;
-	set<string> deserialized_stream_ids;
-	map<string, Input *> deserialized_inputs;
+	struct timeval serialize_start;
+	set<string> deserialized_urls;
 	map<int, Acceptor *> deserialized_acceptors;
-	if (argc == 4 && strcmp(argv[2], "-state") == 0) {
-		is_reexec = true;
-
-		fprintf(stderr, "Deserializing state from previous process... ");
-		int state_fd = atoi(argv[3]);
-		loaded_state = read_tempfile(state_fd);
+	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.
+		map<string, string> stream_headers_for_url;  // See below.
 		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);
+
+			if (deleted_urls.count(stream.url()) != 0) {
+				// Delete the stream backlogs.
+				for (int j = 0; j < stream.data_fds_size(); ++j) {
+					safe_close(stream.data_fds(j));
+				}
+			} else {
+				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_urls.insert(stream.url());
+
+				stream_headers_for_url.insert(make_pair(stream.url(), stream.stream_header()));
+			}
 		}
 
-		// 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(),
-				create_input(loaded_state.inputs(i))));
-		} 
+			InputProto serialized_input = loaded_state.inputs(i);
+
+			// Older versions did not store the stream header in the input,
+			// only in each stream. We need to have the stream header in the
+			// input as well, in case we create a new stream reusing the same input.
+			// Thus, we put it into place here if it's missing.
+			if (!serialized_input.has_stream_header() &&
+			    stream_headers_for_url.count(serialized_input.url()) != 0) {
+				serialized_input.set_stream_header(stream_headers_for_url[serialized_input.url()]);
+			}
 
-		// Convert the acceptor from older serialized formats.
-		if (loaded_state.has_server_sock() && loaded_state.has_port()) {
-			AcceptorProto *acceptor = loaded_state.add_acceptors();
-			acceptor->set_server_sock(loaded_state.server_sock());
-			acceptor->set_port(loaded_state.port());
-		}
+			InputWithRefcount iwr;
+			iwr.input = create_input(serialized_input);
+			iwr.refcount = 0;
+			inputs.insert(make_pair(serialized_input.url(), iwr));
+		} 
 
 		// Deserialize the acceptors.
 		for (int i = 0; i < loaded_state.acceptors_size(); ++i) {
@@ -310,49 +455,72 @@ int main(int argc, char **argv)
 				new Acceptor(loaded_state.acceptors(i))));
 		}
 
-		fprintf(stderr, "done.\n");
+		log(INFO, "Deserialization done.");
 	}
 
-	// Find all streams in the configuration file, and create them.
-	create_streams(config, deserialized_stream_ids, &deserialized_inputs);
-
-	servers->run();
-
-	vector<Acceptor *> acceptors = create_acceptors(config, &deserialized_acceptors);
-	vector<Input *> inputs = create_inputs(config, &deserialized_inputs);
+	// Add any new inputs coming from the config.
+	create_config_inputs(config, &inputs);
 	
-	// All deserialized inputs should now have been taken care of, one way or the other.
-	assert(deserialized_inputs.empty());
+	// Find all streams in the configuration file, create them, and connect to the inputs.
+	create_streams(config, deserialized_urls, &inputs);
+	vector<Acceptor *> acceptors = create_acceptors(config, &deserialized_acceptors);
 	
-	if (is_reexec) {	
-		// 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) {
+	// 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) {
+		if (deleted_urls.count(loaded_state.clients(i).url()) != 0) {
+			safe_close(loaded_state.clients(i).sock());
+		} else {
 			servers->add_client_from_serialized(loaded_state.clients(i));
 		}
 	}
+	
+	servers->run();
+
+	// 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;
+		}
+	}
 
 	// Start writing statistics.
 	StatsThread *stats_thread = NULL;
 	if (!config.stats_file.empty()) {
 		stats_thread = new StatsThread(config.stats_file, config.stats_interval);
 		stats_thread->run();
-	} else if (config.stats_interval != -1) {
-		fprintf(stderr, "WARNING: 'stats_interval' given, but no 'stats_file'. No statistics will be written.\n");
 	}
 
-	signal(SIGHUP, hup);
-	
+	InputStatsThread *input_stats_thread = NULL;
+	if (!config.input_stats_file.empty()) {
+		vector<Input*> inputs_no_refcount;
+		for (multimap<string, InputWithRefcount>::iterator input_it = inputs.begin();
+		     input_it != inputs.end(); ++input_it) {
+			inputs_no_refcount.push_back(input_it->second.input);
+		}
+
+		input_stats_thread = new InputStatsThread(config.input_stats_file, config.input_stats_interval, inputs_no_refcount);
+		input_stats_thread->run();
+	}
+
 	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) {
@@ -362,29 +530,71 @@ 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 (input_stats_thread != NULL) {
+		input_stats_thread->stop();
+		delete input_stats_thread;
+	}
 	if (stats_thread != NULL) {
 		stats_thread->stop();
+		delete stats_thread;
 	}
 	for (size_t i = 0; i < acceptors.size(); ++i) {
 		acceptors[i]->stop();
 	}
-	for (size_t i = 0; i < inputs.size(); ++i) {
-		inputs[i]->stop();
+	for (multimap<string, InputWithRefcount>::iterator input_it = inputs.begin();
+	     input_it != inputs.end();
+	     ++input_it) {
+		input_it->second.input->stop();
 	}
 	servers->stop();
 
-	fprintf(stderr, "Serializing state and re-execing...\n");
-	int state_fd = make_tempfile(collect_state(
-		serialize_start, acceptors, inputs, servers, config.num_servers));
+	CubemapStateProto state;
+	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 (unsigned i = 0; i < mark_pools.size(); ++i) {
+		delete mark_pools[i];
+	}
+	mark_pools.clear();
+
+	access_log->stop();
+	delete access_log;
+	shut_down_logging();
+
+	if (stopped) {
+		exit(0);
+	}
+
+	// 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);
 	}
 }