From cbe6f88b5ee7fa3394e916a8a7fe893066923b73 Mon Sep 17 00:00:00 2001
From: Samuel Thibault <samuel.thibault@ens-lyon.org>
Date: Mon, 17 Dec 2012 00:05:04 +0100
Subject: Make io_select return errors

instead of returning 0 and make clients have to actually read the error.
This makes implementing "poll" much easier.

Based on Svante Signell's patch.

* pflocal/io.c (S_io_select): Return error returned by pipe_wait_readable or
pipe_wait_writable.
* trans/fifo.c (trivfs_S_io_select): Return error returned by
pipe_wait_readable or pipe_wait_writable. Return EBADF on bogus access mode.
* trans/new-fifo.c (trivfs_S_io_select): Likewise.
---
 trans/new-fifo.c | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

(limited to 'trans/new-fifo.c')

diff --git a/trans/new-fifo.c b/trans/new-fifo.c
index cc16b988..5cc44b58 100644
--- a/trans/new-fifo.c
+++ b/trans/new-fifo.c
@@ -610,12 +610,21 @@ trivfs_S_io_select (struct trivfs_protid *cred,
       if (cred->po->openmodes & O_READ)
 	{
 	  pthread_mutex_lock (&pipe->lock);
-	  if (pipe_wait_readable (pipe, 1, 1) != EWOULDBLOCK)
-	    ready |= SELECT_READ; /* Data immediately readable (or error). */
+	  err = pipe_wait_readable (pipe, 1, 1);
+	  if (err == EWOULDBLOCK)
+	    err = 0; /* Not readable, actually not an error.  */
+	  else
+	    ready |= SELECT_READ; /* Data immediately readable (or error).  */
 	  pthread_mutex_unlock (&pipe->lock);
 	}
       else
-	ready |= SELECT_READ;	/* Error immediately available...  */
+	{
+	  err = EBADF;
+	  ready |= SELECT_READ;	/* Error immediately available...  */
+	}
+      if (err)
+	/* Prevent write test from overwriting err.  */
+	*select_type &= ~SELECT_WRITE;
     }
 
   if (*select_type & SELECT_WRITE)
@@ -623,12 +632,18 @@ trivfs_S_io_select (struct trivfs_protid *cred,
       if (cred->po->openmodes & O_WRITE)
 	{
 	  pthread_mutex_lock (&pipe->lock);
-	  if (pipe_wait_writable (pipe, 1) != EWOULDBLOCK)
-	    ready |= SELECT_WRITE; /* Data immediately writable (or error). */
+	  err = pipe_wait_writable (pipe, 1);
+	  if (err == EWOULDBLOCK)
+	    err = 0; /* Not writable, actually not an error.  */
+	  else
+	    ready |= SELECT_WRITE; /* Data immediately writable (or error).  */
 	  pthread_mutex_unlock (&pipe->lock);
 	}
       else
-	ready |= SELECT_WRITE;	/* Error immediately available...  */
+	{
+	  err = EBADF;
+	  ready |= SELECT_WRITE;	/* Error immediately available...  */
+	}
     }
 
   if (ready)
-- 
cgit v1.2.3