diff options
author | Fabian Frederick <fabf@skynet.be> | 2014-06-04 19:12:13 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-06-04 19:54:21 -0400 |
commit | d1826f2a3d64f76b18e9ac4a6874c5db5be24750 (patch) | |
tree | 7b69ceda50439632f982611f94e39998533b2526 /fs/efs | |
parent | f403d1dbac6d1ef28f553f3996d5bb5cea90ce15 (diff) |
fs/efs: convert printk(KERN_DEBUG to pr_debug
All KERN_DEBUG callsites being under #ifdef DEBUG we can safely convert
everything to pr_debug without changing current behaviour.
Remove #ifdef DEBUG around pr_debugs only (suggested by Joe Perches)
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/efs')
-rw-r--r-- | fs/efs/dir.c | 7 | ||||
-rw-r--r-- | fs/efs/inode.c | 18 | ||||
-rw-r--r-- | fs/efs/super.c | 18 |
3 files changed, 17 insertions, 26 deletions
diff --git a/fs/efs/dir.c b/fs/efs/dir.c index 7f970315b6c3..ce63b24f7c3e 100644 --- a/fs/efs/dir.c +++ b/fs/efs/dir.c | |||
@@ -71,10 +71,9 @@ static int efs_readdir(struct file *file, struct dir_context *ctx) | |||
71 | inodenum = be32_to_cpu(dirslot->inode); | 71 | inodenum = be32_to_cpu(dirslot->inode); |
72 | namelen = dirslot->namelen; | 72 | namelen = dirslot->namelen; |
73 | nameptr = dirslot->name; | 73 | nameptr = dirslot->name; |
74 | 74 | pr_debug("%s(): block %d slot %d/%d: inode %u, name \"%s\", namelen %u\n", | |
75 | #ifdef DEBUG | 75 | __func__, block, slot, dirblock->slots-1, |
76 | printk(KERN_DEBUG "EFS: readdir(): block %d slot %d/%d: inode %u, name \"%s\", namelen %u\n", block, slot, dirblock->slots-1, inodenum, nameptr, namelen); | 76 | inodenum, nameptr, namelen); |
77 | #endif | ||
78 | if (!namelen) | 77 | if (!namelen) |
79 | continue; | 78 | continue; |
80 | /* found the next entry */ | 79 | /* found the next entry */ |
diff --git a/fs/efs/inode.c b/fs/efs/inode.c index 6c9684aa7158..079d20306ee1 100644 --- a/fs/efs/inode.c +++ b/fs/efs/inode.c | |||
@@ -138,12 +138,8 @@ struct inode *efs_iget(struct super_block *super, unsigned long ino) | |||
138 | } | 138 | } |
139 | 139 | ||
140 | brelse(bh); | 140 | brelse(bh); |
141 | 141 | pr_debug("efs_iget(): inode %lu, extents %d, mode %o\n", | |
142 | #ifdef DEBUG | 142 | inode->i_ino, in->numextents, inode->i_mode); |
143 | printk(KERN_DEBUG "EFS: efs_iget(): inode %lu, extents %d, mode %o\n", | ||
144 | inode->i_ino, in->numextents, inode->i_mode); | ||
145 | #endif | ||
146 | |||
147 | switch (inode->i_mode & S_IFMT) { | 143 | switch (inode->i_mode & S_IFMT) { |
148 | case S_IFDIR: | 144 | case S_IFDIR: |
149 | inode->i_op = &efs_dir_inode_operations; | 145 | inode->i_op = &efs_dir_inode_operations; |
@@ -239,9 +235,8 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) { | |||
239 | return 0; | 235 | return 0; |
240 | } | 236 | } |
241 | 237 | ||
242 | #ifdef DEBUG | 238 | pr_debug("%s(): indirect search for logical block %u\n", |
243 | printk(KERN_DEBUG "EFS: map_block(): indirect search for logical block %u\n", block); | 239 | __func__, block); |
244 | #endif | ||
245 | direxts = in->extents[0].cooked.ex_offset; | 240 | direxts = in->extents[0].cooked.ex_offset; |
246 | indexts = in->numextents; | 241 | indexts = in->numextents; |
247 | 242 | ||
@@ -285,9 +280,8 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) { | |||
285 | __func__, iblock); | 280 | __func__, iblock); |
286 | return 0; | 281 | return 0; |
287 | } | 282 | } |
288 | #ifdef DEBUG | 283 | pr_debug("%s(): read indirect extent block %d\n", |
289 | printk(KERN_DEBUG "EFS: map_block(): read indirect extent block %d\n", iblock); | 284 | __func__, iblock); |
290 | #endif | ||
291 | first = 0; | 285 | first = 0; |
292 | lastblock = iblock; | 286 | lastblock = iblock; |
293 | } | 287 | } |
diff --git a/fs/efs/super.c b/fs/efs/super.c index cd1399e5fc83..7fca462ea4e3 100644 --- a/fs/efs/super.c +++ b/fs/efs/super.c | |||
@@ -184,7 +184,7 @@ static efs_block_t efs_validate_vh(struct volume_header *vh) { | |||
184 | } | 184 | } |
185 | 185 | ||
186 | #ifdef DEBUG | 186 | #ifdef DEBUG |
187 | printk(KERN_DEBUG "EFS: bf: \"%16s\"\n", vh->vh_bootfile); | 187 | pr_debug("bf: \"%16s\"\n", vh->vh_bootfile); |
188 | 188 | ||
189 | for(i = 0; i < NVDIR; i++) { | 189 | for(i = 0; i < NVDIR; i++) { |
190 | int j; | 190 | int j; |
@@ -196,9 +196,8 @@ static efs_block_t efs_validate_vh(struct volume_header *vh) { | |||
196 | name[j] = (char) 0; | 196 | name[j] = (char) 0; |
197 | 197 | ||
198 | if (name[0]) { | 198 | if (name[0]) { |
199 | printk(KERN_DEBUG "EFS: vh: %8s block: 0x%08x size: 0x%08x\n", | 199 | pr_debug("vh: %8s block: 0x%08x size: 0x%08x\n", |
200 | name, | 200 | name, (int) be32_to_cpu(vh->vh_vd[i].vd_lbn), |
201 | (int) be32_to_cpu(vh->vh_vd[i].vd_lbn), | ||
202 | (int) be32_to_cpu(vh->vh_vd[i].vd_nbytes)); | 201 | (int) be32_to_cpu(vh->vh_vd[i].vd_nbytes)); |
203 | } | 202 | } |
204 | } | 203 | } |
@@ -211,12 +210,11 @@ static efs_block_t efs_validate_vh(struct volume_header *vh) { | |||
211 | } | 210 | } |
212 | #ifdef DEBUG | 211 | #ifdef DEBUG |
213 | if (be32_to_cpu(vh->vh_pt[i].pt_nblks)) { | 212 | if (be32_to_cpu(vh->vh_pt[i].pt_nblks)) { |
214 | printk(KERN_DEBUG "EFS: pt %2d: start: %08d size: %08d type: 0x%02x (%s)\n", | 213 | pr_debug("pt %2d: start: %08d size: %08d type: 0x%02x (%s)\n", |
215 | i, | 214 | i, (int)be32_to_cpu(vh->vh_pt[i].pt_firstlbn), |
216 | (int) be32_to_cpu(vh->vh_pt[i].pt_firstlbn), | 215 | (int)be32_to_cpu(vh->vh_pt[i].pt_nblks), |
217 | (int) be32_to_cpu(vh->vh_pt[i].pt_nblks), | 216 | pt_type, (pt_entry->pt_name) ? |
218 | pt_type, | 217 | pt_entry->pt_name : "unknown"); |
219 | (pt_entry->pt_name) ? pt_entry->pt_name : "unknown"); | ||
220 | } | 218 | } |
221 | #endif | 219 | #endif |
222 | if (IS_EFS(pt_type)) { | 220 | if (IS_EFS(pt_type)) { |