[Patch 3/7] tabled: Reduce verbosity in CLD client

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I got into habit of watching logs and there was too much useless
messaging in them. Some messages were stubs and reminders, so I left
them in the code, but commented out.

Signed-off-by: Pete Zaitcev <zaitcev@xxxxxxxxxx>

---
 server/cldu.c      |    9 ++-------
 server/storage.c   |    2 ++
 server/storparse.c |    5 +----
 3 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/server/cldu.c b/server/cldu.c
index ea2266f..b48d86c 100644
--- a/server/cldu.c
+++ b/server/cldu.c
@@ -609,8 +609,8 @@ static int cldu_open_x_cb(struct cldc_call_opts *carg, enum cle_err_codes errc)
 		return 0;
 	}
 
-	if (debugging)
-		applog(LOG_DEBUG, "CLD directory \"%s\" opened", sp->xfname);
+	// if (debugging)
+	// 	applog(LOG_DEBUG, "CLD directory \"%s\" opened", sp->xfname);
 
 	/*
 	 * Read the directory.
@@ -713,9 +713,6 @@ static void next_chunk(struct cld_session *sp)
 	}
 	sp->yfname = mem;
 
-	if (debugging)
-		applog(LOG_DEBUG, "opening chunk parameters %s", sp->yfname);
-
 	memset(&copts, 0, sizeof(copts));
 	copts.cb = cldu_open_y_cb;
 	copts.private = sp;
@@ -780,8 +777,6 @@ static int cldu_get_y_cb(struct cldc_call_opts *carg, enum cle_err_codes errc)
 
 	ptr = carg->u.get.buf;
 	len = carg->u.get.size;
-	if (debugging)
-		applog(LOG_DEBUG, "got %d bytes from %s", len, sp->yfname);
 	stor_parse(sp->yfname, ptr, len);
 
 close_and_next:
diff --git a/server/storage.c b/server/storage.c
index e1368a6..f1822f7 100644
--- a/server/storage.c
+++ b/server/storage.c
@@ -387,6 +387,7 @@ static int stor_add_node_addr(struct storage_node *sn,
 		sn->addr_af = res->ai_family;
 		sn->alen = res->ai_addrlen;
 
+#if 0
 		if (debugging) {
 			char nhost[41];
 			char nport[6];
@@ -400,6 +401,7 @@ static int stor_add_node_addr(struct storage_node *sn,
 				applog(LOG_INFO, "Found Chunk host");
 			}
 		}
+#endif
 
 		/* Use just the first address for now. */
 		freeaddrinfo(res0);
diff --git a/server/storparse.c b/server/storparse.c
index d1c43a9..25841d2 100644
--- a/server/storparse.c
+++ b/server/storparse.c
@@ -145,7 +145,7 @@ static void cfg_elm_end_geo(struct config_context *cc)
 	}
 
 	if (!cc->loc.rack) {
-		applog(LOG_WARNING, "%s: No rack in Geo element", cc->fname);
+		// applog(LOG_WARNING, "%s: No rack in Geo element", cc->fname);
 		goto end;
 	}
 
@@ -389,9 +389,6 @@ void stor_parse(char *fname, const char *text, size_t len)
 		applog(LOG_WARNING, "%s: No useable Socket clause", fname);
 		goto out_free_all;
 	}
-	if (debugging)
-		applog(LOG_DEBUG, "Adding NID %u host %s port %s",
-		       ctx.nid, ctx.stor_ok_host, ctx.stor_ok_port);
 	stor_add_node(ctx.nid, ctx.stor_ok_host, ctx.stor_ok_port, &ctx.loc);
 
 out_free_all:
--
To unsubscribe from this list: send the line "unsubscribe hail-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Fedora Clound]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux