Re: [PATCH] net: sunrpc: xdr: Change message to pr_debug level

3 messages, 3 authors, 2012-10-04 · open the first message on its own page

Re: [PATCH] net: sunrpc: xdr: Change message to pr_debug level

From: Marek Vasut <hidden>
Date: 2012-10-04 14:25:23

Dear Fabio Estevam,
commit 49f6189 (SUNRPC: Optimise away unnecessary data moves in
xdr_align_pages) has added a pr_info that floods dmesg when booting via
NFS.

There is also a note in the commit log: "Warning! Contains debugging
printks!"

,so change it to pr_debug instead.

Signed-off-by: Fabio Estevam <redacted>
Reviewed-by: Marek Vasut <redacted>
quoted hunk
---
 net/sunrpc/xdr.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/sunrpc/xdr.c b/net/sunrpc/xdr.c
index e30842b..6983efc 100644
--- a/net/sunrpc/xdr.c
+++ b/net/sunrpc/xdr.c
@@ -749,7 +749,7 @@ static unsigned int xdr_align_pages(struct xdr_stream
*xdr, unsigned int len) xdr_shrink_pagelen(buf, buf->page_len - len);
 		xdr->nwords = XDR_QUADLEN(buf->len - cur);
 	} else
-		pr_info("%s: Optimised away a data move!\n", __func__);
+		pr_debug("%s: Optimised away a data move!\n", __func__);
 	return len;
 }
Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

RE: [PATCH] net: sunrpc: xdr: Change message to pr_debug level

From: Myklebust, Trond <hidden>
Date: 2012-10-04 14:51:56

-----Original Message-----
From: Marek Vasut [mailto:marex@denx.de]
Sent: Thursday, October 04, 2012 7:25 AM
To: Fabio Estevam
Cc: davem@davemloft.net; Myklebust, Trond; netdev@vger.kernel.org;
linux-nfs@vger.kernel.org
Subject: Re: [PATCH] net: sunrpc: xdr: Change message to pr_debug level

Dear Fabio Estevam,
quoted
commit 49f6189 (SUNRPC: Optimise away unnecessary data moves in
xdr_align_pages) has added a pr_info that floods dmesg when booting
via NFS.

There is also a note in the commit log: "Warning! Contains debugging
printks!"
...which meant that it contained debugging code, and was obviously not appropriate for merging upstream. The final version has already dropped the debugging printks.

Trond

Re: [PATCH] net: sunrpc: xdr: Change message to pr_debug level

From: Marek Vasut <marex@denx.de>
Date: 2012-10-04 14:58:03

Dear Myklebust, Trond,
quoted
-----Original Message-----
From: Marek Vasut [mailto:marex@denx.de]
Sent: Thursday, October 04, 2012 7:25 AM
To: Fabio Estevam
Cc: davem@davemloft.net; Myklebust, Trond; netdev@vger.kernel.org;
linux-nfs@vger.kernel.org
Subject: Re: [PATCH] net: sunrpc: xdr: Change message to pr_debug level

Dear Fabio Estevam,
quoted
commit 49f6189 (SUNRPC: Optimise away unnecessary data moves in
xdr_align_pages) has added a pr_info that floods dmesg when booting
via NFS.

There is also a note in the commit log: "Warning! Contains debugging
printks!"
...which meant that it contained debugging code, and was obviously not
appropriate for merging upstream. The final version has already dropped
the debugging printks.
Uh oh, the version with debug prints made it into -next .

Best regards,
Marek Vasut
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help