Remove configuration of servers.

[dcpomatic.git] / src / lib / server.cc

diff --git a/src/lib/server.cc b/src/lib/server.cc
index 5010a2051df0e92e7e6b23efac3af963fb4f83f1..1f3f61e425a0392b791453245e0fcd6b2cd7b671 100644 (file)
--- a/src/lib/server.cc
+++ b/src/lib/server.cc
@@ -46,6 +46,9 @@ using std::multimap;
 using std::vector;
 using std::list;
 using std::cout;
+using std::cerr;
+using std::setprecision;
+using std::fixed;
 using boost::shared_ptr;
 using boost::algorithm::is_any_of;
 using boost::algorithm::split;
@@ -56,44 +59,18 @@ using boost::optional;
 using boost::lexical_cast;
 using libdcp::Size;
 
-ServerDescription::ServerDescription (shared_ptr<const cxml::Node> node)
-{
-       _host_name = node->string_child ("HostName");
-       _threads = node->number_child<int> ("Threads");
-}
-
-void
-ServerDescription::as_xml (xmlpp::Node* root) const
-{
-       root->add_child("HostName")->add_child_text (_host_name);
-       root->add_child("Threads")->add_child_text (boost::lexical_cast<string> (_threads));
-}
-
-/** Create a server description from a string of metadata returned from as_metadata().
- *  @param v Metadata.
- *  @return ServerDescription, or 0.
- */
-optional<ServerDescription>
-ServerDescription::create_from_metadata (string v)
-{
-       vector<string> b;
-       split (b, v, is_any_of (" "));
-
-       if (b.size() != 2) {
-               return optional<ServerDescription> ();
-       }
-
-       return ServerDescription (b[0], atoi (b[1].c_str ()));
-}
-
-Server::Server (shared_ptr<Log> log)
+Server::Server (shared_ptr<Log> log, bool verbose)
        : _log (log)
+       , _verbose (verbose)
 {
 
 }
 
+/** @param after_read Filled in with gettimeofday() after reading the input from the network.
+ *  @param after_encode Filled in with gettimeofday() after encoding the image.
+ */
 int
-Server::process (shared_ptr<Socket> socket)
+Server::process (shared_ptr<Socket> socket, struct timeval& after_read, struct timeval& after_encode)
 {
        uint32_t length = socket->read_uint32 ();
        scoped_array<char> buffer (new char[length]);
@@ -103,6 +80,7 @@ Server::process (shared_ptr<Socket> socket)
        shared_ptr<cxml::Document> xml (new cxml::Document ("EncodingRequest"));
        xml->read_stream (s);
        if (xml->number_child<int> ("Version") != SERVER_LINK_VERSION) {
+               cerr << "Mismatched server/client versions\n";
                _log->log ("Mismatched server/client versions");
                return -1;
        }
@@ -115,8 +93,13 @@ Server::process (shared_ptr<Socket> socket)
 
        image->read_from_socket (socket);
        DCPVideoFrame dcp_video_frame (image, xml, _log);
+
+       gettimeofday (&after_read, 0);
        
        shared_ptr<EncodedData> encoded = dcp_video_frame.encode_locally ();
+
+       gettimeofday (&after_encode, 0);
+       
        try {
                encoded->send (socket);
        } catch (std::exception& e) {
@@ -146,16 +129,24 @@ Server::worker_thread ()
                lock.unlock ();
 
                int frame = -1;
+               string ip;
 
                struct timeval start;
+               struct timeval after_read;
+               struct timeval after_encode;
+               struct timeval end;
+               
                gettimeofday (&start, 0);
                
                try {
-                       frame = process (socket);
+                       frame = process (socket, after_read, after_encode);
+                       ip = socket->socket().remote_endpoint().address().to_string();
                } catch (std::exception& e) {
                        _log->log (String::compose ("Error: %1", e.what()));
                }
-               
+
+               gettimeofday (&end, 0);
+
                socket.reset ();
                
                lock.lock ();
@@ -163,8 +154,20 @@ Server::worker_thread ()
                if (frame >= 0) {
                        struct timeval end;
                        gettimeofday (&end, 0);
-                       cout << String::compose ("Encoded frame %1 in %2", frame, seconds (end) - seconds (start)) << "\n";
-                       _log->log (String::compose ("Encoded frame %1 in %2", frame, seconds (end) - seconds (start)));
+
+                       stringstream message;
+                       message.precision (2);
+                       message << fixed
+                               << "Encoded frame " << frame << " from " << ip << ": "
+                               << "receive " << (seconds(after_read) - seconds(start)) << "s "
+                               << "encode " << (seconds(after_encode) - seconds(after_read)) << "s "
+                               << "send " << (seconds(end) - seconds(after_encode)) << "s.";
+                                                  
+                       if (_verbose) {
+                               cout << message.str() << "\n";
+                       }
+
+                       _log->log (message.str ());
                }
                
                _worker_condition.notify_all ();
@@ -174,7 +177,10 @@ Server::worker_thread ()
 void
 Server::run (int num_threads)
 {
-       _log->log (String::compose (N_("Server starting with %1 threads"), num_threads));
+       _log->log (String::compose ("Server starting with %1 threads", num_threads));
+       if (_verbose) {
+               cout << "DCP-o-matic server starting with " << num_threads << " threads.\n";
+       }
        
        for (int i = 0; i < num_threads; ++i) {
                _worker_threads.push_back (new thread (bind (&Server::worker_thread, this)));
@@ -207,6 +213,7 @@ Server::run (int num_threads)
 
 void
 Server::broadcast_thread ()
+try
 {
        boost::asio::io_service io_service;
 
@@ -225,6 +232,10 @@ Server::broadcast_thread ()
 
        io_service.run ();
 }
+catch (...)
+{
+       store_current ();
+}
 
 void
 Server::broadcast_received ()