diff options
author | Joe Perches <joe@perches.com> | 2010-11-09 16:35:20 -0800 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2010-11-30 10:30:41 +0000 |
commit | 5e69069c1afb655b5f1a154856ccdb4bb7327b81 (patch) | |
tree | f6bddfefc266a534b3a2879aa83f2c82ff3b3d81 /fs/gfs2/glock.c | |
parent | 2ae51ed7b548c1d943d080da617515e801ea5c3e (diff) | |
download | op-kernel-dev-5e69069c1afb655b5f1a154856ccdb4bb7327b81.zip op-kernel-dev-5e69069c1afb655b5f1a154856ccdb4bb7327b81.tar.gz |
GFS2: fs/gfs2/glock.c: Use printf extension %pV
Using %pV reduces the number of printk calls and
eliminates any possible message interleaving from
other printk calls.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/glock.c')
-rw-r--r-- | fs/gfs2/glock.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c index 13155f6..a9b53a4 100644 --- a/fs/gfs2/glock.c +++ b/fs/gfs2/glock.c @@ -951,17 +951,22 @@ int gfs2_glock_wait(struct gfs2_holder *gh) void gfs2_print_dbg(struct seq_file *seq, const char *fmt, ...) { + struct va_format vaf; va_list args; va_start(args, fmt); + if (seq) { struct gfs2_glock_iter *gi = seq->private; vsprintf(gi->string, fmt, args); seq_printf(seq, gi->string); } else { - printk(KERN_ERR " "); - vprintk(fmt, args); + vaf.fmt = fmt; + vaf.va = &args; + + printk(KERN_ERR " %pV", &vaf); } + va_end(args); } |