From bbfab4335b270484136e210ca96d24fb92d2c9ce Mon Sep 17 00:00:00 2001
From: Jacob Potter <jdpotter@andrew.cmu.edu>
Date: Thu, 4 Dec 2008 17:04:19 -0500
Subject: [PATCH 1/1] attempting to increase performance

---
 lwip/src/include/lwipopts.h |  7 +++--
 net/rfb.c                   | 57 +++++++++++++++++++++++--------------
 2 files changed, 40 insertions(+), 24 deletions(-)

diff --git a/lwip/src/include/lwipopts.h b/lwip/src/include/lwipopts.h
index 0b9ebbe..a085242 100644
--- a/lwip/src/include/lwipopts.h
+++ b/lwip/src/include/lwipopts.h
@@ -12,7 +12,10 @@
 #define LWIP_DHCP	1
 
 #define MEM_SIZE	65536
-#define TCP_SND_BUF     2048
-#define TCP_MSS         2048
+#define TCP_MSS         1400
+#define TCP_WND		24000
+#define TCP_SND_BUF     (16 * TCP_MSS)
+
+#define MEMP_NUM_PBUF	128
 
 #endif
diff --git a/net/rfb.c b/net/rfb.c
index eac1bdd..eff9c41 100644
--- a/net/rfb.c
+++ b/net/rfb.c
@@ -169,7 +169,7 @@ static void send_fsm(struct tcp_pcb *pcb, struct rfb_state *state) {
 	case SST_NEEDS_UPDATE:
 		outputf("RFB send: sending header");
 		/* Send a header */
-		state->frame_bytes = fb->curmode.xres * fb->curmode.yres * 3; /* XXX */
+		state->frame_bytes = fb->curmode.xres * fb->curmode.yres * fb->curmode.bytestride;
 		hdr.msgtype = 0;
 		hdr.nrects = htons(1);
 		hdr.xpos = htons(0);
@@ -186,33 +186,46 @@ static void send_fsm(struct tcp_pcb *pcb, struct rfb_state *state) {
 		/* FALL THROUGH */
 
 	case SST_SENDING:
-		left = state->frame_bytes - state->update_pos;
 
-		if (left > tcp_sndbuf(pcb)) {
-			sndlength = tcp_sndbuf(pcb);
-		} else {
-			sndlength = left;
-		}
+		while (1) {
+			left = state->frame_bytes - state->update_pos;
 
-		do {
-			err = tcp_write(pcb, fb->fbaddr + state->update_pos, sndlength, 0);
-			if (err == ERR_MEM) {
-				outputf("RFB: ERR_MEM sending %d", sndlength);
-				sndlength /= 2;
+			if (left == 0) {
+				state->send_state = SST_IDLE;
+				break;
 			}
-		} while (err == ERR_MEM && sndlength > 1);
 
-		if (err == ERR_OK) {
-			outputf("RFB: sent %d", sndlength);
-			state->update_pos += sndlength;
-		} else {
-			outputf("RFB: send error %d", err);
-		}
+			if (left > tcp_mss(pcb)) {
+				sndlength = tcp_mss(pcb);
+			} else {
+				sndlength = left;
+			}
 
-		tcp_output(pcb);
+			do {
+				err = tcp_write(pcb, fb->fbaddr + state->update_pos, sndlength, 0);
+				if (err == ERR_MEM) {
+					outputf("RFB: ERR_MEM sending %d", sndlength);
+					sndlength /= 2;
+				}
+			} while (err == ERR_MEM && sndlength > 1);
 
-		if (state->update_pos == state->frame_bytes) {
-			state->send_state = SST_IDLE;
+			if (err == ERR_OK) {
+				outputf("RFB: attempting send %d", sndlength);
+			} else {
+				outputf("RFB: send error %d", err);
+				break;
+			}
+
+			if (tcp_output(pcb) != ERR_OK) {
+				outputf("RFB: tcp_output has had enough");
+				break;
+			}
+
+			state->update_pos += sndlength;
+
+			if (tcp_sndbuf(pcb) == 0) {
+				break;
+			}
 		}
 
 		break;
-- 
2.43.0