hfsplus: fix FS driver name in printks

Correct the name of the hfsplus FS driver as used in printk calls. "hfs:" -> "hfsplus:".

Signed-off-by Mike Fleetwood <[email protected]>

---

Hi,

Is there a current reason why the hfsplus FS driver uses "hfs:" almost exclusively rather than "hfsplus:" as its name in printk calls?

Assuming not here's a patch to fix.

(Any code which may have been copied between hfs and hfsplus has since diverged significantly).

Thanks,
Mike

---
 fs/hfsplus/bfind.c   |    2 +-
 fs/hfsplus/bnode.c   |    4 ++--
 fs/hfsplus/brec.c    |    7 ++++---
 fs/hfsplus/btree.c   |   24 ++++++++++++------------
 fs/hfsplus/catalog.c |    4 ++--
 fs/hfsplus/dir.c     |   14 +++++++-------
 fs/hfsplus/extents.c |    6 +++---
 fs/hfsplus/inode.c   |    2 +-
 fs/hfsplus/options.c |   22 +++++++++++-----------
 fs/hfsplus/super.c   |   34 +++++++++++++++++-----------------
 fs/hfsplus/wrapper.c |    6 +++---
 11 files changed, 63 insertions(+), 62 deletions(-)

fs/hfsplus/bfind.c +1 -1

index 5d799c1..b00e446 100644
@@ -137,7 +137,7 @@ int hfs_brec_find(struct hfs_find_data *fd)
 	return res;
 
 invalid:
