X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=udp_send.c;h=7a1a89f73e1ab79d91fa2e0978072eaf9e10fc46;hp=ba7163a8dcdbcffcc9f60772a7d34c5f3c0aec28;hb=fb5d56ca5ec72163e2ef0665a857a9020679be0a;hpb=939ece34a0c1ae2f8c32436d31ed18de6b3c24b5

diff --git a/udp_send.c b/udp_send.c
index ba7163a8..7a1a89f7 100644
--- a/udp_send.c
+++ b/udp_send.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2005-2010 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 2005-2011 Andre Noll <maan@systemlinux.org>
  *
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
@@ -9,11 +9,11 @@
 
 #include <regex.h>
 #include <sys/time.h>
-#include <dirent.h>
 #include <sys/socket.h>
 #include <netinet/udp.h>
 #include <net/if.h>
 #include <osl.h>
+#include <stdbool.h>
 
 #include "server.cmdline.h"
 #include "para.h"
@@ -30,7 +30,6 @@
 #include "fd.h"
 #include "sched.h"
 #include "close_on_fork.h"
-#include "chunk_queue.h"
 
 /**
  * Time window during which ICMP Destination/Port Unreachable messages are
@@ -43,8 +42,6 @@
 struct udp_target {
 	/** Track time (seconds) of last ICMP Port Unreachable error */
 	time_t last_unreachable;
-	/** Common sender client data */
-	struct sender_client *sc;
 	/** The opaque structure returned by vss_add_fec_client(). */
 	struct fec_client *fc;
 	/** The FEC parameters for this target. */
@@ -56,10 +53,16 @@ static int sender_status;
 
 static void udp_close_target(struct sender_client *sc)
 {
-	if (sc->cq != NULL) {
-		cq_destroy(sc->cq);
-		sc->cq = NULL;
-	}
+	const char *buf;
+	size_t len = vss_get_fec_eof_packet(&buf);
+
+	/*
+	 * Ignore the return value of wirte() since we are closing the target
+	 * anyway. The sole purpose of the "do_nothing" statement is to silence
+	 * gcc.
+	 */
+	if (write(sc->fd, buf, len))
+		do_nothing;
 }
 
 static void udp_delete_target(struct sender_client *sc, const char *msg)
@@ -153,29 +156,16 @@ err:
 	return -ERRNO_TO_PARA_ERROR(errno);
 }
 
-/** The maximal size of the per-target chunk queue. */
-#define UDP_CQ_BYTES 40000
-
 static void udp_init_session(struct sender_client *sc)
 {
-	if (sc->cq == NULL) {
-		sc->cq = cq_new(UDP_CQ_BYTES);
-		PARA_NOTICE_LOG("sending to udp %s\n", sc->name);
-	}
+	PARA_NOTICE_LOG("sending to udp %s\n", sc->name);
 }
 
 static void udp_shutdown_targets(void)
 {
 	struct sender_client *sc, *tmp;
-	const char *buf;
-	size_t len = vss_get_fec_eof_packet(&buf);
-
 	list_for_each_entry_safe(sc, tmp, &targets, node)
-		if (sc->cq != NULL) {
-			/* ignore return value, closing the target anyway. */
-			(void)write(sc->fd, buf, len);
-			udp_close_target(sc);
-		}
+		udp_close_target(sc);
 }
 
 static int udp_resolve_target(const char *url, struct sender_command_data *scd)
@@ -294,19 +284,11 @@ static int udp_send_fec(struct sender_client *sc, char *buf, size_t len)
 
 	if (sender_status == SENDER_OFF)
 		return 0;
-	if (len == 0 && !cq_peek(sc->cq))
+	if (len == 0)
 		return 0;
 	ret = udp_check_socket_state(sc);
 	if (ret < 0)
 		goto fail;
-	ret = send_queued_chunks(sc->fd, sc->cq);
-	if (ret < 0)
-		goto fail;
-	if (!ret) { /* still data left in the queue */
-		ret = cq_force_enqueue(sc->cq, buf, len);
-		assert(ret >= 0);
-		return 0;
-	}
 	ret = write_nonblock(sc->fd, buf, len);
 	if (ret == -ERRNO_TO_PARA_ERROR(ECONNREFUSED)) {
 		/*
@@ -317,10 +299,6 @@ static int udp_send_fec(struct sender_client *sc, char *buf, size_t len)
 	}
 	if (ret < 0)
 		goto fail;
-	if (ret != len) {
-		ret = cq_force_enqueue(sc->cq, buf + ret, len - ret);
-		assert(ret >= 0);
-	}
 	return 1;
 fail:
 	udp_delete_target(sc, para_strerror(-ret));
@@ -339,11 +317,12 @@ static int udp_com_add(struct sender_command_data *scd)
 		return -E_TARGET_EXISTS;
 	}
 	ut = para_calloc(sizeof(*ut));
-	sc = ut->sc = para_calloc(sizeof(*sc));
+	sc = para_calloc(sizeof(*sc));
 	ut->fcp.slices_per_group      = scd->slices_per_group;
 	ut->fcp.data_slices_per_group = scd->data_slices_per_group;
 	ut->fcp.init_fec	      = udp_init_fec;
 	ut->fcp.send_fec	      = udp_send_fec;
+	ut->fcp.need_periodic_header  = true;
 
 	sc->private_data = ut;
 	sc->fd = -1;