X-Git-Url: https://git.archive.openwrt.org/?p=project%2Flibubox.git;a=blobdiff_plain;f=ustream-fd.c;h=c8f0ef9aa1b2aa4dca63a1eed1dbe9d91cf9b492;hp=0cd2eb3d7915c1e18e8e7d683afbc8aa52006dca;hb=d01922625d21f97fcab757d0299586178333f1e0;hpb=300a809a7a6b30595962308c4215ead6ec361053

diff --git a/ustream-fd.c b/ustream-fd.c
index 0cd2eb3..c8f0ef9 100644
--- a/ustream-fd.c
+++ b/ustream-fd.c
@@ -21,7 +21,7 @@
 #include <stdio.h>
 #include "ustream.h"
 
-static void ustream_fd_set_uloop(struct ustream *s)
+static void ustream_fd_set_uloop(struct ustream *s, bool write)
 {
 	struct ustream_fd *sf = container_of(s, struct ustream_fd, stream);
 	struct ustream_buf *buf;
@@ -31,13 +31,15 @@ static void ustream_fd_set_uloop(struct ustream *s)
 		flags |= ULOOP_READ;
 
 	buf = s->w.head;
-	if (buf && s->w.data_bytes && !s->write_error)
+	if (write || (buf && s->w.data_bytes && !s->write_error))
 		flags |= ULOOP_WRITE;
 
 	uloop_fd_add(&sf->fd, flags);
+}
 
-	if (flags & ULOOP_READ)
-		sf->fd.cb(&sf->fd, ULOOP_READ);
+static void ustream_fd_set_read_blocked(struct ustream *s)
+{
+	ustream_fd_set_uloop(s, false);
 }
 
 static void ustream_fd_read_pending(struct ustream_fd *sf, bool *more)
@@ -53,17 +55,21 @@ static void ustream_fd_read_pending(struct ustream_fd *sf, bool *more)
 			break;
 
 		len = read(sf->fd.fd, buf, buflen);
-		if (!len) {
-			sf->fd.eof = true;
-			return;
-		}
-
 		if (len < 0) {
 			if (errno == EINTR)
 				continue;
 
 			if (errno == EAGAIN)
 				return;
+
+			len = 0;
+		}
+
+		if (!len) {
+			s->eof = true;
+			ustream_state_change(s);
+			ustream_fd_set_uloop(s, false);
+			return;
 		}
 
 		ustream_fill_read(s, len);
@@ -74,31 +80,38 @@ static void ustream_fd_read_pending(struct ustream_fd *sf, bool *more)
 static int ustream_fd_write(struct ustream *s, const char *buf, int buflen, bool more)
 {
 	struct ustream_fd *sf = container_of(s, struct ustream_fd, stream);
-	ssize_t len;
+	ssize_t ret = 0, len;
 
 	if (!buflen)
 		return 0;
 
-retry:
-	len = write(sf->fd.fd, buf, buflen);
-	if (!len)
-		goto retry;
+	while (buflen) {
+		len = write(sf->fd.fd, buf, buflen);
+
+		if (len < 0) {
+			if (errno == EINTR)
+				continue;
+
+			if (errno == EAGAIN || errno == EWOULDBLOCK)
+				break;
 
-	if (len < 0) {
-		if (errno == EINTR)
-			goto retry;
+			return -1;
+		}
 
-		if (errno == EAGAIN || errno == EWOULDBLOCK)
-			return 0;
+		ret += len;
+		buf += len;
+		buflen -= len;
 	}
 
-	return len;
+	if (buflen)
+		ustream_fd_set_uloop(s, true);
+
+	return ret;
 }
 
 static bool __ustream_fd_poll(struct ustream_fd *sf, unsigned int events)
 {
 	struct ustream *s = &sf->stream;
-	struct uloop_fd *fd = &sf->fd;
 	bool more = false;
 
 	if (events & ULOOP_READ)
@@ -106,13 +119,7 @@ static bool __ustream_fd_poll(struct ustream_fd *sf, unsigned int events)
 
 	if (events & ULOOP_WRITE) {
 		if (!ustream_write_pending(s))
-			ustream_fd_set_uloop(s);
-	}
-
-	if (!s->eof && fd->eof) {
-		s->eof = true;
-		ustream_fd_set_uloop(s);
-		ustream_state_change(s);
+			ustream_fd_set_uloop(s, false);
 	}
 
 	return more;
@@ -147,9 +154,9 @@ void ustream_fd_init(struct ustream_fd *sf, int fd)
 
 	sf->fd.fd = fd;
 	sf->fd.cb = ustream_uloop_cb;
-	s->set_read_blocked = ustream_fd_set_uloop;
+	s->set_read_blocked = ustream_fd_set_read_blocked;
 	s->write = ustream_fd_write;
 	s->free = ustream_fd_free;
 	s->poll = ustream_fd_poll;
-	ustream_fd_set_uloop(s);
+	ustream_fd_set_uloop(s, false);
 }