-	printk(KERN_ERR "hfs: inconsistency in B*Tree (%d,%d,%d,%u,%u)\n",
+	printk(KERN_ERR "hfsplus: inconsistency in B*Tree (%d,%d,%d,%u,%u)\n",
 		height, bnode->height, bnode->type, nidx, parent);
 	res = -EIO;
 release:

fs/hfsplus/bnode.c +2 -2

index 1c42cc5..db020bf 100644
@@ -384,7 +384,7 @@ struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid)
 	struct hfs_bnode *node;
 
 	if (cnid >= tree->node_count) {
-		printk(KERN_ERR "hfs: request for non-existent node "
+		printk(KERN_ERR "hfsplus: request for non-existent node "
 				"%d in B*Tree\n",
 			cnid);
 		return NULL;

@@ -407,7 +407,7 @@ static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
 	loff_t off;
 
 	if (cnid >= tree->node_count) {
-		printk(KERN_ERR "hfs: request for non-existent node "
+		printk(KERN_ERR "hfsplus: request for non-existent node "
 				"%d in B*Tree\n",
 			cnid);
 		return NULL;

fs/hfsplus/brec.c +4 -3

index 2a734cf..c512fc3 100644
@@ -44,13 +44,14 @@ u16 hfs_brec_keylen(struct hfs_bnode *node, u16 rec)
 		if (!recoff)
 			return 0;
 		if (recoff > node->tree->node_size - 2) {
-			printk(KERN_ERR "hfs: recoff %d too large\n", recoff);
+			printk(KERN_ERR "hfsplus: recoff %d too large\n",
+				recoff);
 			return 0;
 		}
 
 		retval = hfs_bnode_read_u16(node, recoff) + 2;
 		if (retval > node->tree->max_key_len + 2) {
-			printk(KERN_ERR "hfs: keylen %d too large\n",
+			printk(KERN_ERR "hfsplus: keylen %d too large\n",
 				retval);
 			retval = 0;
 		}

@@ -388,7 +389,7 @@ again:
 		end_off = hfs_bnode_read_u16(parent, end_rec_off);
 		if (end_rec_off - end_off < diff) {
 
-			dprint(DBG_BNODE_MOD, "hfs: splitting index node.\n");
+			dprint(DBG_BNODE_MOD, "hfsplus: splitting index node.\n");
 			fd->bnode = parent;
 			new_node = hfs_bnode_split(fd);
 			if (IS_ERR(new_node))

fs/hfsplus/btree.c +12 -12

index 685d07d..cc13ccf 100644
@@ -41,7 +41,7 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 
 	if (!HFSPLUS_I(tree->inode)->first_blocks) {
 		printk(KERN_ERR
-		       "hfs: invalid btree extent records (0 size).\n");
+			"hfsplus: invalid btree extent records (0 size).\n");
 		goto free_inode;
 	}
 

@@ -68,12 +68,12 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 	switch (id) {
 	case HFSPLUS_EXT_CNID:
 		if (tree->max_key_len != HFSPLUS_EXT_KEYLEN - sizeof(u16)) {
-			printk(KERN_ERR "hfs: invalid extent max_key_len %d\n",
+			printk(KERN_ERR "hfsplus: invalid extent max_key_len %d\n",
 				tree->max_key_len);
 			goto fail_page;
 		}
 		if (tree->attributes & HFS_TREE_VARIDXKEYS) {
-			printk(KERN_ERR "hfs: invalid extent btree flag\n");
+			printk(KERN_ERR "hfsplus: invalid extent btree flag\n");
 			goto fail_page;
 		}
 

@@ -81,12 +81,12 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 		break;
 	case HFSPLUS_CAT_CNID:
 		if (tree->max_key_len != HFSPLUS_CAT_KEYLEN - sizeof(u16)) {
-			printk(KERN_ERR "hfs: invalid catalog max_key_len %d\n",
+			printk(KERN_ERR "hfsplus: invalid catalog max_key_len %d\n",
 				tree->max_key_len);
 			goto fail_page;
 		}
 		if (!(tree->attributes & HFS_TREE_VARIDXKEYS)) {
-			printk(KERN_ERR "hfs: invalid catalog btree flag\n");
+			printk(KERN_ERR "hfsplus: invalid catalog btree flag\n");
 			goto fail_page;
 		}
 

@@ -99,12 +99,12 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 		}
 		break;
 	default:
-		printk(KERN_ERR "hfs: unknown B*Tree requested\n");
+		printk(KERN_ERR "hfsplus: unknown B*Tree requested\n");
 		goto fail_page;
 	}
 
 	if (!(tree->attributes & HFS_TREE_BIGKEYS)) {
-		printk(KERN_ERR "hfs: invalid btree flag\n");
+		printk(KERN_ERR "hfsplus: invalid btree flag\n");
 		goto fail_page;
 	}
 

@@ -147,7 +147,7 @@ void hfs_btree_close(struct hfs_btree *tree)
 		while ((node = tree->node_hash[i])) {
 			tree->node_hash[i] = node->next_hash;
 			if (atomic_read(&node->refcnt))
-				printk(KERN_CRIT "hfs: node %d:%d "
+				printk(KERN_CRIT "hfsplus: node %d:%d "
 						"still has %d user(s)!\n",
 					node->tree->cnid, node->this,
 					atomic_read(&node->refcnt));

@@ -295,7 +295,7 @@ struct hfs_bnode *hfs_bmap_alloc(struct hfs_btree *tree)
 		kunmap(*pagep);
 		nidx = node->next;
 		if (!nidx) {
-			dprint(DBG_BNODE_MOD, "hfs: create new bmap node.\n");
+			dprint(DBG_BNODE_MOD, "hfsplus: create new bmap node.\n");
 			next_node = hfs_bmap_new_bmap(node, idx);
 		} else
 			next_node = hfs_bnode_find(tree, nidx);

@@ -337,7 +337,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
 		hfs_bnode_put(node);
 		if (!i) {
 			/* panic */;
-			printk(KERN_CRIT "hfs: unable to free bnode %u. "
+			printk(KERN_CRIT "hfsplus: unable to free bnode %u. "
 					"bmap not found!\n",
 				node->this);
 			return;

@@ -347,7 +347,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
 			return;
 		if (node->type != HFS_NODE_MAP) {
 			/* panic */;
-			printk(KERN_CRIT "hfs: invalid bmap found! "
+			printk(KERN_CRIT "hfsplus: invalid bmap found! "
 					"(%u,%d)\n",
 				node->this, node->type);
 			hfs_bnode_put(node);

@@ -362,7 +362,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
 	m = 1 << (~nidx & 7);
 	byte = data[off];
 	if (!(byte & m)) {
-		printk(KERN_CRIT "hfs: trying to free free bnode "
+		printk(KERN_CRIT "hfsplus: trying to free free bnode "
 				"%u(%d)\n",
 			node->this, node->type);
 		kunmap(page);

fs/hfsplus/catalog.c +2 -2

index 798d9c4..f2178e7 100644
@@ -186,12 +186,12 @@ int hfsplus_find_cat(struct super_block *sb, u32 cnid,
 
 	type = be16_to_cpu(tmp.type);
 	if (type != HFSPLUS_FOLDER_THREAD && type != HFSPLUS_FILE_THREAD) {
-		printk(KERN_ERR "hfs: found bad thread record in catalog\n");
+		printk(KERN_ERR "hfsplus: found bad thread record in catalog\n");
 		return -EIO;
 	}
 
 	if (be16_to_cpu(tmp.thread.nodeName.length) > 255) {
-		printk(KERN_ERR "hfs: catalog name length corrupted\n");
+		printk(KERN_ERR "hfsplus: catalog name length corrupted\n");
 		return -EIO;
 	}
 

fs/hfsplus/dir.c +7 -7

index 6b9f921..8f133ab 100644
@@ -102,7 +102,7 @@ again:
 		} else if (!dentry->d_fsdata)
 			dentry->d_fsdata = (void *)(unsigned long)cnid;
 	} else {
-		printk(KERN_ERR "hfs: invalid catalog entry type in lookup\n");
+		printk(KERN_ERR "hfsplus: invalid catalog entry type in lookup\n");
 		err = -EIO;
 		goto fail;
 	}

@@ -158,12 +158,12 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 		hfs_bnode_read(fd.bnode, &entry, fd.entryoffset,
 			fd.entrylength);
 		if (be16_to_cpu(entry.type) != HFSPLUS_FOLDER_THREAD) {
-			printk(KERN_ERR "hfs: bad catalog folder thread\n");
+			printk(KERN_ERR "hfsplus: bad catalog folder thread\n");
 			err = -EIO;
 			goto out;
 		}
 		if (fd.entrylength < HFSPLUS_MIN_THREAD_SZ) {
-			printk(KERN_ERR "hfs: truncated catalog thread\n");
+			printk(KERN_ERR "hfsplus: truncated catalog thread\n");
 			err = -EIO;
 			goto out;
 		}

@@ -182,7 +182,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 
 	for (;;) {
 		if (be32_to_cpu(fd.key->cat.parent) != inode->i_ino) {
-			printk(KERN_ERR "hfs: walked past end of dir\n");
+			printk(KERN_ERR "hfsplus: walked past end of dir\n");
 			err = -EIO;
 			goto out;
 		}

@@ -202,7 +202,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 		if (type == HFSPLUS_FOLDER) {
 			if (fd.entrylength <
 					sizeof(struct hfsplus_cat_folder)) {
-				printk(KERN_ERR "hfs: small dir entry\n");
+				printk(KERN_ERR "hfsplus: small dir entry\n");
 				err = -EIO;
 				goto out;
 			}

@@ -215,7 +215,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 				break;
 		} else if (type == HFSPLUS_FILE) {
 			if (fd.entrylength < sizeof(struct hfsplus_cat_file)) {
-				printk(KERN_ERR "hfs: small file entry\n");
+				printk(KERN_ERR "hfsplus: small file entry\n");
 				err = -EIO;
 				goto out;
 			}

@@ -223,7 +223,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 				    be32_to_cpu(entry.file.id), DT_REG))
 				break;
 		} else {
-			printk(KERN_ERR "hfs: bad catalog entry type\n");
+			printk(KERN_ERR "hfsplus: bad catalog entry type\n");
 			err = -EIO;
 			goto out;
 		}

fs/hfsplus/extents.c +3 -3

index eba76ea..df17086 100644
@@ -348,7 +348,7 @@ found:
 		if (count <= block_nr) {
 			err = hfsplus_block_free(sb, start, count);
 			if (err) {
-				printk(KERN_ERR "hfs: can't free extent\n");
+				printk(KERN_ERR "hfsplus: can't free extent\n");
 				dprint(DBG_EXTENT, " start: %u count: %u\n",
 					start, count);
 			}

@@ -359,7 +359,7 @@ found:
 			count -= block_nr;
 			err = hfsplus_block_free(sb, start + count, block_nr);
 			if (err) {
-				printk(KERN_ERR "hfs: can't free extent\n");
+				printk(KERN_ERR "hfsplus: can't free extent\n");
 				dprint(DBG_EXTENT, " start: %u count: %u\n",
 					start, count);
 			}

@@ -432,7 +432,7 @@ int hfsplus_file_extend(struct inode *inode)
 	if (sbi->alloc_file->i_size * 8 <
 	    sbi->total_blocks - sbi->free_blocks + 8) {
 		/* extend alloc file */
-		printk(KERN_ERR "hfs: extend alloc file! "
+		printk(KERN_ERR "hfsplus: extend alloc file! "
 				"(%llu,%u,%u)\n",
 			sbi->alloc_file->i_size * 8,
 			sbi->total_blocks, sbi->free_blocks);

fs/hfsplus/inode.c +1 -1

index 799b336..324dcda 100644
@@ -559,7 +559,7 @@ int hfsplus_cat_read_inode(struct inode *inode, struct hfs_find_data *fd)
 		inode->i_ctime = hfsp_mt2ut(file->attribute_mod_date);
 		HFSPLUS_I(inode)->create_date = file->create_date;
 	} else {
-		printk(KERN_ERR "hfs: bad catalog entry used to create inode\n");
+		printk(KERN_ERR "hfsplus: bad catalog entry used to create inode\n");
 		res = -EIO;
 	}
 	return res;

fs/hfsplus/options.c +11 -11

index ed257c6..9e5264d 100644
@@ -113,67 +113,67 @@ int hfsplus_parse_options(char *input, struct hfsplus_sb_info *sbi)
 		switch (token) {
 		case opt_creator:
 			if (match_fourchar(&args[0], &sbi->creator)) {
-				printk(KERN_ERR "hfs: creator requires a 4 character value\n");
+				printk(KERN_ERR "hfsplus: creator requires a 4 character value\n");
 				return 0;
 			}
 			break;
 		case opt_type:
 			if (match_fourchar(&args[0], &sbi->type)) {
-				printk(KERN_ERR "hfs: type requires a 4 character value\n");
+				printk(KERN_ERR "hfsplus: type requires a 4 character value\n");
 				return 0;
 			}
 			break;
 		case opt_umask:
 			if (match_octal(&args[0], &tmp)) {
-				printk(KERN_ERR "hfs: umask requires a value\n");
+				printk(KERN_ERR "hfsplus: umask requires a value\n");
 				return 0;
 			}
 			sbi->umask = (umode_t)tmp;
 			break;
 		case opt_uid:
 			if (match_int(&args[0], &tmp)) {
-				printk(KERN_ERR "hfs: uid requires an argument\n");
+				printk(KERN_ERR "hfsplus: uid requires an argument\n");
 				return 0;
 			}
 			sbi->uid = make_kuid(current_user_ns(), (uid_t)tmp);
 			if (!uid_valid(sbi->uid)) {
-				printk(KERN_ERR "hfs: invalid uid specified\n");
+				printk(KERN_ERR "hfsplus: invalid uid specified\n");
 				return 0;
 			}
 			break;
 		case opt_gid:
 			if (match_int(&args[0], &tmp)) {
-				printk(KERN_ERR "hfs: gid requires an argument\n");
+				printk(KERN_ERR "hfsplus: gid requires an argument\n");
 				return 0;
 			}
 			sbi->gid = make_kgid(current_user_ns(), (gid_t)tmp);
 			if (!gid_valid(sbi->gid)) {
-				printk(KERN_ERR "hfs: invalid gid specified\n");
+				printk(KERN_ERR "hfsplus: invalid gid specified\n");
 				return 0;
 			}
 			break;
 		case opt_part:
 			if (match_int(&args[0], &sbi->part)) {
-				printk(KERN_ERR "hfs: part requires an argument\n");
+				printk(KERN_ERR "hfsplus: part requires an argument\n");
 				return 0;
 			}
 			break;
 		case opt_session:
 			if (match_int(&args[0], &sbi->session)) {
-				printk(KERN_ERR "hfs: session requires an argument\n");
+				printk(KERN_ERR "hfsplus: session requires an argument\n");
 				return 0;
 			}
 			break;
 		case opt_nls:
 			if (sbi->nls) {
-				printk(KERN_ERR "hfs: unable to change nls mapping\n");
+				printk(KERN_ERR "hfsplus: unable to change nls mapping\n");
 				return 0;
 			}
 			p = match_strdup(&args[0]);
 			if (p)
 				sbi->nls = load_nls(p);
 			if (!sbi->nls) {
-				printk(KERN_ERR "hfs: unable to load "
+				printk(KERN_ERR "hfsplus: unable to load "
 						"nls mapping \"%s\"\n",
 					p);
 				kfree(p);

fs/hfsplus/super.c +17 -17

index 796198d..31cbf46 100644
@@ -130,7 +130,7 @@ static int hfsplus_system_write_inode(struct inode *inode)
 	if (tree) {
 		int err = hfs_btree_write(tree);
 		if (err) {
-			printk(KERN_ERR "hfs: b-tree write err: %d, ino %lu\n",
+			printk(KERN_ERR "hfsplus: b-tree write err: %d, ino %lu\n",
 					err, inode->i_ino);
 			return err;
 		}

@@ -243,7 +243,7 @@ static void delayed_sync_fs(struct work_struct *work)
 
 	err = hfsplus_sync_fs(sbi->alloc_file->i_sb, 1);
 	if (err)
-		printk(KERN_ERR "hfs: delayed sync fs err %d\n", err);
+		printk(KERN_ERR "hfsplus: delayed sync fs err %d\n", err);
 }
 
 void hfsplus_mark_mdb_dirty(struct super_block *sb)

@@ -324,7 +324,7 @@ static int hfsplus_remount(struct super_block *sb, int *flags, char *data)
 			return -EINVAL;
 
 		if (!(vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_UNMNT))) {
-			printk(KERN_WARNING "hfs: filesystem was "
+			printk(KERN_WARNING "hfsplus: filesystem was "
 					"not cleanly unmounted, "
 					"running fsck.hfsplus is recommended.  "
 					"leaving read-only.\n");

@@ -334,13 +334,13 @@ static int hfsplus_remount(struct super_block *sb, int *flags, char *data)
 			/* nothing */
 		} else if (vhdr->attributes &
 				cpu_to_be32(HFSPLUS_VOL_SOFTLOCK)) {
-			printk(KERN_WARNING "hfs: filesystem is marked locked, "
+			printk(KERN_WARNING "hfsplus: filesystem is marked locked, "
 					"leaving read-only.\n");
 			sb->s_flags |= MS_RDONLY;
 			*flags |= MS_RDONLY;
 		} else if (vhdr->attributes &
 				cpu_to_be32(HFSPLUS_VOL_JOURNALED)) {
-			printk(KERN_WARNING "hfs: filesystem is "
+			printk(KERN_WARNING "hfsplus: filesystem is "
 					"marked journaled, "
 					"leaving read-only.\n");
 			sb->s_flags |= MS_RDONLY;

@@ -388,7 +388,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 
 	err = -EINVAL;
 	if (!hfsplus_parse_options(data, sbi)) {
-		printk(KERN_ERR "hfs: unable to parse mount options\n");
+		printk(KERN_ERR "hfsplus: unable to parse mount options\n");
 		goto out_unload_nls;
 	}
 

@@ -396,14 +396,14 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	nls = sbi->nls;
 	sbi->nls = load_nls("utf8");
 	if (!sbi->nls) {
-		printk(KERN_ERR "hfs: unable to load nls for utf8\n");
+		printk(KERN_ERR "hfsplus: unable to load nls for utf8\n");
 		goto out_unload_nls;
 	}
 
 	/* Grab the volume header */
 	if (hfsplus_read_wrapper(sb)) {
 		if (!silent)
-			printk(KERN_WARNING "hfs: unable to find HFS+ superblock\n");
+			printk(KERN_WARNING "hfsplus: unable to find HFS+ superblock\n");
 		goto out_unload_nls;
 	}
 	vhdr = sbi->s_vhdr;

@@ -412,7 +412,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	sb->s_magic = HFSPLUS_VOLHEAD_SIG;
 	if (be16_to_cpu(vhdr->version) < HFSPLUS_MIN_VERSION ||
 	    be16_to_cpu(vhdr->version) > HFSPLUS_CURRENT_VERSION) {
-		printk(KERN_ERR "hfs: wrong filesystem version\n");
+		printk(KERN_ERR "hfsplus: wrong filesystem version\n");
 		goto out_free_vhdr;
 	}
 	sbi->total_blocks = be32_to_cpu(vhdr->total_blocks);

@@ -436,7 +436,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 
 	if ((last_fs_block > (sector_t)(~0ULL) >> (sbi->alloc_blksz_shift - 9)) ||
 	    (last_fs_page > (pgoff_t)(~0ULL))) {
-		printk(KERN_ERR "hfs: filesystem size too large.\n");
+		printk(KERN_ERR "hfsplus: filesystem size too large.\n");
 		goto out_free_vhdr;
 	}
 

@@ -445,7 +445,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	sb->s_maxbytes = MAX_LFS_FILESIZE;
 
 	if (!(vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_UNMNT))) {
-		printk(KERN_WARNING "hfs: Filesystem was "
+		printk(KERN_WARNING "hfsplus: Filesystem was "
 				"not cleanly unmounted, "
 				"running fsck.hfsplus is recommended.  "
 				"mounting read-only.\n");

@@ -453,11 +453,11 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	} else if (test_and_clear_bit(HFSPLUS_SB_FORCE, &sbi->flags)) {
 		/* nothing */
 	} else if (vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_SOFTLOCK)) {
-		printk(KERN_WARNING "hfs: Filesystem is marked locked, mounting read-only.\n");
+		printk(KERN_WARNING "hfsplus: Filesystem is marked locked, mounting read-only.\n");
 		sb->s_flags |= MS_RDONLY;
 	} else if ((vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_JOURNALED)) &&
 			!(sb->s_flags & MS_RDONLY)) {
-		printk(KERN_WARNING "hfs: write access to "
+		printk(KERN_WARNING "hfsplus: write access to "
 				"a journaled filesystem is not supported, "
 				"use the force option at your own risk, "
 				"mounting read-only.\n");

@@ -469,18 +469,18 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	/* Load metadata objects (B*Trees) */
 	sbi->ext_tree = hfs_btree_open(sb, HFSPLUS_EXT_CNID);
 	if (!sbi->ext_tree) {
-		printk(KERN_ERR "hfs: failed to load extents file\n");
+		printk(KERN_ERR "hfsplus: failed to load extents file\n");
 		goto out_free_vhdr;
 	}
 	sbi->cat_tree = hfs_btree_open(sb, HFSPLUS_CAT_CNID);
 	if (!sbi->cat_tree) {
-		printk(KERN_ERR "hfs: failed to load catalog file\n");
+		printk(KERN_ERR "hfsplus: failed to load catalog file\n");
 		goto out_close_ext_tree;
 	}
 
 	inode = hfsplus_iget(sb, HFSPLUS_ALLOC_CNID);
 	if (IS_ERR(inode)) {
-		printk(KERN_ERR "hfs: failed to load allocation file\n");
+		printk(KERN_ERR "hfsplus: failed to load allocation file\n");
 		err = PTR_ERR(inode);
 		goto out_close_cat_tree;
 	}

@@ -489,7 +489,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	/* Load the root directory */
 	root = hfsplus_iget(sb, HFSPLUS_ROOT_CNID);
 	if (IS_ERR(root)) {
-		printk(KERN_ERR "hfs: failed to load root directory\n");
+		printk(KERN_ERR "hfsplus: failed to load root directory\n");
 		err = PTR_ERR(root);
 		goto out_put_alloc_file;
 	}

fs/hfsplus/wrapper.c +3 -3

index 90effcc..2f1b39b 100644
@@ -156,7 +156,7 @@ static int hfsplus_get_last_session(struct super_block *sb,
 			*start = (sector_t)te.cdte_addr.lba << 2;
 			return 0;
 		}
-		printk(KERN_ERR "hfs: invalid session number or type of track\n");
+		printk(KERN_ERR "hfsplus: invalid session number or type of track\n");
 		return -EINVAL;
 	}
 	ms_info.addr_format = CDROM_LBA;

@@ -235,7 +235,7 @@ reread:
 	error = -EINVAL;
 	if (sbi->s_backup_vhdr->signature != sbi->s_vhdr->signature) {
 		printk(KERN_WARNING
-			"hfs: invalid secondary volume header\n");
+			"hfsplus: invalid secondary volume header\n");
 		goto out_free_backup_vhdr;
 	}
 

@@ -259,7 +259,7 @@ reread:
 		blocksize >>= 1;
 
 	if (sb_set_blocksize(sb, blocksize) != blocksize) {
-		printk(KERN_ERR "hfs: unable to set blocksize to %u!\n",
+		printk(KERN_ERR "hfsplus: unable to set blocksize to %u!\n",
 			blocksize);
 		goto out_free_backup_vhdr;
 	}

Quoted 10 lines · Mike Fleetwood

Correct the name of the hfsplus FS driver as used in printk calls. "hfs:" -> "hfsplus:".

Signed-off-by Mike Fleetwood <[email protected]>

---

Hi,

Is there a current reason why the hfsplus FS driver uses "hfs:" almost exclusively rather than "hfsplus:" as its name in printk calls?

There are as minimum two reason for leaving "hfs:" prefix in peace: (1) historical - it is like code style of "old" library; (2) the prefix "hfs:" is shorter - so, it gives opportunity to make more descriptive comments by means of one line under 80 symbols kernel code style requirement.

By the way, did you check your patch by scripts/checkpatch.pl script?

Moreover, there are hfsplus driver's patches in linux-next that uses "hfs:" prefix.

I doubt that this patch can improve hfsplus driver quality. It looks like changes in many places without changing anything in essence.

With the best regards,
Vyacheslav Dubeyko.

Quoted 569 lines

Assuming not here's a patch to fix.

(Any code which may have been copied between hfs and hfsplus has since diverged significantly).

Thanks,
Mike

---
 fs/hfsplus/bfind.c   |    2 +-
 fs/hfsplus/bnode.c   |    4 ++--
 fs/hfsplus/brec.c    |    7 ++++---
 fs/hfsplus/btree.c   |   24 ++++++++++++------------
 fs/hfsplus/catalog.c |    4 ++--
 fs/hfsplus/dir.c     |   14 +++++++-------
 fs/hfsplus/extents.c |    6 +++---
 fs/hfsplus/inode.c   |    2 +-
 fs/hfsplus/options.c |   22 +++++++++++-----------
 fs/hfsplus/super.c   |   34 +++++++++++++++++-----------------
 fs/hfsplus/wrapper.c |    6 +++---
 11 files changed, 63 insertions(+), 62 deletions(-)

fs/hfsplus/bfind.c +1 -1

index 5d799c1..b00e446 100644
@@ -137,7 +137,7 @@ int hfs_brec_find(struct hfs_find_data *fd)
 	return res;
 
 invalid:
-	printk(KERN_ERR "hfs: inconsistency in B*Tree (%d,%d,%d,%u,%u)\n",
+	printk(KERN_ERR "hfsplus: inconsistency in B*Tree (%d,%d,%d,%u,%u)\n",
 		height, bnode->height, bnode->type, nidx, parent);
 	res = -EIO;
 release:

fs/hfsplus/bnode.c +2 -2

index 1c42cc5..db020bf 100644
@@ -384,7 +384,7 @@ struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid)
 	struct hfs_bnode *node;
 
 	if (cnid >= tree->node_count) {
-		printk(KERN_ERR "hfs: request for non-existent node "
+		printk(KERN_ERR "hfsplus: request for non-existent node "
 				"%d in B*Tree\n",
 			cnid);
 		return NULL;

@@ -407,7 +407,7 @@ static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
 	loff_t off;
 
 	if (cnid >= tree->node_count) {
-		printk(KERN_ERR "hfs: request for non-existent node "
+		printk(KERN_ERR "hfsplus: request for non-existent node "
 				"%d in B*Tree\n",
 			cnid);
 		return NULL;

fs/hfsplus/brec.c +4 -3

index 2a734cf..c512fc3 100644
@@ -44,13 +44,14 @@ u16 hfs_brec_keylen(struct hfs_bnode *node, u16 rec)
 		if (!recoff)
 			return 0;
 		if (recoff > node->tree->node_size - 2) {
-			printk(KERN_ERR "hfs: recoff %d too large\n", recoff);
+			printk(KERN_ERR "hfsplus: recoff %d too large\n",
+				recoff);
 			return 0;
 		}
 
 		retval = hfs_bnode_read_u16(node, recoff) + 2;
 		if (retval > node->tree->max_key_len + 2) {
-			printk(KERN_ERR "hfs: keylen %d too large\n",
+			printk(KERN_ERR "hfsplus: keylen %d too large\n",
 				retval);
 			retval = 0;
 		}

@@ -388,7 +389,7 @@ again:
 		end_off = hfs_bnode_read_u16(parent, end_rec_off);
 		if (end_rec_off - end_off < diff) {
 
-			dprint(DBG_BNODE_MOD, "hfs: splitting index node.\n");
+			dprint(DBG_BNODE_MOD, "hfsplus: splitting index node.\n");
 			fd->bnode = parent;
 			new_node = hfs_bnode_split(fd);
 			if (IS_ERR(new_node))

fs/hfsplus/btree.c +12 -12

index 685d07d..cc13ccf 100644
@@ -41,7 +41,7 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 
 	if (!HFSPLUS_I(tree->inode)->first_blocks) {
 		printk(KERN_ERR
-		       "hfs: invalid btree extent records (0 size).\n");
+			"hfsplus: invalid btree extent records (0 size).\n");
 		goto free_inode;
 	}
 

@@ -68,12 +68,12 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 	switch (id) {
 	case HFSPLUS_EXT_CNID:
 		if (tree->max_key_len != HFSPLUS_EXT_KEYLEN - sizeof(u16)) {
-			printk(KERN_ERR "hfs: invalid extent max_key_len %d\n",
+			printk(KERN_ERR "hfsplus: invalid extent max_key_len %d\n",
 				tree->max_key_len);
 			goto fail_page;
 		}
 		if (tree->attributes & HFS_TREE_VARIDXKEYS) {
-			printk(KERN_ERR "hfs: invalid extent btree flag\n");
+			printk(KERN_ERR "hfsplus: invalid extent btree flag\n");
 			goto fail_page;
 		}
 

@@ -81,12 +81,12 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 		break;
 	case HFSPLUS_CAT_CNID:
 		if (tree->max_key_len != HFSPLUS_CAT_KEYLEN - sizeof(u16)) {
-			printk(KERN_ERR "hfs: invalid catalog max_key_len %d\n",
+			printk(KERN_ERR "hfsplus: invalid catalog max_key_len %d\n",
 				tree->max_key_len);
 			goto fail_page;
 		}
 		if (!(tree->attributes & HFS_TREE_VARIDXKEYS)) {
-			printk(KERN_ERR "hfs: invalid catalog btree flag\n");
+			printk(KERN_ERR "hfsplus: invalid catalog btree flag\n");
 			goto fail_page;
 		}
 

@@ -99,12 +99,12 @@ struct hfs_btree *hfs_btree_open(struct super_block *sb, u32 id)
 		}
 		break;
 	default:
-		printk(KERN_ERR "hfs: unknown B*Tree requested\n");
+		printk(KERN_ERR "hfsplus: unknown B*Tree requested\n");
 		goto fail_page;
 	}
 
 	if (!(tree->attributes & HFS_TREE_BIGKEYS)) {
-		printk(KERN_ERR "hfs: invalid btree flag\n");
+		printk(KERN_ERR "hfsplus: invalid btree flag\n");
 		goto fail_page;
 	}
 

@@ -147,7 +147,7 @@ void hfs_btree_close(struct hfs_btree *tree)
 		while ((node = tree->node_hash[i])) {
 			tree->node_hash[i] = node->next_hash;
 			if (atomic_read(&node->refcnt))
-				printk(KERN_CRIT "hfs: node %d:%d "
+				printk(KERN_CRIT "hfsplus: node %d:%d "
 						"still has %d user(s)!\n",
 					node->tree->cnid, node->this,
 					atomic_read(&node->refcnt));

@@ -295,7 +295,7 @@ struct hfs_bnode *hfs_bmap_alloc(struct hfs_btree *tree)
 		kunmap(*pagep);
 		nidx = node->next;
 		if (!nidx) {
-			dprint(DBG_BNODE_MOD, "hfs: create new bmap node.\n");
+			dprint(DBG_BNODE_MOD, "hfsplus: create new bmap node.\n");
 			next_node = hfs_bmap_new_bmap(node, idx);
 		} else
 			next_node = hfs_bnode_find(tree, nidx);

@@ -337,7 +337,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
 		hfs_bnode_put(node);
 		if (!i) {
 			/* panic */;
-			printk(KERN_CRIT "hfs: unable to free bnode %u. "
+			printk(KERN_CRIT "hfsplus: unable to free bnode %u. "
 					"bmap not found!\n",
 				node->this);
 			return;

@@ -347,7 +347,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
 			return;
 		if (node->type != HFS_NODE_MAP) {
 			/* panic */;
-			printk(KERN_CRIT "hfs: invalid bmap found! "
+			printk(KERN_CRIT "hfsplus: invalid bmap found! "
 					"(%u,%d)\n",
 				node->this, node->type);
 			hfs_bnode_put(node);

@@ -362,7 +362,7 @@ void hfs_bmap_free(struct hfs_bnode *node)
 	m = 1 << (~nidx & 7);
 	byte = data[off];
 	if (!(byte & m)) {
-		printk(KERN_CRIT "hfs: trying to free free bnode "
+		printk(KERN_CRIT "hfsplus: trying to free free bnode "
 				"%u(%d)\n",
 			node->this, node->type);
 		kunmap(page);

fs/hfsplus/catalog.c +2 -2

index 798d9c4..f2178e7 100644
@@ -186,12 +186,12 @@ int hfsplus_find_cat(struct super_block *sb, u32 cnid,
 
 	type = be16_to_cpu(tmp.type);
 	if (type != HFSPLUS_FOLDER_THREAD && type != HFSPLUS_FILE_THREAD) {
-		printk(KERN_ERR "hfs: found bad thread record in catalog\n");
+		printk(KERN_ERR "hfsplus: found bad thread record in catalog\n");
 		return -EIO;
 	}
 
 	if (be16_to_cpu(tmp.thread.nodeName.length) > 255) {
-		printk(KERN_ERR "hfs: catalog name length corrupted\n");
+		printk(KERN_ERR "hfsplus: catalog name length corrupted\n");
 		return -EIO;
 	}
 

fs/hfsplus/dir.c +7 -7

index 6b9f921..8f133ab 100644
@@ -102,7 +102,7 @@ again:
 		} else if (!dentry->d_fsdata)
 			dentry->d_fsdata = (void *)(unsigned long)cnid;
 	} else {
-		printk(KERN_ERR "hfs: invalid catalog entry type in lookup\n");
+		printk(KERN_ERR "hfsplus: invalid catalog entry type in lookup\n");
 		err = -EIO;
 		goto fail;
 	}

@@ -158,12 +158,12 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 		hfs_bnode_read(fd.bnode, &entry, fd.entryoffset,
 			fd.entrylength);
 		if (be16_to_cpu(entry.type) != HFSPLUS_FOLDER_THREAD) {
-			printk(KERN_ERR "hfs: bad catalog folder thread\n");
+			printk(KERN_ERR "hfsplus: bad catalog folder thread\n");
 			err = -EIO;
 			goto out;
 		}
 		if (fd.entrylength < HFSPLUS_MIN_THREAD_SZ) {
-			printk(KERN_ERR "hfs: truncated catalog thread\n");
+			printk(KERN_ERR "hfsplus: truncated catalog thread\n");
 			err = -EIO;
 			goto out;
 		}

@@ -182,7 +182,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 
 	for (;;) {
 		if (be32_to_cpu(fd.key->cat.parent) != inode->i_ino) {
-			printk(KERN_ERR "hfs: walked past end of dir\n");
+			printk(KERN_ERR "hfsplus: walked past end of dir\n");
 			err = -EIO;
 			goto out;
 		}

@@ -202,7 +202,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 		if (type == HFSPLUS_FOLDER) {
 			if (fd.entrylength <
 					sizeof(struct hfsplus_cat_folder)) {
-				printk(KERN_ERR "hfs: small dir entry\n");
+				printk(KERN_ERR "hfsplus: small dir entry\n");
 				err = -EIO;
 				goto out;
 			}

@@ -215,7 +215,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 				break;
 		} else if (type == HFSPLUS_FILE) {
 			if (fd.entrylength < sizeof(struct hfsplus_cat_file)) {
-				printk(KERN_ERR "hfs: small file entry\n");
+				printk(KERN_ERR "hfsplus: small file entry\n");
 				err = -EIO;
 				goto out;
 			}

@@ -223,7 +223,7 @@ static int hfsplus_readdir(struct file *filp, void *dirent, filldir_t filldir)
 				    be32_to_cpu(entry.file.id), DT_REG))
 				break;
 		} else {
-			printk(KERN_ERR "hfs: bad catalog entry type\n");
+			printk(KERN_ERR "hfsplus: bad catalog entry type\n");
 			err = -EIO;
 			goto out;
 		}

fs/hfsplus/extents.c +3 -3

index eba76ea..df17086 100644
@@ -348,7 +348,7 @@ found:
 		if (count <= block_nr) {
 			err = hfsplus_block_free(sb, start, count);
 			if (err) {
-				printk(KERN_ERR "hfs: can't free extent\n");
+				printk(KERN_ERR "hfsplus: can't free extent\n");
 				dprint(DBG_EXTENT, " start: %u count: %u\n",
 					start, count);
 			}

@@ -359,7 +359,7 @@ found:
 			count -= block_nr;
 			err = hfsplus_block_free(sb, start + count, block_nr);
 			if (err) {
-				printk(KERN_ERR "hfs: can't free extent\n");
+				printk(KERN_ERR "hfsplus: can't free extent\n");
 				dprint(DBG_EXTENT, " start: %u count: %u\n",
 					start, count);
 			}

@@ -432,7 +432,7 @@ int hfsplus_file_extend(struct inode *inode)
 	if (sbi->alloc_file->i_size * 8 <
 	    sbi->total_blocks - sbi->free_blocks + 8) {
 		/* extend alloc file */
-		printk(KERN_ERR "hfs: extend alloc file! "
+		printk(KERN_ERR "hfsplus: extend alloc file! "
 				"(%llu,%u,%u)\n",
 			sbi->alloc_file->i_size * 8,
 			sbi->total_blocks, sbi->free_blocks);

fs/hfsplus/inode.c +1 -1

index 799b336..324dcda 100644
@@ -559,7 +559,7 @@ int hfsplus_cat_read_inode(struct inode *inode, struct hfs_find_data *fd)
 		inode->i_ctime = hfsp_mt2ut(file->attribute_mod_date);
 		HFSPLUS_I(inode)->create_date = file->create_date;
 	} else {
-		printk(KERN_ERR "hfs: bad catalog entry used to create inode\n");
+		printk(KERN_ERR "hfsplus: bad catalog entry used to create inode\n");
 		res = -EIO;
 	}
 	return res;

fs/hfsplus/options.c +11 -11

index ed257c6..9e5264d 100644
@@ -113,67 +113,67 @@ int hfsplus_parse_options(char *input, struct hfsplus_sb_info *sbi)
 		switch (token) {
 		case opt_creator:
 			if (match_fourchar(&args[0], &sbi->creator)) {
-				printk(KERN_ERR "hfs: creator requires a 4 character value\n");
+				printk(KERN_ERR "hfsplus: creator requires a 4 character value\n");
 				return 0;
 			}
 			break;
 		case opt_type:
 			if (match_fourchar(&args[0], &sbi->type)) {
-				printk(KERN_ERR "hfs: type requires a 4 character value\n");
+				printk(KERN_ERR "hfsplus: type requires a 4 character value\n");
 				return 0;
 			}
 			break;
 		case opt_umask:
 			if (match_octal(&args[0], &tmp)) {
-				printk(KERN_ERR "hfs: umask requires a value\n");
+				printk(KERN_ERR "hfsplus: umask requires a value\n");
 				return 0;
 			}
 			sbi->umask = (umode_t)tmp;
 			break;
 		case opt_uid:
 			if (match_int(&args[0], &tmp)) {
-				printk(KERN_ERR "hfs: uid requires an argument\n");
+				printk(KERN_ERR "hfsplus: uid requires an argument\n");
 				return 0;
 			}
 			sbi->uid = make_kuid(current_user_ns(), (uid_t)tmp);
 			if (!uid_valid(sbi->uid)) {
-				printk(KERN_ERR "hfs: invalid uid specified\n");
+				printk(KERN_ERR "hfsplus: invalid uid specified\n");
 				return 0;
 			}
 			break;
 		case opt_gid:
 			if (match_int(&args[0], &tmp)) {
-				printk(KERN_ERR "hfs: gid requires an argument\n");
+				printk(KERN_ERR "hfsplus: gid requires an argument\n");
 				return 0;
 			}
 			sbi->gid = make_kgid(current_user_ns(), (gid_t)tmp);
 			if (!gid_valid(sbi->gid)) {
-				printk(KERN_ERR "hfs: invalid gid specified\n");
+				printk(KERN_ERR "hfsplus: invalid gid specified\n");
 				return 0;
 			}
 			break;
 		case opt_part:
 			if (match_int(&args[0], &sbi->part)) {
-				printk(KERN_ERR "hfs: part requires an argument\n");
+				printk(KERN_ERR "hfsplus: part requires an argument\n");
 				return 0;
 			}
 			break;
 		case opt_session:
 			if (match_int(&args[0], &sbi->session)) {
-				printk(KERN_ERR "hfs: session requires an argument\n");
+				printk(KERN_ERR "hfsplus: session requires an argument\n");
 				return 0;
 			}
 			break;
 		case opt_nls:
 			if (sbi->nls) {
-				printk(KERN_ERR "hfs: unable to change nls mapping\n");
+				printk(KERN_ERR "hfsplus: unable to change nls mapping\n");
 				return 0;
 			}
 			p = match_strdup(&args[0]);
 			if (p)
 				sbi->nls = load_nls(p);
 			if (!sbi->nls) {
-				printk(KERN_ERR "hfs: unable to load "
+				printk(KERN_ERR "hfsplus: unable to load "
 						"nls mapping \"%s\"\n",
 					p);
 				kfree(p);

fs/hfsplus/super.c +17 -17

index 796198d..31cbf46 100644
@@ -130,7 +130,7 @@ static int hfsplus_system_write_inode(struct inode *inode)
 	if (tree) {
 		int err = hfs_btree_write(tree);
 		if (err) {
-			printk(KERN_ERR "hfs: b-tree write err: %d, ino %lu\n",
+			printk(KERN_ERR "hfsplus: b-tree write err: %d, ino %lu\n",
 					err, inode->i_ino);
 			return err;
 		}

@@ -243,7 +243,7 @@ static void delayed_sync_fs(struct work_struct *work)
 
 	err = hfsplus_sync_fs(sbi->alloc_file->i_sb, 1);
 	if (err)
-		printk(KERN_ERR "hfs: delayed sync fs err %d\n", err);
+		printk(KERN_ERR "hfsplus: delayed sync fs err %d\n", err);
 }
 
 void hfsplus_mark_mdb_dirty(struct super_block *sb)

@@ -324,7 +324,7 @@ static int hfsplus_remount(struct super_block *sb, int *flags, char *data)
 			return -EINVAL;
 
 		if (!(vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_UNMNT))) {
-			printk(KERN_WARNING "hfs: filesystem was "
+			printk(KERN_WARNING "hfsplus: filesystem was "
 					"not cleanly unmounted, "
 					"running fsck.hfsplus is recommended.  "
 					"leaving read-only.\n");

@@ -334,13 +334,13 @@ static int hfsplus_remount(struct super_block *sb, int *flags, char *data)
 			/* nothing */
 		} else if (vhdr->attributes &
 				cpu_to_be32(HFSPLUS_VOL_SOFTLOCK)) {
-			printk(KERN_WARNING "hfs: filesystem is marked locked, "
+			printk(KERN_WARNING "hfsplus: filesystem is marked locked, "
 					"leaving read-only.\n");
 			sb->s_flags |= MS_RDONLY;
 			*flags |= MS_RDONLY;
 		} else if (vhdr->attributes &
 				cpu_to_be32(HFSPLUS_VOL_JOURNALED)) {
-			printk(KERN_WARNING "hfs: filesystem is "
+			printk(KERN_WARNING "hfsplus: filesystem is "
 					"marked journaled, "
 					"leaving read-only.\n");
 			sb->s_flags |= MS_RDONLY;

@@ -388,7 +388,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 
 	err = -EINVAL;
 	if (!hfsplus_parse_options(data, sbi)) {
-		printk(KERN_ERR "hfs: unable to parse mount options\n");
+		printk(KERN_ERR "hfsplus: unable to parse mount options\n");
 		goto out_unload_nls;
 	}
 

@@ -396,14 +396,14 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	nls = sbi->nls;
 	sbi->nls = load_nls("utf8");
 	if (!sbi->nls) {
-		printk(KERN_ERR "hfs: unable to load nls for utf8\n");
+		printk(KERN_ERR "hfsplus: unable to load nls for utf8\n");
 		goto out_unload_nls;
 	}
 
 	/* Grab the volume header */
 	if (hfsplus_read_wrapper(sb)) {
 		if (!silent)
-			printk(KERN_WARNING "hfs: unable to find HFS+ superblock\n");
+			printk(KERN_WARNING "hfsplus: unable to find HFS+ superblock\n");
 		goto out_unload_nls;
 	}
 	vhdr = sbi->s_vhdr;

@@ -412,7 +412,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	sb->s_magic = HFSPLUS_VOLHEAD_SIG;
 	if (be16_to_cpu(vhdr->version) < HFSPLUS_MIN_VERSION ||
 	    be16_to_cpu(vhdr->version) > HFSPLUS_CURRENT_VERSION) {
-		printk(KERN_ERR "hfs: wrong filesystem version\n");
+		printk(KERN_ERR "hfsplus: wrong filesystem version\n");
 		goto out_free_vhdr;
 	}
 	sbi->total_blocks = be32_to_cpu(vhdr->total_blocks);

@@ -436,7 +436,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 
 	if ((last_fs_block > (sector_t)(~0ULL) >> (sbi->alloc_blksz_shift - 9)) ||
 	    (last_fs_page > (pgoff_t)(~0ULL))) {
-		printk(KERN_ERR "hfs: filesystem size too large.\n");
+		printk(KERN_ERR "hfsplus: filesystem size too large.\n");
 		goto out_free_vhdr;
 	}
 

@@ -445,7 +445,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	sb->s_maxbytes = MAX_LFS_FILESIZE;
 
 	if (!(vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_UNMNT))) {
-		printk(KERN_WARNING "hfs: Filesystem was "
+		printk(KERN_WARNING "hfsplus: Filesystem was "
 				"not cleanly unmounted, "
 				"running fsck.hfsplus is recommended.  "
 				"mounting read-only.\n");

@@ -453,11 +453,11 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	} else if (test_and_clear_bit(HFSPLUS_SB_FORCE, &sbi->flags)) {
 		/* nothing */
 	} else if (vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_SOFTLOCK)) {
-		printk(KERN_WARNING "hfs: Filesystem is marked locked, mounting read-only.\n");
+		printk(KERN_WARNING "hfsplus: Filesystem is marked locked, mounting read-only.\n");
 		sb->s_flags |= MS_RDONLY;
 	} else if ((vhdr->attributes & cpu_to_be32(HFSPLUS_VOL_JOURNALED)) &&
 			!(sb->s_flags & MS_RDONLY)) {
-		printk(KERN_WARNING "hfs: write access to "
+		printk(KERN_WARNING "hfsplus: write access to "
 				"a journaled filesystem is not supported, "
 				"use the force option at your own risk, "
 				"mounting read-only.\n");

@@ -469,18 +469,18 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	/* Load metadata objects (B*Trees) */
 	sbi->ext_tree = hfs_btree_open(sb, HFSPLUS_EXT_CNID);
 	if (!sbi->ext_tree) {
-		printk(KERN_ERR "hfs: failed to load extents file\n");
+		printk(KERN_ERR "hfsplus: failed to load extents file\n");
 		goto out_free_vhdr;
 	}
 	sbi->cat_tree = hfs_btree_open(sb, HFSPLUS_CAT_CNID);
 	if (!sbi->cat_tree) {
-		printk(KERN_ERR "hfs: failed to load catalog file\n");
+		printk(KERN_ERR "hfsplus: failed to load catalog file\n");
 		goto out_close_ext_tree;
 	}
 
 	inode = hfsplus_iget(sb, HFSPLUS_ALLOC_CNID);
 	if (IS_ERR(inode)) {
-		printk(KERN_ERR "hfs: failed to load allocation file\n");
+		printk(KERN_ERR "hfsplus: failed to load allocation file\n");
 		err = PTR_ERR(inode);
 		goto out_close_cat_tree;
 	}

@@ -489,7 +489,7 @@ static int hfsplus_fill_super(struct super_block *sb, void *data, int silent)
 	/* Load the root directory */
 	root = hfsplus_iget(sb, HFSPLUS_ROOT_CNID);
 	if (IS_ERR(root)) {
-		printk(KERN_ERR "hfs: failed to load root directory\n");
+		printk(KERN_ERR "hfsplus: failed to load root directory\n");
 		err = PTR_ERR(root);
 		goto out_put_alloc_file;
 	}

fs/hfsplus/wrapper.c +2 -2

index 90effcc..2f1b39b 100644
@@ -156,7 +156,7 @@ static int hfsplus_get_last_session(struct super_block *sb,
 			*start = (sector_t)te.cdte_addr.lba << 2;
 			return 0;
 		}
-		printk(KERN_ERR "hfs: invalid session number or type of track\n");
+		printk(KERN_ERR "hfsplus: invalid session number or type of track\n");
 		return -EINVAL;
 	}
 	ms_info.addr_format = CDROM_LBA;

@@ -235,7 +235,7 @@ reread:
 	error = -EINVAL;
 	if (sbi->s_backup_vhdr->signature != sbi->s_vhdr->signature) {
 		printk(KERN_WARNING
-			"hfs: invalid secondary volume header\n");
+			"hfsplus: invalid secondary volume header\n");
 		goto out_free_backup_vhdr;
 	}
 

fs/hfsplus/wrapper.c reread: · line 265

Quoted 8 lines

fs/hfsplus/wrapper.c +1 -1

@@ -259,7 +259,7 @@ reread:
 		blocksize >>= 1;
 
 	if (sb_set_blocksize(sb, blocksize) != blocksize) {
-		printk(KERN_ERR "hfs: unable to set blocksize to %u!\n",
+		printk(KERN_ERR "hfsplus: unable to set blocksize to %u!\n",
 			blocksize);
 		goto out_free_backup_vhdr;
 	}

--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Quoted 20 lines · Vyacheslav Dubeyko

Quoted 10 lines · Mike Fleetwood

Correct the name of the hfsplus FS driver as used in printk calls. "hfs:" -> "hfsplus:".

Signed-off-by Mike Fleetwood <[email protected]>

---

Hi,

Is there a current reason why the hfsplus FS driver uses "hfs:" almost exclusively rather than "hfsplus:" as its name in printk calls?

There are as minimum two reason for leaving "hfs:" prefix in peace: (1) historical - it is like code style of "old" library; (2) the prefix "hfs:" is shorter - so, it gives opportunity to make more descriptive comments by means of one line under 80 symbols kernel code style requirement.

By the way, did you check your patch by scripts/checkpatch.pl script?

Quoted 9 lines

Moreover, there are hfsplus driver's patches in linux-next that uses "hfs:" prefix.

I doubt that this patch can improve hfsplus driver quality. It looks like changes in many places without changing anything in essence.

With the best regards,
Vyacheslav Dubeyko.

In terms of line length I was applying the exception in the CodingStyle which says "never break user-visible strings such as printk messages, because that breaks the ability to grep for them." to allow lines be longer than 80 characters.

I did use checkpatch.pl. It reported this for every printk:
WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ...

After seeing that ext2/3/4, btrfs and xfs use printk and not any of those functions I followed the majority. It also reported a couple of:
WARNING: line over 80 characters
I was applying the above exception.

Absoutely this patch doesn't fix any faults and is not for 3.8.0-rc* but for linux next. I just though that it would be useful for users to be told the name of the FS driver generating the message rather than a different one.

Would an equlivant patch be accepted for linux-next? How do I send it to linux-next?

Thank you Vyacheslav for reviewing my patch,
Mike

Quoted 9 lines

Quoted 9 lines

Assuming not here's a patch to fix.

(Any code which may have been copied between hfs and hfsplus has since diverged significantly).

Thanks,
Mike

Quoted 1 line · Mike Fleetwood

On Tue, Jan 29, 2013 at 09:49:25AM +0400, Vyacheslav

Quoted 1 line · Dubeyko

Quoted 1 line

On Mon, 2013-01-28 at 20:23 +0000, Mike Fleetwood

Quoted 1 line · wrote:

Quoted 1 line

Quoted 1 line

Correct the name of the hfsplus FS driver as

used in printk calls.

Quoted 8 lines

Quoted 8 lines

Quoted 8 lines

"hfs:" -> "hfsplus:".

Signed-off-by Mike Fleetwood <[email protected]>

---

Hi,

Is there a current reason why the hfsplus FS

driver uses "hfs:" almost

Quoted 1 line

Quoted 1 line

Quoted 1 line

exclusively rather than "hfsplus:" as its

name in printk calls?

Quoted 1 line

Quoted 1 line

Quoted 1 line

Quoted 1 line

There are as minimum two reason for leaving "hfs:"

prefix in peace: (1)

Quoted 1 line

Quoted 1 line

historical - it is like code style of "old"

library; (2) the prefix

Quoted 1 line

Quoted 1 line

"hfs:" is shorter - so, it gives opportunity to

make more descriptive

Quoted 1 line

Quoted 1 line

comments by means of one line under 80 symbols

kernel code style

Quoted 1 line

Quoted 1 line

requirement.

Quoted 1 line

Quoted 1 line

By the way, did you check your patch by

scripts/checkpatch.pl script?

Quoted 2 lines

Quoted 2 lines

Moreover, there are hfsplus driver's patches in

linux-next that uses

Quoted 3 lines

Quoted 3 lines

"hfs:" prefix.

I doubt that this patch can improve hfsplus driver

quality. It looks

Quoted 1 line

Quoted 1 line

like changes in many places without changing

anything in essence.

Quoted 6 lines

Quoted 4 lines

With the best regards,
Vyacheslav Dubeyko.

In terms of line length I was applying the exception in

the CodingStyle

Quoted 1 line

which says "never break user-visible strings such as

printk messages,

Quoted 1 line

because that breaks the ability to grep for them." to

allow lines be

Quoted 3 lines

longer than 80 characters.

I did use checkpatch.pl.  It reported this for

every printk:

Quoted 1 line

     WARNING: Prefer

netdev_err(netdev, ... then dev_err(dev, ... then
pr_err(...  to printk(KERN_ERR ...

So, if we begin to modify error messages in the hfsplus driver then,
maybe, it makes sense to exchange the printk() on pr_err() and to
prepare define for "hfsplus:" prefix? Then, we make checkpatch.pl
completely happy.

Quoted 1 line

After seeing that ext2/3/4, btrfs and xfs use printk

and not any of

Quoted 1 line

those functions I followed the majority.  It also

reported a couple of:

Quoted 1 line

     WARNING: line over 80

characters

Quoted 3 lines

I was applying the above exception.

Absoutely this patch doesn't fix any faults and is not

for 3.8.0-rc* but

Quoted 1 line

for linux next.  I just though that it would be

useful for users to be

Quoted 1 line

told the name of the FS driver generating the message

rather than a

Quoted 2 lines

different one.

Anyway, I think that changing "hfs:" on "hfsplus:" is not important.

Quoted 3 lines

Would an equlivant patch be accepted for linux-next? How do I send it to linux-next?

I mean
git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
when I am talking about linux-next. So, you can prepare patches are
based on this repository.

With the best regards,
Vyacheslav Dubeyko.

Quoted 7 lines

Thank you Vyacheslav for reviewing my patch,
Mike

Quoted 4 lines

Quoted 4 lines

Assuming not here's a patch to fix.

(Any code which may have been copied between

hfs and hfsplus has since

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

diverged significantly).

Thanks,
Mike