diff options
author | Joe Perches <joe@perches.com> | 2014-03-06 15:10:45 -0500 |
---|---|---|
committer | Steven Whitehouse <swhiteho@redhat.com> | 2014-03-07 04:30:51 -0500 |
commit | d77d1b58aaf4456946b8502c67f16b52fda60303 (patch) | |
tree | 5d829949b7ce4bfcb31c9517a418931d31e48ac2 /fs/gfs2/dir.c | |
parent | a17d758b661d6fa01a0d466d7bdda3c131bb68f9 (diff) |
GFS2: Use pr_<level> more consistently
Add pr_fmt, remove embedded "GFS2: " prefixes.
This now consistently emits lower case "gfs2: " for each message.
Other miscellanea around these changes:
o Add missing newlines
o Coalesce formats
o Realign arguments
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/dir.c')
-rw-r--r-- | fs/gfs2/dir.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/fs/gfs2/dir.c b/fs/gfs2/dir.c index 39c7081e4c12..1a349f9a9685 100644 --- a/fs/gfs2/dir.c +++ b/fs/gfs2/dir.c | |||
@@ -53,6 +53,8 @@ | |||
53 | * but never before the maximum hash table size has been reached. | 53 | * but never before the maximum hash table size has been reached. |
54 | */ | 54 | */ |
55 | 55 | ||
56 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
57 | |||
56 | #include <linux/slab.h> | 58 | #include <linux/slab.h> |
57 | #include <linux/spinlock.h> | 59 | #include <linux/spinlock.h> |
58 | #include <linux/buffer_head.h> | 60 | #include <linux/buffer_head.h> |
@@ -507,8 +509,8 @@ static int gfs2_check_dirent(struct gfs2_dirent *dent, unsigned int offset, | |||
507 | goto error; | 509 | goto error; |
508 | return 0; | 510 | return 0; |
509 | error: | 511 | error: |
510 | pr_warn("gfs2_check_dirent: %s (%s)\n", msg, | 512 | pr_warn("%s: %s (%s)\n", |
511 | first ? "first in block" : "not first in block"); | 513 | __func__, msg, first ? "first in block" : "not first in block"); |
512 | return -EIO; | 514 | return -EIO; |
513 | } | 515 | } |
514 | 516 | ||
@@ -531,8 +533,7 @@ static int gfs2_dirent_offset(const void *buf) | |||
531 | } | 533 | } |
532 | return offset; | 534 | return offset; |
533 | wrong_type: | 535 | wrong_type: |
534 | pr_warn("gfs2_scan_dirent: wrong block type %u\n", | 536 | pr_warn("%s: wrong block type %u\n", __func__, be32_to_cpu(h->mh_type)); |
535 | be32_to_cpu(h->mh_type)); | ||
536 | return -1; | 537 | return -1; |
537 | } | 538 | } |
538 | 539 | ||
@@ -728,7 +729,7 @@ static int get_leaf(struct gfs2_inode *dip, u64 leaf_no, | |||
728 | 729 | ||
729 | error = gfs2_meta_read(dip->i_gl, leaf_no, DIO_WAIT, bhp); | 730 | error = gfs2_meta_read(dip->i_gl, leaf_no, DIO_WAIT, bhp); |
730 | if (!error && gfs2_metatype_check(GFS2_SB(&dip->i_inode), *bhp, GFS2_METATYPE_LF)) { | 731 | if (!error && gfs2_metatype_check(GFS2_SB(&dip->i_inode), *bhp, GFS2_METATYPE_LF)) { |
731 | /* printk(KERN_INFO "block num=%llu\n", leaf_no); */ | 732 | /* pr_info("block num=%llu\n", leaf_no); */ |
732 | error = -EIO; | 733 | error = -EIO; |
733 | } | 734 | } |
734 | 735 | ||
@@ -1006,7 +1007,8 @@ static int dir_split_leaf(struct inode *inode, const struct qstr *name) | |||
1006 | len = 1 << (dip->i_depth - be16_to_cpu(oleaf->lf_depth)); | 1007 | len = 1 << (dip->i_depth - be16_to_cpu(oleaf->lf_depth)); |
1007 | half_len = len >> 1; | 1008 | half_len = len >> 1; |
1008 | if (!half_len) { | 1009 | if (!half_len) { |
1009 | pr_warn("i_depth %u lf_depth %u index %u\n", dip->i_depth, be16_to_cpu(oleaf->lf_depth), index); | 1010 | pr_warn("i_depth %u lf_depth %u index %u\n", |
1011 | dip->i_depth, be16_to_cpu(oleaf->lf_depth), index); | ||
1010 | gfs2_consist_inode(dip); | 1012 | gfs2_consist_inode(dip); |
1011 | error = -EIO; | 1013 | error = -EIO; |
1012 | goto fail_brelse; | 1014 | goto fail_brelse; |