diff options
| author | Joe Perches | 2010-11-10 01:35:20 +0100 |
|---|---|---|
| committer | Steven Whitehouse | 2010-11-30 11:30:41 +0100 |
| commit | 5e69069c1afb655b5f1a154856ccdb4bb7327b81 (patch) | |
| tree | f6bddfefc266a534b3a2879aa83f2c82ff3b3d81 /fs | |
| parent | GFS2: Clean up duplicated setattr code (diff) | |
| download | kernel-qcow2-linux-5e69069c1afb655b5f1a154856ccdb4bb7327b81.tar.gz kernel-qcow2-linux-5e69069c1afb655b5f1a154856ccdb4bb7327b81.tar.xz kernel-qcow2-linux-5e69069c1afb655b5f1a154856ccdb4bb7327b81.zip | |
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')
| -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 13155f60b59d..a9b53a48abeb 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); } |