Using %pV reduces the number of printk calls and eliminates any possible message interleaving from other printk calls. Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> --- 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html