From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: [PATCH 9/9] net/sunrpc/svc.c: Use printf extension %pV Date: Tue, 9 Nov 2010 16:35:23 -0800 Message-ID: References: Cc: "J. Bruce Fields" , Neil Brown , Trond Myklebust , "David S. Miller" , linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Return-path: In-Reply-To: Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org Using %pV reduces the number of printk calls and eliminates any possible message interleaving from other printk calls. Signed-off-by: Joe Perches --- net/sunrpc/svc.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/net/sunrpc/svc.c b/net/sunrpc/svc.c index 6359c42..e28ddb3 100644 --- a/net/sunrpc/svc.c +++ b/net/sunrpc/svc.c @@ -962,6 +962,7 @@ static int __attribute__ ((format (printf, 2, 3))) svc_printk(struct svc_rqst *rqstp, const char *fmt, ...) { + struct va_format vaf; va_list args; int r; char buf[RPC_MAX_ADDRBUFLEN]; @@ -969,11 +970,14 @@ svc_printk(struct svc_rqst *rqstp, const char *fmt, ...) if (!net_ratelimit()) return 0; - printk(KERN_WARNING "svc: %s: ", - svc_print_addr(rqstp, buf, sizeof(buf))); - va_start(args, fmt); - r = vprintk(fmt, args); + + vaf.fmt = fmt; + vaf.va = &args; + + r = printk(KERN_WARNING "svc: %s: %pV", + svc_print_addr(rqstp, buf, sizeof(buf)), &vaf); + va_end(args); return r; -- 1.7.3.1.g432b3.dirty -- 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