Experiment of always writing data if available. Might waste a writev() with

EAGAIN but always saves a select() - needs testing with bandwidth-limited
and CPU-limited situations.

1 files changed, 9 insertions, 7 deletions

diff --git a/common-session.c b/common-session.c
index 6bcb7fa..e23be66 100644
--- a/common-session.c
+++ b/common-session.c
@@ -186,13 +186,7 @@ void session_loop(void(*loophandler)()) {
 		/* check for auth timeout, rekeying required etc */
 		checktimeouts();
 
-		/* process session socket's incoming/outgoing data */
-		if (ses.sock_out != -1) {
-			if (FD_ISSET(ses.sock_out, &writefd) && !isempty(&ses.writequeue)) {
-				write_packet();
-			}
-		}
-
+		/* process session socket's incoming data */
 		if (ses.sock_in != -1) {
 			if (FD_ISSET(ses.sock_in, &readfd)) {
 				if (!ses.remoteident) {
@@ -218,6 +212,14 @@ void session_loop(void(*loophandler)()) {
 		 * during rekeying ) */
 		channelio(&readfd, &writefd);
 
+		/* process session socket's outgoing data */
+		if (ses.sock_out != -1) {
+			if (!isempty(&ses.writequeue)) {
+				write_packet();
+			}
+		}
+
+
 		if (loophandler) {
 			loophandler();
 		}