[Sheepdog] [PATCH 1/2] sheep: clean up client_rx_handler() info

Liu Yuan namei.unix at gmail.com
Tue May 1 12:25:56 CEST 2012


From: Liu Yuan <tailai.ly at taobao.com>

It is really annoying to see below redundant debug info for only one
queue_request(). Let's have a 1:1 mapping for this.

May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 client_rx_handler(406) connection from: ::1:35638
May 01 17:11:12 queue_request(263) 3

Signed-off-by: Liu Yuan <tailai.ly at taobao.com>
---
 sheep/sdnet.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/sheep/sdnet.c b/sheep/sdnet.c
index b281e2c..5188d4f 100644
--- a/sheep/sdnet.c
+++ b/sheep/sdnet.c
@@ -403,8 +403,6 @@ static void client_rx_handler(struct client_info *ci)
 	struct sd_req *hdr = &conn->rx_hdr;
 	struct request *req;
 
-	dprintf("connection from: %s:%d\n", ci->conn.ipstr, ci->conn.port);
-
 	if (!ci->rx_req && sys->outstanding_data_size > MAX_OUTSTANDING_DATA_SIZE) {
 		dprintf("too many requests (%p)\n", &ci->conn);
 		conn_rx_off(&ci->conn);
@@ -466,6 +464,7 @@ static void client_rx_handler(struct client_info *ci)
 
 	req->done = req_done;
 
+	dprintf("connection from: %s:%d\n", ci->conn.ipstr, ci->conn.port);
 	queue_request(req);
 }
 
-- 
1.7.8.2




More information about the sheepdog mailing list