Revert "io: closing one side of a duplex connection closes both."
authorRusty Russell <rusty@rustcorp.com.au>
Thu, 14 Nov 2013 07:43:31 +0000 (18:13 +1030)
committerRusty Russell <rusty@rustcorp.com.au>
Sun, 1 Dec 2013 10:08:19 +0000 (20:38 +1030)
This reverts commit 490b63852f281f0d72eb6f6dfa5e0dce36bcbe0d.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Conflicts:
ccan/io/io.c

ccan/io/io.c
ccan/io/io.h
ccan/io/test/run-12-bidir.c
ccan/io/test/run-14-duplex-both-read.c

index 54ba7da529ec433661163f2d01fb4ab335562fa8..a58b3f01a71f7526dba3f4e69fb987d8ebaea703 100644 (file)
@@ -46,12 +46,6 @@ struct io_plan io_debug(struct io_plan plan)
        current->plan = plan;
        backend_plan_changed(current);
 
        current->plan = plan;
        backend_plan_changed(current);
 
-       /* If it closed, close duplex. */
-       if (!current->plan.next && current->duplex) {
-               current->duplex->plan = io_close_();
-               backend_plan_changed(current->duplex);
-       }
-
        /* Call back into the loop immediately. */
        io_loop_return = do_io_loop(&ready);
 
        /* Call back into the loop immediately. */
        io_loop_return = do_io_loop(&ready);
 
@@ -460,14 +454,6 @@ void io_ready(struct io_conn *conn)
                backend_plan_changed(conn);
        }
        set_current(NULL);
                backend_plan_changed(conn);
        }
        set_current(NULL);
-
-       /* If it closed, close duplex if not already */
-       if (!conn->plan.next && conn->duplex && conn->duplex->plan.next) {
-               set_current(conn->duplex);
-               conn->duplex->plan = io_close();
-               backend_plan_changed(conn->duplex);
-               set_current(NULL);
-       }
 }
 
 /* Close the connection, we're done. */
 }
 
 /* Close the connection, we're done. */
index df0764bd17e1ff7f50633ffe91a45de4c02c6a22..4468cee3ab0d10d14aacd6cc3cfa3462bc8ec114 100644 (file)
@@ -394,7 +394,7 @@ bool io_timeout_(struct io_conn *conn, struct timespec ts,
  * to have two connections for the same fd, and use one for read
  * operations and one for write.
  *
  * to have two connections for the same fd, and use one for read
  * operations and one for write.
  *
- * Returning io_close() on one will close both fds!
+ * You must io_close() both of them to close the fd.
  *
  * Example:
  *     static void setup_read_write(int fd,
  *
  * Example:
  *     static void setup_read_write(int fd,
index f7a0ecab7f3871045458e4886c6653e865f0ec5f..1ab0a218d02a200e7918a079966d7eda941cd4c6 100644 (file)
@@ -25,7 +25,7 @@ static void finish_ok(struct io_conn *conn, struct data *d)
 static struct io_plan write_done(struct io_conn *conn, struct data *d)
 {
        d->state++;
 static struct io_plan write_done(struct io_conn *conn, struct data *d)
 {
        d->state++;
-       return io_idle();
+       return io_close();
 }
 
 static void init_conn(int fd, struct data *d)
 }
 
 static void init_conn(int fd, struct data *d)
index 70bdec0ad39a502967cc6aaa50c9387c97bd0d4d..d1868e2ee63e562f319e7963bfa10c6f0bacaa2b 100644 (file)
@@ -28,16 +28,7 @@ static void finish_ok(struct io_conn *conn, struct data *d)
 static struct io_plan end(struct io_conn *conn, struct data *d)
 {
        d->state++;
 static struct io_plan end(struct io_conn *conn, struct data *d)
 {
        d->state++;
-
-       /* last one out closes. */
-       if (conn == d->c1 && io_is_idle(d->c2))
-               return io_close();
-
-       /* last one out closes. */
-       if (conn == d->c2 && io_is_idle(d->c1))
-               return io_close();
-
-       return io_idle();
+       return io_close();
 }
 
 static struct io_plan make_duplex(struct io_conn *conn, struct data *d)
 }
 
 static struct io_plan make_duplex(struct io_conn *conn, struct data *d)