X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=stdin.c;h=e625f3726789972386de10c8af86d68b206e82c4;hp=868e1ed0e031744e1df39368ed241547e02fd836;hb=ab273892c54e29087d2a6b0d52de8081be1b905f;hpb=d1e8efc42f76643529053877950034bf99b55c5f

diff --git a/stdin.c b/stdin.c
index 868e1ed0..e625f372 100644
--- a/stdin.c
+++ b/stdin.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2006-2009 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 2006-2011 Andre Noll <maan@systemlinux.org>
  *
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
@@ -31,22 +31,6 @@
  * of \a s.
  */
 static void stdin_pre_select(struct sched *s, struct task *t)
-{
-	struct stdin_task *sit = container_of(t, struct stdin_task, task);
-
-	if (sit->output_error && *sit->output_error < 0) {
-		t->error = *sit->output_error;
-		return;
-	}
-	t->error = 0;
-	sit->check_fd = 0;
-	if (sit->loaded >= sit->bufsize)
-		return;
-	sit->check_fd = 1;
-	para_fd_set(STDIN_FILENO, &s->rfds, &s->max_fileno);
-}
-
-static void stdin_pre_select_btr(struct sched *s, struct task *t)
 {
 	struct stdin_task *sit = container_of(t, struct stdin_task, task);
 	int ret;
@@ -70,37 +54,14 @@ static void stdin_pre_select_btr(struct sched *s, struct task *t)
  *
  * This function checks if \p STDIN_FILENO was included by in the read fd set
  * of \a s during the previous pre_select call.  If yes, and \p STDIN_FILENO
- * appears to be readable, data is read from stdin into the buffer of the
- * stdin task.
+ * appears to be readable, data is read from stdin and fed into the buffer
+ * tree.
  */
 static void stdin_post_select(struct sched *s, struct task *t)
 {
 	struct stdin_task *sit = container_of(t, struct stdin_task, task);
 	ssize_t ret;
-
-	if (sit->output_error && *sit->output_error < 0) {
-		t->error = *sit->output_error;
-		return;
-	}
-	t->error = 0;
-	if (!sit->check_fd)
-		return;
-	if (!FD_ISSET(STDIN_FILENO, &s->rfds))
-		return;
-	ret = read(STDIN_FILENO, sit->buf + sit->loaded, sit->bufsize - sit->loaded);
-	if (ret < 0)
-		t->error = ERRNO_TO_PARA_ERROR(errno);
-	else if (ret > 0)
-		sit->loaded += ret;
-	else
-		t->error = -E_STDIN_EOF;
-}
-
-static void stdin_post_select_btr(struct sched *s, struct task *t)
-{
-	struct stdin_task *sit = container_of(t, struct stdin_task, task);
-	ssize_t ret;
-	size_t sz;
+	size_t sz, n;
 	char *buf = NULL;
 
 	t->error = 0;
@@ -109,8 +70,6 @@ static void stdin_post_select_btr(struct sched *s, struct task *t)
 		goto err;
 	if (ret == 0)
 		return;
-	if (!FD_ISSET(STDIN_FILENO, &s->rfds))
-		return;
 	sz = btr_pool_get_buffer(sit->btrp, &buf);
 	if (sz == 0)
 		return;
@@ -120,15 +79,11 @@ static void stdin_post_select_btr(struct sched *s, struct task *t)
 	 * reference can not be freed, we're stuck.
 	 */
 	sz = PARA_MIN(sz, btr_pool_size(sit->btrp) / 2);
-	ret = read(STDIN_FILENO, buf, sz);
-	if (ret < 0)
-		ret = -ERRNO_TO_PARA_ERROR(errno);
-	if (ret == 0)
-		ret = -E_STDIN_EOF;
-	if (ret < 0)
-		goto err;
-	btr_add_output_pool(sit->btrp, ret, sit->btrn);
-	return;
+	ret = read_nonblock(STDIN_FILENO, buf, sz, &s->rfds, &n);
+	if (n > 0)
+		btr_add_output_pool(sit->btrp, n, sit->btrn);
+	if (ret >= 0)
+		return;
 err:
 	btr_remove_node(sit->btrn);
 	//btr_pool_free(sit->btrp);
@@ -142,26 +97,19 @@ err:
  *
  * This fills in the pre/post select function pointers of the task structure
  * given by \a sit. Moreover, the stdin file desctiptor is set to nonblocking
- * mode and \a bufsize is initialized (but no buffer is allocated).
+ * mode, and a buffer tree is created.
  */
 void stdin_set_defaults(struct stdin_task *sit)
 {
 	int ret;
 
-	sit->bufsize = 32 * 1024;
-	if (sit->btrn) {
-		sit->task.pre_select = stdin_pre_select_btr;
-		sit->task.post_select = stdin_post_select_btr;
-		sit->btrp = btr_pool_new("stdin", 64 * 1024);
-	} else {
-		sit->task.pre_select = stdin_pre_select;
-		sit->task.post_select = stdin_post_select;
-	}
+	sit->task.pre_select = stdin_pre_select;
+	sit->task.post_select = stdin_post_select;
+	sit->btrp = btr_pool_new("stdin", 64 * 1024);
 	sprintf(sit->task.status, "stdin reader");
 	ret = mark_fd_nonblocking(STDIN_FILENO);
 	if (ret >= 0)
 		return;
-	sit->output_error = NULL;
 	PARA_EMERG_LOG("%s\n", para_strerror(-ret));
 	exit(EXIT_FAILURE);
 }