diff options
author | Roman Zippel <zippel@linux-m68k.org> | 2006-01-18 20:43:07 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-18 22:20:23 -0500 |
commit | 7cf3cc3036cb7b1147350bf7c3f1ab98c160eb7b (patch) | |
tree | 16bf33c636dec614c47959edfab9c2ef724ac93d | |
parent | 634725a92938b0f282b17cec0b007dca77adebd2 (diff) |
[PATCH] hfs: cleanup HFS prints
Add the log level and a "hfs: " prefix to all kernel prints.
Signed-off-by: Roman Zippel <zippel@linux-m68k.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/hfs/bfind.c | 3 | ||||
-rw-r--r-- | fs/hfs/bnode.c | 6 | ||||
-rw-r--r-- | fs/hfs/brec.c | 2 | ||||
-rw-r--r-- | fs/hfs/btree.c | 10 | ||||
-rw-r--r-- | fs/hfs/catalog.c | 2 | ||||
-rw-r--r-- | fs/hfs/dir.c | 12 | ||||
-rw-r--r-- | fs/hfs/hfs_fs.h | 3 | ||||
-rw-r--r-- | fs/hfs/inode.c | 1 | ||||
-rw-r--r-- | fs/hfs/mdb.c | 22 | ||||
-rw-r--r-- | fs/hfs/super.c | 40 |
10 files changed, 48 insertions, 53 deletions
diff --git a/fs/hfs/bfind.c b/fs/hfs/bfind.c index 89450ae32228..f13f1494d4fe 100644 --- a/fs/hfs/bfind.c +++ b/fs/hfs/bfind.c | |||
@@ -64,7 +64,6 @@ int __hfs_brec_find(struct hfs_bnode *bnode, struct hfs_find_data *fd) | |||
64 | else | 64 | else |
65 | e = rec - 1; | 65 | e = rec - 1; |
66 | } while (b <= e); | 66 | } while (b <= e); |
67 | //printk("%d: %d,%d,%d\n", bnode->this, b, e, rec); | ||
68 | if (rec != e && e >= 0) { | 67 | if (rec != e && e >= 0) { |
69 | len = hfs_brec_lenoff(bnode, e, &off); | 68 | len = hfs_brec_lenoff(bnode, e, &off); |
70 | keylen = hfs_brec_keylen(bnode, e); | 69 | keylen = hfs_brec_keylen(bnode, e); |
@@ -127,7 +126,7 @@ int hfs_brec_find(struct hfs_find_data *fd) | |||
127 | return res; | 126 | return res; |
128 | 127 | ||
129 | invalid: | 128 | invalid: |
130 | printk("HFS: inconsistency in B*Tree (%d,%d,%d,%u,%u)\n", | 129 | printk(KERN_ERR "hfs: inconsistency in B*Tree (%d,%d,%d,%u,%u)\n", |
131 | height, bnode->height, bnode->type, nidx, parent); | 130 | height, bnode->height, bnode->type, nidx, parent); |
132 | res = -EIO; | 131 | res = -EIO; |
133 | release: | 132 | release: |
diff --git a/fs/hfs/bnode.c b/fs/hfs/bnode.c index 3d5cdc6847c0..a7a7d77f3fd3 100644 --- a/fs/hfs/bnode.c +++ b/fs/hfs/bnode.c | |||
@@ -198,7 +198,7 @@ void hfs_bnode_unlink(struct hfs_bnode *node) | |||
198 | 198 | ||
199 | // move down? | 199 | // move down? |
200 | if (!node->prev && !node->next) { | 200 | if (!node->prev && !node->next) { |
201 | printk("hfs_btree_del_level\n"); | 201 | printk(KERN_DEBUG "hfs_btree_del_level\n"); |
202 | } | 202 | } |
203 | if (!node->parent) { | 203 | if (!node->parent) { |
204 | tree->root = 0; | 204 | tree->root = 0; |
@@ -219,7 +219,7 @@ struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid) | |||
219 | struct hfs_bnode *node; | 219 | struct hfs_bnode *node; |
220 | 220 | ||
221 | if (cnid >= tree->node_count) { | 221 | if (cnid >= tree->node_count) { |
222 | printk("HFS: request for non-existent node %d in B*Tree\n", cnid); | 222 | printk(KERN_ERR "hfs: request for non-existent node %d in B*Tree\n", cnid); |
223 | return NULL; | 223 | return NULL; |
224 | } | 224 | } |
225 | 225 | ||
@@ -242,7 +242,7 @@ static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid) | |||
242 | loff_t off; | 242 | loff_t off; |
243 | 243 | ||
244 | if (cnid >= tree->node_count) { | 244 | if (cnid >= tree->node_count) { |
245 | printk("HFS: request for non-existent node %d in B*Tree\n", cnid); | 245 | printk(KERN_ERR "hfs: request for non-existent node %d in B*Tree\n", cnid); |
246 | return NULL; | 246 | return NULL; |
247 | } | 247 | } |
248 | 248 | ||
diff --git a/fs/hfs/brec.c b/fs/hfs/brec.c index 7d8fff2c25fc..5c87cf4801fc 100644 --- a/fs/hfs/brec.c +++ b/fs/hfs/brec.c | |||
@@ -362,7 +362,7 @@ again: | |||
362 | end_off = hfs_bnode_read_u16(parent, end_rec_off); | 362 | end_off = hfs_bnode_read_u16(parent, end_rec_off); |
363 | if (end_rec_off - end_off < diff) { | 363 | if (end_rec_off - end_off < diff) { |
364 | 364 | ||
365 | printk("splitting index node...\n"); | 365 | printk(KERN_DEBUG "hfs: splitting index node...\n"); |
366 | fd->bnode = parent; | 366 | fd->bnode = parent; |
367 | new_node = hfs_bnode_split(fd); | 367 | new_node = hfs_bnode_split(fd); |
368 | if (IS_ERR(new_node)) | 368 | if (IS_ERR(new_node)) |
diff --git a/fs/hfs/btree.c b/fs/hfs/btree.c index 394725efa1c8..7bb11edd1488 100644 --- a/fs/hfs/btree.c +++ b/fs/hfs/btree.c | |||
@@ -111,7 +111,7 @@ void hfs_btree_close(struct hfs_btree *tree) | |||
111 | while ((node = tree->node_hash[i])) { | 111 | while ((node = tree->node_hash[i])) { |
112 | tree->node_hash[i] = node->next_hash; | 112 | tree->node_hash[i] = node->next_hash; |
113 | if (atomic_read(&node->refcnt)) | 113 | if (atomic_read(&node->refcnt)) |
114 | printk("HFS: node %d:%d still has %d user(s)!\n", | 114 | printk(KERN_ERR "hfs: node %d:%d still has %d user(s)!\n", |
115 | node->tree->cnid, node->this, atomic_read(&node->refcnt)); | 115 | node->tree->cnid, node->this, atomic_read(&node->refcnt)); |
116 | hfs_bnode_free(node); | 116 | hfs_bnode_free(node); |
117 | tree->node_hash_cnt--; | 117 | tree->node_hash_cnt--; |
@@ -252,7 +252,7 @@ struct hfs_bnode *hfs_bmap_alloc(struct hfs_btree *tree) | |||
252 | kunmap(*pagep); | 252 | kunmap(*pagep); |
253 | nidx = node->next; | 253 | nidx = node->next; |
254 | if (!nidx) { | 254 | if (!nidx) { |
255 | printk("create new bmap node...\n"); | 255 | printk(KERN_DEBUG "hfs: create new bmap node...\n"); |
256 | next_node = hfs_bmap_new_bmap(node, idx); | 256 | next_node = hfs_bmap_new_bmap(node, idx); |
257 | } else | 257 | } else |
258 | next_node = hfs_bnode_find(tree, nidx); | 258 | next_node = hfs_bnode_find(tree, nidx); |
@@ -292,7 +292,7 @@ void hfs_bmap_free(struct hfs_bnode *node) | |||
292 | hfs_bnode_put(node); | 292 | hfs_bnode_put(node); |
293 | if (!i) { | 293 | if (!i) { |
294 | /* panic */; | 294 | /* panic */; |
295 | printk("HFS: unable to free bnode %u. bmap not found!\n", node->this); | 295 | printk(KERN_CRIT "hfs: unable to free bnode %u. bmap not found!\n", node->this); |
296 | return; | 296 | return; |
297 | } | 297 | } |
298 | node = hfs_bnode_find(tree, i); | 298 | node = hfs_bnode_find(tree, i); |
@@ -300,7 +300,7 @@ void hfs_bmap_free(struct hfs_bnode *node) | |||
300 | return; | 300 | return; |
301 | if (node->type != HFS_NODE_MAP) { | 301 | if (node->type != HFS_NODE_MAP) { |
302 | /* panic */; | 302 | /* panic */; |
303 | printk("HFS: invalid bmap found! (%u,%d)\n", node->this, node->type); | 303 | printk(KERN_CRIT "hfs: invalid bmap found! (%u,%d)\n", node->this, node->type); |
304 | hfs_bnode_put(node); | 304 | hfs_bnode_put(node); |
305 | return; | 305 | return; |
306 | } | 306 | } |
@@ -313,7 +313,7 @@ void hfs_bmap_free(struct hfs_bnode *node) | |||
313 | m = 1 << (~nidx & 7); | 313 | m = 1 << (~nidx & 7); |
314 | byte = data[off]; | 314 | byte = data[off]; |
315 | if (!(byte & m)) { | 315 | if (!(byte & m)) { |
316 | printk("HFS: trying to free free bnode %u(%d)\n", node->this, node->type); | 316 | printk(KERN_CRIT "hfs: trying to free free bnode %u(%d)\n", node->this, node->type); |
317 | kunmap(page); | 317 | kunmap(page); |
318 | hfs_bnode_put(node); | 318 | hfs_bnode_put(node); |
319 | return; | 319 | return; |
diff --git a/fs/hfs/catalog.c b/fs/hfs/catalog.c index 2fcd679f0238..ba851576ebb1 100644 --- a/fs/hfs/catalog.c +++ b/fs/hfs/catalog.c | |||
@@ -184,7 +184,7 @@ int hfs_cat_find_brec(struct super_block *sb, u32 cnid, | |||
184 | 184 | ||
185 | type = rec.type; | 185 | type = rec.type; |
186 | if (type != HFS_CDR_THD && type != HFS_CDR_FTH) { | 186 | if (type != HFS_CDR_THD && type != HFS_CDR_FTH) { |
187 | printk("HFS-fs: Found bad thread record in catalog\n"); | 187 | printk(KERN_ERR "hfs: found bad thread record in catalog\n"); |
188 | return -EIO; | 188 | return -EIO; |
189 | } | 189 | } |
190 | 190 | ||
diff --git a/fs/hfs/dir.c b/fs/hfs/dir.c index e1f24befba58..534e5a7480ef 100644 --- a/fs/hfs/dir.c +++ b/fs/hfs/dir.c | |||
@@ -81,12 +81,12 @@ static int hfs_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
81 | case 1: | 81 | case 1: |
82 | hfs_bnode_read(fd.bnode, &entry, fd.entryoffset, fd.entrylength); | 82 | hfs_bnode_read(fd.bnode, &entry, fd.entryoffset, fd.entrylength); |
83 | if (entry.type != HFS_CDR_THD) { | 83 | if (entry.type != HFS_CDR_THD) { |
84 | printk("HFS: bad catalog folder thread\n"); | 84 | printk(KERN_ERR "hfs: bad catalog folder thread\n"); |
85 | err = -EIO; | 85 | err = -EIO; |
86 | goto out; | 86 | goto out; |
87 | } | 87 | } |
88 | //if (fd.entrylength < HFS_MIN_THREAD_SZ) { | 88 | //if (fd.entrylength < HFS_MIN_THREAD_SZ) { |
89 | // printk("HFS: truncated catalog thread\n"); | 89 | // printk(KERN_ERR "hfs: truncated catalog thread\n"); |
90 | // err = -EIO; | 90 | // err = -EIO; |
91 | // goto out; | 91 | // goto out; |
92 | //} | 92 | //} |
@@ -105,7 +105,7 @@ static int hfs_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
105 | 105 | ||
106 | for (;;) { | 106 | for (;;) { |
107 | if (be32_to_cpu(fd.key->cat.ParID) != inode->i_ino) { | 107 | if (be32_to_cpu(fd.key->cat.ParID) != inode->i_ino) { |
108 | printk("HFS: walked past end of dir\n"); | 108 | printk(KERN_ERR "hfs: walked past end of dir\n"); |
109 | err = -EIO; | 109 | err = -EIO; |
110 | goto out; | 110 | goto out; |
111 | } | 111 | } |
@@ -114,7 +114,7 @@ static int hfs_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
114 | len = hfs_mac2asc(sb, strbuf, &fd.key->cat.CName); | 114 | len = hfs_mac2asc(sb, strbuf, &fd.key->cat.CName); |
115 | if (type == HFS_CDR_DIR) { | 115 | if (type == HFS_CDR_DIR) { |
116 | if (fd.entrylength < sizeof(struct hfs_cat_dir)) { | 116 | if (fd.entrylength < sizeof(struct hfs_cat_dir)) { |
117 | printk("HFS: small dir entry\n"); | 117 | printk(KERN_ERR "hfs: small dir entry\n"); |
118 | err = -EIO; | 118 | err = -EIO; |
119 | goto out; | 119 | goto out; |
120 | } | 120 | } |
@@ -123,7 +123,7 @@ static int hfs_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
123 | break; | 123 | break; |
124 | } else if (type == HFS_CDR_FIL) { | 124 | } else if (type == HFS_CDR_FIL) { |
125 | if (fd.entrylength < sizeof(struct hfs_cat_file)) { | 125 | if (fd.entrylength < sizeof(struct hfs_cat_file)) { |
126 | printk("HFS: small file entry\n"); | 126 | printk(KERN_ERR "hfs: small file entry\n"); |
127 | err = -EIO; | 127 | err = -EIO; |
128 | goto out; | 128 | goto out; |
129 | } | 129 | } |
@@ -131,7 +131,7 @@ static int hfs_readdir(struct file *filp, void *dirent, filldir_t filldir) | |||
131 | be32_to_cpu(entry.file.FlNum), DT_REG)) | 131 | be32_to_cpu(entry.file.FlNum), DT_REG)) |
132 | break; | 132 | break; |
133 | } else { | 133 | } else { |
134 | printk("HFS: bad catalog entry type %d\n", type); | 134 | printk(KERN_ERR "hfs: bad catalog entry type %d\n", type); |
135 | err = -EIO; | 135 | err = -EIO; |
136 | goto out; | 136 | goto out; |
137 | } | 137 | } |
diff --git a/fs/hfs/hfs_fs.h b/fs/hfs/hfs_fs.h index cc5dcd52e23d..18ce47ab1b71 100644 --- a/fs/hfs/hfs_fs.h +++ b/fs/hfs/hfs_fs.h | |||
@@ -35,9 +35,6 @@ | |||
35 | #define dprint(flg, fmt, args...) \ | 35 | #define dprint(flg, fmt, args...) \ |
36 | if (flg & DBG_MASK) printk(fmt , ## args) | 36 | if (flg & DBG_MASK) printk(fmt , ## args) |
37 | 37 | ||
38 | #define hfs_warn(format, args...) printk(KERN_WARNING format , ## args) | ||
39 | #define hfs_error(format, args...) printk(KERN_ERR format , ## args) | ||
40 | |||
41 | /* | 38 | /* |
42 | * struct hfs_inode_info | 39 | * struct hfs_inode_info |
43 | * | 40 | * |
diff --git a/fs/hfs/inode.c b/fs/hfs/inode.c index 050a49276499..39fd85b9b916 100644 --- a/fs/hfs/inode.c +++ b/fs/hfs/inode.c | |||
@@ -95,7 +95,6 @@ static int hfs_releasepage(struct page *page, gfp_t mask) | |||
95 | } while (--i && nidx < tree->node_count); | 95 | } while (--i && nidx < tree->node_count); |
96 | spin_unlock(&tree->hash_lock); | 96 | spin_unlock(&tree->hash_lock); |
97 | } | 97 | } |
98 | //printk("releasepage: %lu,%x = %d\n", page->index, mask, res); | ||
99 | return res ? try_to_free_buffers(page) : 0; | 98 | return res ? try_to_free_buffers(page) : 0; |
100 | } | 99 | } |
101 | 100 | ||
diff --git a/fs/hfs/mdb.c b/fs/hfs/mdb.c index 0a473f79c89f..b4651e128d7f 100644 --- a/fs/hfs/mdb.c +++ b/fs/hfs/mdb.c | |||
@@ -47,7 +47,7 @@ static int hfs_get_last_session(struct super_block *sb, | |||
47 | *start = (sector_t)te.cdte_addr.lba << 2; | 47 | *start = (sector_t)te.cdte_addr.lba << 2; |
48 | return 0; | 48 | return 0; |
49 | } | 49 | } |
50 | printk(KERN_ERR "HFS: Invalid session number or type of track\n"); | 50 | printk(KERN_ERR "hfs: invalid session number or type of track\n"); |
51 | return -EINVAL; | 51 | return -EINVAL; |
52 | } | 52 | } |
53 | ms_info.addr_format = CDROM_LBA; | 53 | ms_info.addr_format = CDROM_LBA; |
@@ -100,7 +100,7 @@ int hfs_mdb_get(struct super_block *sb) | |||
100 | 100 | ||
101 | HFS_SB(sb)->alloc_blksz = size = be32_to_cpu(mdb->drAlBlkSiz); | 101 | HFS_SB(sb)->alloc_blksz = size = be32_to_cpu(mdb->drAlBlkSiz); |
102 | if (!size || (size & (HFS_SECTOR_SIZE - 1))) { | 102 | if (!size || (size & (HFS_SECTOR_SIZE - 1))) { |
103 | hfs_warn("hfs_fs: bad allocation block size %d\n", size); | 103 | printk(KERN_ERR "hfs: bad allocation block size %d\n", size); |
104 | goto out_bh; | 104 | goto out_bh; |
105 | } | 105 | } |
106 | 106 | ||
@@ -117,7 +117,7 @@ int hfs_mdb_get(struct super_block *sb) | |||
117 | size >>= 1; | 117 | size >>= 1; |
118 | brelse(bh); | 118 | brelse(bh); |
119 | if (!sb_set_blocksize(sb, size)) { | 119 | if (!sb_set_blocksize(sb, size)) { |
120 | printk("hfs_fs: unable to set blocksize to %u\n", size); | 120 | printk(KERN_ERR "hfs: unable to set blocksize to %u\n", size); |
121 | goto out; | 121 | goto out; |
122 | } | 122 | } |
123 | 123 | ||
@@ -161,8 +161,8 @@ int hfs_mdb_get(struct super_block *sb) | |||
161 | } | 161 | } |
162 | 162 | ||
163 | if (!HFS_SB(sb)->alt_mdb) { | 163 | if (!HFS_SB(sb)->alt_mdb) { |
164 | hfs_warn("hfs_fs: unable to locate alternate MDB\n"); | 164 | printk(KERN_WARNING "hfs: unable to locate alternate MDB\n"); |
165 | hfs_warn("hfs_fs: continuing without an alternate MDB\n"); | 165 | printk(KERN_WARNING "hfs: continuing without an alternate MDB\n"); |
166 | } | 166 | } |
167 | 167 | ||
168 | HFS_SB(sb)->bitmap = (__be32 *)__get_free_pages(GFP_KERNEL, PAGE_SIZE < 8192 ? 1 : 0); | 168 | HFS_SB(sb)->bitmap = (__be32 *)__get_free_pages(GFP_KERNEL, PAGE_SIZE < 8192 ? 1 : 0); |
@@ -177,7 +177,7 @@ int hfs_mdb_get(struct super_block *sb) | |||
177 | while (size) { | 177 | while (size) { |
178 | bh = sb_bread(sb, off >> sb->s_blocksize_bits); | 178 | bh = sb_bread(sb, off >> sb->s_blocksize_bits); |
179 | if (!bh) { | 179 | if (!bh) { |
180 | hfs_warn("hfs_fs: unable to read volume bitmap\n"); | 180 | printk(KERN_ERR "hfs: unable to read volume bitmap\n"); |
181 | goto out; | 181 | goto out; |
182 | } | 182 | } |
183 | off2 = off & (sb->s_blocksize - 1); | 183 | off2 = off & (sb->s_blocksize - 1); |
@@ -191,23 +191,23 @@ int hfs_mdb_get(struct super_block *sb) | |||
191 | 191 | ||
192 | HFS_SB(sb)->ext_tree = hfs_btree_open(sb, HFS_EXT_CNID, hfs_ext_keycmp); | 192 | HFS_SB(sb)->ext_tree = hfs_btree_open(sb, HFS_EXT_CNID, hfs_ext_keycmp); |
193 | if (!HFS_SB(sb)->ext_tree) { | 193 | if (!HFS_SB(sb)->ext_tree) { |
194 | hfs_warn("hfs_fs: unable to open extent tree\n"); | 194 | printk(KERN_ERR "hfs: unable to open extent tree\n"); |
195 | goto out; | 195 | goto out; |
196 | } | 196 | } |
197 | HFS_SB(sb)->cat_tree = hfs_btree_open(sb, HFS_CAT_CNID, hfs_cat_keycmp); | 197 | HFS_SB(sb)->cat_tree = hfs_btree_open(sb, HFS_CAT_CNID, hfs_cat_keycmp); |
198 | if (!HFS_SB(sb)->cat_tree) { | 198 | if (!HFS_SB(sb)->cat_tree) { |
199 | hfs_warn("hfs_fs: unable to open catalog tree\n"); | 199 | printk(KERN_ERR "hfs: unable to open catalog tree\n"); |
200 | goto out; | 200 | goto out; |
201 | } | 201 | } |
202 | 202 | ||
203 | attrib = mdb->drAtrb; | 203 | attrib = mdb->drAtrb; |
204 | if (!(attrib & cpu_to_be16(HFS_SB_ATTRIB_UNMNT))) { | 204 | if (!(attrib & cpu_to_be16(HFS_SB_ATTRIB_UNMNT))) { |
205 | hfs_warn("HFS-fs warning: Filesystem was not cleanly unmounted, " | 205 | printk(KERN_WARNING "hfs: filesystem was not cleanly unmounted, " |
206 | "running fsck.hfs is recommended. mounting read-only.\n"); | 206 | "running fsck.hfs is recommended. mounting read-only.\n"); |
207 | sb->s_flags |= MS_RDONLY; | 207 | sb->s_flags |= MS_RDONLY; |
208 | } | 208 | } |
209 | if ((attrib & cpu_to_be16(HFS_SB_ATTRIB_SLOCK))) { | 209 | if ((attrib & cpu_to_be16(HFS_SB_ATTRIB_SLOCK))) { |
210 | hfs_warn("HFS-fs: Filesystem is marked locked, mounting read-only.\n"); | 210 | printk(KERN_WARNING "hfs: filesystem is marked locked, mounting read-only.\n"); |
211 | sb->s_flags |= MS_RDONLY; | 211 | sb->s_flags |= MS_RDONLY; |
212 | } | 212 | } |
213 | if (!(sb->s_flags & MS_RDONLY)) { | 213 | if (!(sb->s_flags & MS_RDONLY)) { |
@@ -303,7 +303,7 @@ void hfs_mdb_commit(struct super_block *sb) | |||
303 | while (size) { | 303 | while (size) { |
304 | bh = sb_bread(sb, block); | 304 | bh = sb_bread(sb, block); |
305 | if (!bh) { | 305 | if (!bh) { |
306 | hfs_warn("hfs_fs: unable to read volume bitmap\n"); | 306 | printk(KERN_ERR "hfs: unable to read volume bitmap\n"); |
307 | break; | 307 | break; |
308 | } | 308 | } |
309 | len = min((int)sb->s_blocksize - off, size); | 309 | len = min((int)sb->s_blocksize - off, size); |
diff --git a/fs/hfs/super.c b/fs/hfs/super.c index c5074aeafcae..1181d116117d 100644 --- a/fs/hfs/super.c +++ b/fs/hfs/super.c | |||
@@ -101,12 +101,12 @@ static int hfs_remount(struct super_block *sb, int *flags, char *data) | |||
101 | return 0; | 101 | return 0; |
102 | if (!(*flags & MS_RDONLY)) { | 102 | if (!(*flags & MS_RDONLY)) { |
103 | if (!(HFS_SB(sb)->mdb->drAtrb & cpu_to_be16(HFS_SB_ATTRIB_UNMNT))) { | 103 | if (!(HFS_SB(sb)->mdb->drAtrb & cpu_to_be16(HFS_SB_ATTRIB_UNMNT))) { |
104 | printk("HFS-fs warning: Filesystem was not cleanly unmounted, " | 104 | printk(KERN_WARNING "hfs: filesystem was not cleanly unmounted, " |
105 | "running fsck.hfs is recommended. leaving read-only.\n"); | 105 | "running fsck.hfs is recommended. leaving read-only.\n"); |
106 | sb->s_flags |= MS_RDONLY; | 106 | sb->s_flags |= MS_RDONLY; |
107 | *flags |= MS_RDONLY; | 107 | *flags |= MS_RDONLY; |
108 | } else if (HFS_SB(sb)->mdb->drAtrb & cpu_to_be16(HFS_SB_ATTRIB_SLOCK)) { | 108 | } else if (HFS_SB(sb)->mdb->drAtrb & cpu_to_be16(HFS_SB_ATTRIB_SLOCK)) { |
109 | printk("HFS-fs: Filesystem is marked locked, leaving read-only.\n"); | 109 | printk(KERN_WARNING "hfs: filesystem is marked locked, leaving read-only.\n"); |
110 | sb->s_flags |= MS_RDONLY; | 110 | sb->s_flags |= MS_RDONLY; |
111 | *flags |= MS_RDONLY; | 111 | *flags |= MS_RDONLY; |
112 | } | 112 | } |
@@ -229,21 +229,21 @@ static int parse_options(char *options, struct hfs_sb_info *hsb) | |||
229 | switch (token) { | 229 | switch (token) { |
230 | case opt_uid: | 230 | case opt_uid: |
231 | if (match_int(&args[0], &tmp)) { | 231 | if (match_int(&args[0], &tmp)) { |
232 | printk("HFS: uid requires an argument\n"); | 232 | printk(KERN_ERR "hfs: uid requires an argument\n"); |
233 | return 0; | 233 | return 0; |
234 | } | 234 | } |
235 | hsb->s_uid = (uid_t)tmp; | 235 | hsb->s_uid = (uid_t)tmp; |
236 | break; | 236 | break; |
237 | case opt_gid: | 237 | case opt_gid: |
238 | if (match_int(&args[0], &tmp)) { | 238 | if (match_int(&args[0], &tmp)) { |
239 | printk("HFS: gid requires an argument\n"); | 239 | printk(KERN_ERR "hfs: gid requires an argument\n"); |
240 | return 0; | 240 | return 0; |
241 | } | 241 | } |
242 | hsb->s_gid = (gid_t)tmp; | 242 | hsb->s_gid = (gid_t)tmp; |
243 | break; | 243 | break; |
244 | case opt_umask: | 244 | case opt_umask: |
245 | if (match_octal(&args[0], &tmp)) { | 245 | if (match_octal(&args[0], &tmp)) { |
246 | printk("HFS: umask requires a value\n"); | 246 | printk(KERN_ERR "hfs: umask requires a value\n"); |
247 | return 0; | 247 | return 0; |
248 | } | 248 | } |
249 | hsb->s_file_umask = (umode_t)tmp; | 249 | hsb->s_file_umask = (umode_t)tmp; |
@@ -251,39 +251,39 @@ static int parse_options(char *options, struct hfs_sb_info *hsb) | |||
251 | break; | 251 | break; |
252 | case opt_file_umask: | 252 | case opt_file_umask: |
253 | if (match_octal(&args[0], &tmp)) { | 253 | if (match_octal(&args[0], &tmp)) { |
254 | printk("HFS: file_umask requires a value\n"); | 254 | printk(KERN_ERR "hfs: file_umask requires a value\n"); |
255 | return 0; | 255 | return 0; |
256 | } | 256 | } |
257 | hsb->s_file_umask = (umode_t)tmp; | 257 | hsb->s_file_umask = (umode_t)tmp; |
258 | break; | 258 | break; |
259 | case opt_dir_umask: | 259 | case opt_dir_umask: |
260 | if (match_octal(&args[0], &tmp)) { | 260 | if (match_octal(&args[0], &tmp)) { |
261 | printk("HFS: dir_umask requires a value\n"); | 261 | printk(KERN_ERR "hfs: dir_umask requires a value\n"); |
262 | return 0; | 262 | return 0; |
263 | } | 263 | } |
264 | hsb->s_dir_umask = (umode_t)tmp; | 264 | hsb->s_dir_umask = (umode_t)tmp; |
265 | break; | 265 | break; |
266 | case opt_part: | 266 | case opt_part: |
267 | if (match_int(&args[0], &hsb->part)) { | 267 | if (match_int(&args[0], &hsb->part)) { |
268 | printk("HFS: part requires an argument\n"); | 268 | printk(KERN_ERR "hfs: part requires an argument\n"); |
269 | return 0; | 269 | return 0; |
270 | } | 270 | } |
271 | break; | 271 | break; |
272 | case opt_session: | 272 | case opt_session: |
273 | if (match_int(&args[0], &hsb->session)) { | 273 | if (match_int(&args[0], &hsb->session)) { |
274 | printk("HFS: session requires an argument\n"); | 274 | printk(KERN_ERR "hfs: session requires an argument\n"); |
275 | return 0; | 275 | return 0; |
276 | } | 276 | } |
277 | break; | 277 | break; |
278 | case opt_type: | 278 | case opt_type: |
279 | if (match_fourchar(&args[0], &hsb->s_type)) { | 279 | if (match_fourchar(&args[0], &hsb->s_type)) { |
280 | printk("HFS+-fs: type requires a 4 character value\n"); | 280 | printk(KERN_ERR "hfs: type requires a 4 character value\n"); |
281 | return 0; | 281 | return 0; |
282 | } | 282 | } |
283 | break; | 283 | break; |
284 | case opt_creator: | 284 | case opt_creator: |
285 | if (match_fourchar(&args[0], &hsb->s_creator)) { | 285 | if (match_fourchar(&args[0], &hsb->s_creator)) { |
286 | printk("HFS+-fs: creator requires a 4 character value\n"); | 286 | printk(KERN_ERR "hfs: creator requires a 4 character value\n"); |
287 | return 0; | 287 | return 0; |
288 | } | 288 | } |
289 | break; | 289 | break; |
@@ -292,13 +292,13 @@ static int parse_options(char *options, struct hfs_sb_info *hsb) | |||
292 | break; | 292 | break; |
293 | case opt_codepage: | 293 | case opt_codepage: |
294 | if (hsb->nls_disk) { | 294 | if (hsb->nls_disk) { |
295 | printk("HFS+-fs: unable to change codepage\n"); | 295 | printk(KERN_ERR "hfs: unable to change codepage\n"); |
296 | return 0; | 296 | return 0; |
297 | } | 297 | } |
298 | p = match_strdup(&args[0]); | 298 | p = match_strdup(&args[0]); |
299 | hsb->nls_disk = load_nls(p); | 299 | hsb->nls_disk = load_nls(p); |
300 | if (!hsb->nls_disk) { | 300 | if (!hsb->nls_disk) { |
301 | printk("HFS+-fs: unable to load codepage \"%s\"\n", p); | 301 | printk(KERN_ERR "hfs: unable to load codepage \"%s\"\n", p); |
302 | kfree(p); | 302 | kfree(p); |
303 | return 0; | 303 | return 0; |
304 | } | 304 | } |
@@ -306,13 +306,13 @@ static int parse_options(char *options, struct hfs_sb_info *hsb) | |||
306 | break; | 306 | break; |
307 | case opt_iocharset: | 307 | case opt_iocharset: |
308 | if (hsb->nls_io) { | 308 | if (hsb->nls_io) { |
309 | printk("HFS: unable to change iocharset\n"); | 309 | printk(KERN_ERR "hfs: unable to change iocharset\n"); |
310 | return 0; | 310 | return 0; |
311 | } | 311 | } |
312 | p = match_strdup(&args[0]); | 312 | p = match_strdup(&args[0]); |
313 | hsb->nls_io = load_nls(p); | 313 | hsb->nls_io = load_nls(p); |
314 | if (!hsb->nls_io) { | 314 | if (!hsb->nls_io) { |
315 | printk("HFS: unable to load iocharset \"%s\"\n", p); | 315 | printk(KERN_ERR "hfs: unable to load iocharset \"%s\"\n", p); |
316 | kfree(p); | 316 | kfree(p); |
317 | return 0; | 317 | return 0; |
318 | } | 318 | } |
@@ -326,7 +326,7 @@ static int parse_options(char *options, struct hfs_sb_info *hsb) | |||
326 | if (hsb->nls_disk && !hsb->nls_io) { | 326 | if (hsb->nls_disk && !hsb->nls_io) { |
327 | hsb->nls_io = load_nls_default(); | 327 | hsb->nls_io = load_nls_default(); |
328 | if (!hsb->nls_io) { | 328 | if (!hsb->nls_io) { |
329 | printk("HFS: unable to load default iocharset\n"); | 329 | printk(KERN_ERR "hfs: unable to load default iocharset\n"); |
330 | return 0; | 330 | return 0; |
331 | } | 331 | } |
332 | } | 332 | } |
@@ -364,7 +364,7 @@ static int hfs_fill_super(struct super_block *sb, void *data, int silent) | |||
364 | 364 | ||
365 | res = -EINVAL; | 365 | res = -EINVAL; |
366 | if (!parse_options((char *)data, sbi)) { | 366 | if (!parse_options((char *)data, sbi)) { |
367 | hfs_warn("hfs_fs: unable to parse mount options.\n"); | 367 | printk(KERN_ERR "hfs: unable to parse mount options.\n"); |
368 | goto bail; | 368 | goto bail; |
369 | } | 369 | } |
370 | 370 | ||
@@ -375,7 +375,7 @@ static int hfs_fill_super(struct super_block *sb, void *data, int silent) | |||
375 | res = hfs_mdb_get(sb); | 375 | res = hfs_mdb_get(sb); |
376 | if (res) { | 376 | if (res) { |
377 | if (!silent) | 377 | if (!silent) |
378 | hfs_warn("VFS: Can't find a HFS filesystem on dev %s.\n", | 378 | printk(KERN_WARNING "hfs: can't find a HFS filesystem on dev %s.\n", |
379 | hfs_mdb_name(sb)); | 379 | hfs_mdb_name(sb)); |
380 | res = -EINVAL; | 380 | res = -EINVAL; |
381 | goto bail; | 381 | goto bail; |
@@ -407,7 +407,7 @@ static int hfs_fill_super(struct super_block *sb, void *data, int silent) | |||
407 | bail_iput: | 407 | bail_iput: |
408 | iput(root_inode); | 408 | iput(root_inode); |
409 | bail_no_root: | 409 | bail_no_root: |
410 | hfs_warn("hfs_fs: get root inode failed.\n"); | 410 | printk(KERN_ERR "hfs: get root inode failed.\n"); |
411 | bail: | 411 | bail: |
412 | hfs_mdb_put(sb); | 412 | hfs_mdb_put(sb); |
413 | return res; | 413 | return res; |
@@ -454,7 +454,7 @@ static void __exit exit_hfs_fs(void) | |||
454 | { | 454 | { |
455 | unregister_filesystem(&hfs_fs_type); | 455 | unregister_filesystem(&hfs_fs_type); |
456 | if (kmem_cache_destroy(hfs_inode_cachep)) | 456 | if (kmem_cache_destroy(hfs_inode_cachep)) |
457 | printk(KERN_INFO "hfs_inode_cache: not all structures were freed\n"); | 457 | printk(KERN_ERR "hfs_inode_cache: not all structures were freed\n"); |
458 | } | 458 | } |
459 | 459 | ||
460 | module_init(init_hfs_fs) | 460 | module_init(init_hfs_fs) |