X-Git-Url: https://git.sesse.net/?p=cubemap;a=blobdiff_plain;f=udpinput.cpp;h=96e0607d7d27dcb5071d2e14202e09eedd547179;hp=4222515e71c58366cc372c9faf33ea8a5b6788f4;hb=74cd48ffef90d7d0752e37a4515e4ecfb68f7c9d;hpb=b9f87872a42f32818805f3c2520555f2d6e2928d

diff --git a/udpinput.cpp b/udpinput.cpp
index 4222515..96e0607 100644
--- a/udpinput.cpp
+++ b/udpinput.cpp
@@ -1,7 +1,7 @@
 #include <assert.h>
 #include <errno.h>
 #include <poll.h>
-#include <stdio.h>
+#include <stddef.h>
 #include <stdlib.h>
 #include <sys/socket.h>
 #include <unistd.h>
@@ -9,9 +9,11 @@
 
 #include "acceptor.h"
 #include "log.h"
+#include "mutexlock.h"
 #include "serverpool.h"
 #include "state.pb.h"
 #include "udpinput.h"
+#include "util.h"
 #include "version.h"
 
 using namespace std;
@@ -28,6 +30,12 @@ UDPInput::UDPInput(const string &url)
 	assert(ok);
 
 	construct_header();
+
+	pthread_mutex_init(&stats_mutex, NULL);
+	stats.url = url;
+	stats.bytes_received = 0;
+	stats.data_bytes_received = 0;
+	stats.connect_time = time(NULL);
 }
 
 UDPInput::UDPInput(const InputProto &serialized)
@@ -40,6 +48,16 @@ UDPInput::UDPInput(const InputProto &serialized)
 	assert(ok);
 
 	construct_header();
+
+	pthread_mutex_init(&stats_mutex, NULL);
+	stats.url = url;
+	stats.bytes_received = serialized.bytes_received();
+	stats.data_bytes_received = serialized.data_bytes_received();
+	if (serialized.has_connect_time()) {
+		stats.connect_time = serialized.connect_time();
+	} else {
+		stats.connect_time = time(NULL);
+	}
 }
 
 InputProto UDPInput::serialize() const
@@ -47,20 +65,15 @@ InputProto UDPInput::serialize() const
 	InputProto serialized;
 	serialized.set_url(url);
 	serialized.set_sock(sock);
+	serialized.set_bytes_received(stats.bytes_received);
+	serialized.set_data_bytes_received(stats.data_bytes_received);
+	serialized.set_connect_time(stats.connect_time);
 	return serialized;
 }
 
 void UDPInput::close_socket()
 {
-	int ret;
-	do {
-		ret = close(sock);
-	} while (ret == -1 && errno == EINTR);
-
-	if (ret == -1) {
-		log_perror("close()");
-	}
-
+	safe_close(sock);
 	sock = -1;
 }
 	
@@ -74,15 +87,15 @@ void UDPInput::construct_header()
 	        "Connection: close\r\n";
 }
 	
-void UDPInput::add_destination(const string &stream_id)
+void UDPInput::add_destination(int stream_index)
 {
-	stream_ids.push_back(stream_id);
-	servers->set_header(stream_id, http_header, "");
+	stream_indices.push_back(stream_index);
+	servers->set_header(stream_index, http_header, "");
 }
 
 void UDPInput::do_work()
 {
-	while (!should_stop) {
+	while (!should_stop()) {
 		if (sock == -1) {
 			int port_num = atoi(port.c_str());
 			sock = create_server_socket(port_num, UDP_SOCKET);
@@ -94,26 +107,16 @@ void UDPInput::do_work()
 			}
 		}
 
-		// Since we are non-blocking, we need to wait for the right state first.
-		// Wait up to 50 ms, then check should_stop.
-		pollfd pfd;
-		pfd.fd = sock;
-		pfd.events = POLLIN;
-
-		int nfds = poll(&pfd, 1, 50);
-		if (nfds == 0 || (nfds == -1 && errno == EINTR)) {
+		// Wait for a packet, or a wakeup.
+		bool activity = wait_for_activity(sock, POLLIN, NULL);
+		if (!activity) {
+			// Most likely, should_stop was set.
 			continue;
 		}
-		if (nfds == -1) {
-			log_perror("poll");
-			close_socket();
-			continue;	
-		}
 
-		char buf[4096];
 		int ret;
 		do {
-			ret = recv(sock, buf, sizeof(buf), 0);
+			ret = recv(sock, packet_buf, sizeof(packet_buf), 0);
 		} while (ret == -1 && errno == EINTR);
 
 		if (ret <= 0) {
@@ -121,9 +124,21 @@ void UDPInput::do_work()
 			close_socket();
 			continue;
 		}
+
+		{
+			MutexLock lock(&stats_mutex);
+			stats.bytes_received += ret;
+			stats.data_bytes_received += ret;
+		}
 		
-		for (size_t i = 0; i < stream_ids.size(); ++i) {
-			servers->add_data(stream_ids[i], buf, ret);
+		for (size_t i = 0; i < stream_indices.size(); ++i) {
+			servers->add_data(stream_indices[i], packet_buf, ret, SUITABLE_FOR_STREAM_START);
 		}
 	}
 }
+
+InputStats UDPInput::get_stats() const
+{
+	MutexLock lock(&stats_mutex);
+	return stats;
+}