X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=blobdiff_plain;f=src%2Flib%2Fencode_server_finder.cc;h=1234dcd529daa847445f8be7df3bdc2d51d48fbb;hp=267fbb62ac78235180b42f9df680507513870ada;hb=ed0b3ee0c5a0ba11d3a1a1dfee8e71238bcab4bd;hpb=918124fb0b2fdf05bf98aee2c74c85387f1d8638

diff --git a/src/lib/encode_server_finder.cc b/src/lib/encode_server_finder.cc
index 267fbb62a..1234dcd52 100644
--- a/src/lib/encode_server_finder.cc
+++ b/src/lib/encode_server_finder.cc
@@ -56,6 +56,10 @@ EncodeServerFinder::start ()
 {
 	_search_thread = new boost::thread (boost::bind (&EncodeServerFinder::search_thread, this));
 	_listen_thread = new boost::thread (boost::bind (&EncodeServerFinder::listen_thread, this));
+#ifdef DCPOMATIC_LINUX
+	pthread_setname_np (_search_thread->native_handle(), "encode-server-search");
+	pthread_setname_np (_listen_thread->native_handle(), "encode-server-listen");
+#endif
 }
 
 
@@ -94,7 +98,8 @@ EncodeServerFinder::stop ()
 	_listen_thread = 0;
 
 	boost::mutex::scoped_lock lm (_servers_mutex);
-	_servers.clear ();
+	_good_servers.clear ();
+	_bad_servers.clear ();
 }
 
 void
@@ -202,11 +207,14 @@ EncodeServerFinder::handle_accept (boost::system::error_code ec, shared_ptr<Sock
 	xml->read_string (s);
 
 	string const ip = socket->socket().remote_endpoint().address().to_string ();
-	if (!server_found (ip) && xml->optional_number_child<int>("Version").get_value_or (0) == SERVER_LINK_VERSION) {
-		EncodeServerDescription sd (ip, xml->number_child<int> ("Threads"));
-		{
+	if (!server_found (ip)) {
+		EncodeServerDescription sd (ip, xml->number_child<int>("Threads"), xml->optional_number_child<int>("Version").get_value_or(0));
+		if (sd.link_version() == SERVER_LINK_VERSION) {
+			boost::mutex::scoped_lock lm (_servers_mutex);
+			_good_servers.push_back (sd);
+		} else {
 			boost::mutex::scoped_lock lm (_servers_mutex);
-			_servers.push_back (sd);
+			_bad_servers.push_back (sd);
 		}
 		emit (boost::bind (boost::ref (ServersListChanged)));
 	}
@@ -218,12 +226,21 @@ bool
 EncodeServerFinder::server_found (string ip) const
 {
 	boost::mutex::scoped_lock lm (_servers_mutex);
-	list<EncodeServerDescription>::const_iterator i = _servers.begin();
-	while (i != _servers.end() && i->host_name() != ip) {
+	list<EncodeServerDescription>::const_iterator i = _good_servers.begin();
+	while (i != _good_servers.end() && i->host_name() != ip) {
+		++i;
+	}
+
+	if (i != _good_servers.end()) {
+		return true;
+	}
+
+	i = _bad_servers.begin();
+	while (i != _bad_servers.end() && i->host_name() != ip) {
 		++i;
 	}
 
-	return i != _servers.end ();
+	return i != _bad_servers.end ();
 }
 
 EncodeServerFinder*
@@ -245,10 +262,17 @@ EncodeServerFinder::drop ()
 }
 
 list<EncodeServerDescription>
-EncodeServerFinder::servers () const
+EncodeServerFinder::good_servers () const
+{
+	boost::mutex::scoped_lock lm (_servers_mutex);
+	return _good_servers;
+}
+
+list<EncodeServerDescription>
+EncodeServerFinder::bad_servers () const
 {
 	boost::mutex::scoped_lock lm (_servers_mutex);
-	return _servers;
+	return _bad_servers;
 }
 
 void
@@ -257,7 +281,8 @@ EncodeServerFinder::config_changed (Config::Property what)
 	if (what == Config::USE_ANY_SERVERS || what == Config::SERVERS) {
 		{
 			boost::mutex::scoped_lock lm (_servers_mutex);
-			_servers.clear ();
+			_good_servers.clear ();
+			_bad_servers.clear ();
 		}
 		ServersListChanged ();
 		_search_condition.notify_all ();