X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=dccp_send.c;h=738e9b6c26f212227bad88e9074667fefde75684;hp=546056a04282b9b87c27b9a4cfb811fd005f761d;hb=5b30367a7e47fe5b9014599586e819305d76c9b0;hpb=8b2c02c8d6255583982e767374d39375b3380c4d

diff --git a/dccp_send.c b/dccp_send.c
index 546056a0..738e9b6c 100644
--- a/dccp_send.c
+++ b/dccp_send.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2006 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 2006-2007 Andre Noll <maan@systemlinux.org>
  *
  *     This program is free software; you can redistribute it and/or modify
  *     it under the terms of the GNU General Public License as published by
@@ -26,13 +26,15 @@
 #include "server.h"
 #include "net.h"
 #include "list.h"
-#include "afs.h"
+#include "vss.h"
 #include "send.h"
 #include "dccp.h"
 #include "error.h"
 #include "string.h"
+#include "fd.h"
+#include "close_on_fork.h"
 #include "server.cmdline.h"
-extern struct gengetopt_args_info conf;
+
 /** the list of connected clients **/
 static struct list_head clients;
 static int listen_fd = -1;
@@ -74,7 +76,9 @@ static void dccp_post_select(fd_set *rfds, __a_unused fd_set *wfds)
 	}
 	PARA_NOTICE_LOG("connection from %s\n", inet_ntoa(dc->addr.sin_addr));
 	dc->fd = ret;
-	list_add(&dc->node, &clients);
+	para_list_add(&dc->node, &clients);
+	add_close_on_fork_list(dc->fd);
+	mark_fd_nonblock(dc->fd);
 }
 
 static int dccp_open(void)
@@ -101,6 +105,8 @@ static int dccp_open(void)
 	if (ret < 0)
 		return -E_DCCP_LISTEN;
 	PARA_DEBUG_LOG("listening on fd %d\n", listen_fd);
+	add_close_on_fork_list(listen_fd);
+	mark_fd_nonblock(listen_fd);
 	return 1;
 }
 
@@ -109,19 +115,28 @@ static void dccp_shutdown_client(struct dccp_client *dc)
 	PARA_DEBUG_LOG("shutting down %s (fd %d)\n", inet_ntoa(dc->addr.sin_addr),
 		dc->fd);
 	close(dc->fd);
+	del_close_on_fork_list(dc->fd);
 	list_del(&dc->node);
 	free(dc);
 }
 
+/** give up if write would block that many times */
+#define DCCP_WRITE_RETRIES 100
+
 static int dccp_write(int fd, const char *buf, size_t len)
 {
-	size_t send, written = 0;
-	int ret;
+	size_t size, written = 0;
+	int ret, retries = 0;
 again:
-	send = PARA_MIN(1024, len - written);
-	ret = write(fd, buf + written, send);
-	if (ret < 0)
-		goto err_out;
+	size = PARA_MIN(1024, len - written);
+	ret = write(fd, buf + written, size);
+	if (ret < 0) {
+		if (errno != EAGAIN || retries++ > DCCP_WRITE_RETRIES)
+			goto err_out;
+		PARA_DEBUG_LOG("EAGAIN #%d@%zd/%zd\n", retries, written, len);
+		goto again;
+	}
+	retries = 0;
 	written += ret;
 	if (written >= len)
 		return written;
@@ -136,8 +151,9 @@ static void dccp_send(long unsigned current_chunk,
 		__a_unused long unsigned chunks_sent, const char *buf, size_t len)
 {
 	struct dccp_client *dc, *tmp;
-	int ret, header_len;
+	int ret;
 	char *header_buf;
+	size_t header_len;
 
 	if (listen_fd < 0 || !len)
 		return;
@@ -151,26 +167,26 @@ static void dccp_send(long unsigned current_chunk,
 		if (!ret)
 			continue;
 		if (!dc->header_sent && current_chunk) {
-			header_buf = afs_get_header(&header_len);
-			if (!header_buf || header_len <= 0)
-				continue; /* header not yet available */
-			ret = dccp_write(dc->fd, header_buf, header_len);
-			if (ret != header_len) {
-				int err = errno;
-				PARA_ERROR_LOG("header write: %d/%d (%s)\n",
-					ret, header_len, ret < 0?
-					strerror(err) : "");
-				dccp_shutdown_client(dc);
-				continue;
-			}
-			dc->header_sent = 1;
-			ret = write_ok(dc->fd);
-			if (ret < 0) {
-				dccp_shutdown_client(dc);
-				continue;
+			header_buf = vss_get_header(&header_len);
+			if (header_buf && header_len > 0) {
+				ret = dccp_write(dc->fd, header_buf, header_len);
+				if (ret != header_len) {
+					int err = errno;
+					PARA_ERROR_LOG("header write: %d/%zu (%s)\n",
+						ret, header_len, ret < 0?
+						strerror(err) : "");
+					dccp_shutdown_client(dc);
+					continue;
+				}
+				dc->header_sent = 1;
+				ret = write_ok(dc->fd);
+				if (ret < 0) {
+					dccp_shutdown_client(dc);
+					continue;
+				}
+				if (!ret)
+					continue;
 			}
-			if (!ret)
-				continue;
 		}
 //		PARA_DEBUG_LOG("writing %d bytes to fd %d\n", len, dc->fd);
 		ret = dccp_write(dc->fd, buf, len);