X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=udp_recv.c;h=316957d51050fc6e2751bfd6a48f9d1034a9e285;hp=ae2d49f1d44567034f849b7f833923af633e08ed;hb=e5b21a60d783f22886a8671b7be444f1e1205c14;hpb=a5927501e41fa3fca2975452617474e78ffecc48

diff --git a/udp_recv.c b/udp_recv.c
index ae2d49f1..316957d5 100644
--- a/udp_recv.c
+++ b/udp_recv.c
@@ -32,6 +32,7 @@ struct private_udp_recv_data {
 	/** The socket file descriptor. */
 	int fd;
 	struct btr_pool *btrp;
+	struct timeval last_read_time;
 };
 
 static void udp_recv_pre_select(struct sched *s, struct task *t)
@@ -44,14 +45,32 @@ static void udp_recv_pre_select(struct sched *s, struct task *t)
 	para_fd_set(purd->fd, &s->rfds, &s->max_fileno);
 }
 
+static int udp_check_eof(size_t sz, struct iovec iov[2])
+{
+	if (sz < FEC_EOF_PACKET_LEN)
+		return 0;
+	if (iov[0].iov_len >= FEC_EOF_PACKET_LEN) {
+		if (memcmp(iov[0].iov_base, FEC_EOF_PACKET,
+				FEC_EOF_PACKET_LEN) != 0)
+			return 0;
+		return -E_RECV_EOF;
+	}
+	if (memcmp(iov[0].iov_base, FEC_EOF_PACKET, iov[0].iov_len) != 0)
+		return 0;
+	if (memcmp(iov[1].iov_base, FEC_EOF_PACKET + iov[0].iov_len,
+			FEC_EOF_PACKET_LEN - iov[0].iov_len) != 0)
+		return 0;
+	return -E_RECV_EOF;
+}
+
 static void udp_recv_post_select(__a_unused struct sched *s, struct task *t)
 {
 	struct receiver_node *rn = container_of(t, struct receiver_node, task);
 	struct private_udp_recv_data *purd = rn->private_data;
 	struct btr_node *btrn = rn->btrn;
-	int ret;
-	char *buf = NULL;
-	size_t packet_size, bufsize;
+	size_t packet_size;
+	struct iovec iov[2];
+	int ret, iovcnt;
 
 	t->error = 0;
 	ret = btr_node_status(btrn, 0, BTR_NT_ROOT);
@@ -59,26 +78,39 @@ static void udp_recv_post_select(__a_unused struct sched *s, struct task *t)
 		goto err;
 	if (ret == 0)
 		return;
-	if (!FD_ISSET(purd->fd, &s->rfds))
+	if (!FD_ISSET(purd->fd, &s->rfds)) {
+		struct timeval tmp;
+		tv_add(&purd->last_read_time, &(struct timeval)EMBRACE(5, 0),
+			&tmp);
+		ret = -E_UDP_TIMEOUT;
+		if (tv_diff(now, &tmp, NULL) > 0)
+			goto err;
 		return;
-	bufsize = btr_pool_get_buffer(purd->btrp, &buf);
+	}
+	iovcnt = btr_pool_get_buffers(purd->btrp, iov);
 	ret = -E_UDP_OVERRUN;
-	if (bufsize == 0)
+	if (iovcnt == 0)
 		goto err;
-	ret = recv_bin_buffer(purd->fd, buf, bufsize);
+	ret = para_readv(purd->fd, iov, iovcnt);
+	/* EAGAIN is possible even if FD_ISSET */
+	if (ret < 0 && is_errno(-ret, EAGAIN))
+		return;
 	if (ret == 0)
 		ret = -E_RECV_EOF;
 	if (ret < 0)
 		goto err;
 	packet_size = ret;
-	if (packet_size >= FEC_EOF_PACKET_LEN) {
-		if (!memcmp(buf, FEC_EOF_PACKET, FEC_EOF_PACKET_LEN)) {
-			PARA_INFO_LOG("received eof packet\n");
-			ret = -E_RECV_EOF;
-			goto err;
-		}
+	ret = udp_check_eof(packet_size, iov);
+	if (ret < 0)
+		goto err;
+	purd->last_read_time = *now;
+	if (iov[0].iov_len >= packet_size)
+		btr_add_output_pool(purd->btrp, packet_size, btrn);
+	else { /* both buffers contain data */
+		btr_add_output_pool(purd->btrp, iov[0].iov_len, btrn);
+		btr_add_output_pool(purd->btrp, packet_size - iov[0].iov_len,
+			btrn);
 	}
-	btr_add_output_pool(purd->btrp, packet_size, btrn);
 	return;
 err:
 	btr_remove_node(btrn);
@@ -189,7 +221,7 @@ static int udp_recv_open(struct receiver_node *rn)
 	ret = mcast_receiver_setup(purd->fd, iface);
 	if (ret < 0) {
 		close(purd->fd);
-		return ret;
+		goto err;
 	}
 
 	ret = mark_fd_nonblocking(purd->fd);
@@ -200,6 +232,7 @@ static int udp_recv_open(struct receiver_node *rn)
 	PARA_INFO_LOG("receiving from %s:%d, fd=%d\n", c->host_arg,
 		c->port_arg, purd->fd);
 	purd->btrp = btr_pool_new("udp_recv", 320 * 1024);
+	purd->last_read_time = *now;
 	return purd->fd;
 err:
 	free(rn->private_data);