Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e6c32616 authored by Fabian Frederick's avatar Fabian Frederick Committed by Linus Torvalds
Browse files

fs/qnx6: use pr_fmt and __func__ in logging



Remove "qnx6:" and "qnx6: " from each logging instruction.

Signed-off-by: default avatarFabian Frederick <fabf@skynet.be>
Cc: Joe Perches <joe@perches.com>
Cc: Kai Bankett <chaosman@ontika.net>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent e00d5b5a
Loading
Loading
Loading
Loading
+6 −6
Original line number Original line Diff line number Diff line
@@ -77,12 +77,12 @@ static int qnx6_dir_longfilename(struct inode *inode,
	if (de->de_size != 0xff) {
	if (de->de_size != 0xff) {
		/* error - long filename entries always have size 0xff
		/* error - long filename entries always have size 0xff
		   in direntry */
		   in direntry */
		pr_err("qnx6: invalid direntry size (%i).\n", de->de_size);
		pr_err("invalid direntry size (%i).\n", de->de_size);
		return 0;
		return 0;
	}
	}
	lf = qnx6_longname(s, de, &page);
	lf = qnx6_longname(s, de, &page);
	if (IS_ERR(lf)) {
	if (IS_ERR(lf)) {
		pr_err("qnx6:Error reading longname\n");
		pr_err("Error reading longname\n");
		return 0;
		return 0;
	}
	}


@@ -90,7 +90,7 @@ static int qnx6_dir_longfilename(struct inode *inode,


	if (lf_size > QNX6_LONG_NAME_MAX) {
	if (lf_size > QNX6_LONG_NAME_MAX) {
		QNX6DEBUG((KERN_INFO "file %s\n", lf->lf_fname));
		QNX6DEBUG((KERN_INFO "file %s\n", lf->lf_fname));
		pr_err("qnx6:Filename too long (%i)\n", lf_size);
		pr_err("Filename too long (%i)\n", lf_size);
		qnx6_put_page(page);
		qnx6_put_page(page);
		return 0;
		return 0;
	}
	}
@@ -99,7 +99,7 @@ static int qnx6_dir_longfilename(struct inode *inode,
	   mmi 3g filesystem does not have that checksum */
	   mmi 3g filesystem does not have that checksum */
	if (!test_opt(s, MMI_FS) && fs32_to_cpu(sbi, de->de_checksum) !=
	if (!test_opt(s, MMI_FS) && fs32_to_cpu(sbi, de->de_checksum) !=
			qnx6_lfile_checksum(lf->lf_fname, lf_size))
			qnx6_lfile_checksum(lf->lf_fname, lf_size))
		pr_info("qnx6: long filename checksum error.\n");
		pr_info("long filename checksum error.\n");


	QNX6DEBUG((KERN_INFO "qnx6_readdir:%.*s inode:%u\n",
	QNX6DEBUG((KERN_INFO "qnx6_readdir:%.*s inode:%u\n",
					lf_size, lf->lf_fname, de_inode));
					lf_size, lf->lf_fname, de_inode));
@@ -135,7 +135,7 @@ static int qnx6_readdir(struct file *file, struct dir_context *ctx)
		int i = start;
		int i = start;


		if (IS_ERR(page)) {
		if (IS_ERR(page)) {
			pr_err("qnx6_readdir: read failed\n");
			pr_err("%s(): read failed\n", __func__);
			ctx->pos = (n + 1) << PAGE_CACHE_SHIFT;
			ctx->pos = (n + 1) << PAGE_CACHE_SHIFT;
			return PTR_ERR(page);
			return PTR_ERR(page);
		}
		}
@@ -258,7 +258,7 @@ unsigned qnx6_find_entry(int len, struct inode *dir, const char *name,
					if (ino)
					if (ino)
						goto found;
						goto found;
				} else
				} else
					pr_err("qnx6: undefined filename size in inode.\n");
					pr_err("undefined filename size in inode.\n");
			}
			}
			qnx6_put_page(page);
			qnx6_put_page(page);
		}
		}
+22 −22
Original line number Original line Diff line number Diff line
@@ -87,7 +87,7 @@ static int qnx6_get_block(struct inode *inode, sector_t iblock,
static int qnx6_check_blockptr(__fs32 ptr)
static int qnx6_check_blockptr(__fs32 ptr)
{
{
	if (ptr == ~(__fs32)0) {
	if (ptr == ~(__fs32)0) {
		pr_err("qnx6: hit unused blockpointer.\n");
		pr_err("hit unused blockpointer.\n");
		return 0;
		return 0;
	}
	}
	return 1;
	return 1;
@@ -127,7 +127,7 @@ static unsigned qnx6_block_map(struct inode *inode, unsigned no)
	levelptr = no >> bitdelta;
	levelptr = no >> bitdelta;


	if (levelptr > QNX6_NO_DIRECT_POINTERS - 1) {
	if (levelptr > QNX6_NO_DIRECT_POINTERS - 1) {
		pr_err("qnx6:Requested file block number (%u) too big.", no);
		pr_err("Requested file block number (%u) too big.", no);
		return 0;
		return 0;
	}
	}


@@ -136,7 +136,7 @@ static unsigned qnx6_block_map(struct inode *inode, unsigned no)
	for (i = 0; i < depth; i++) {
	for (i = 0; i < depth; i++) {
		bh = sb_bread(s, block);
		bh = sb_bread(s, block);
		if (!bh) {
		if (!bh) {
			pr_err("qnx6:Error reading block (%u)\n", block);
			pr_err("Error reading block (%u)\n", block);
			return 0;
			return 0;
		}
		}
		bitdelta -= ptrbits;
		bitdelta -= ptrbits;
@@ -275,7 +275,7 @@ static struct buffer_head *qnx6_check_first_superblock(struct super_block *s,
	   start with the first superblock */
	   start with the first superblock */
	bh = sb_bread(s, offset);
	bh = sb_bread(s, offset);
	if (!bh) {
	if (!bh) {
		pr_err("qnx6: unable to read the first superblock\n");
		pr_err("unable to read the first superblock\n");
		return NULL;
		return NULL;
	}
	}
	sb = (struct qnx6_super_block *)bh->b_data;
	sb = (struct qnx6_super_block *)bh->b_data;
@@ -290,9 +290,9 @@ static struct buffer_head *qnx6_check_first_superblock(struct super_block *s,
			sbi->s_bytesex = BYTESEX_LE;
			sbi->s_bytesex = BYTESEX_LE;
		if (!silent) {
		if (!silent) {
			if (offset == 0) {
			if (offset == 0) {
				pr_err("qnx6: wrong signature (magic) in superblock #1.\n");
				pr_err("wrong signature (magic) in superblock #1.\n");
			} else {
			} else {
				pr_info("qnx6: wrong signature (magic) at position (0x%lx) - will try alternative position (0x0000).\n",
				pr_info("wrong signature (magic) at position (0x%lx) - will try alternative position (0x0000).\n",
					offset * s->s_blocksize);
					offset * s->s_blocksize);
			}
			}
		}
		}
@@ -324,13 +324,13 @@ static int qnx6_fill_super(struct super_block *s, void *data, int silent)


	/* Superblock always is 512 Byte long */
	/* Superblock always is 512 Byte long */
	if (!sb_set_blocksize(s, QNX6_SUPERBLOCK_SIZE)) {
	if (!sb_set_blocksize(s, QNX6_SUPERBLOCK_SIZE)) {
		pr_err("qnx6: unable to set blocksize\n");
		pr_err("unable to set blocksize\n");
		goto outnobh;
		goto outnobh;
	}
	}


	/* parse the mount-options */
	/* parse the mount-options */
	if (!qnx6_parse_options((char *) data, s)) {
	if (!qnx6_parse_options((char *) data, s)) {
		pr_err("qnx6: invalid mount options.\n");
		pr_err("invalid mount options.\n");
		goto outnobh;
		goto outnobh;
	}
	}
	if (test_opt(s, MMI_FS)) {
	if (test_opt(s, MMI_FS)) {
@@ -350,7 +350,7 @@ static int qnx6_fill_super(struct super_block *s, void *data, int silent)
		/* try again without bootblock offset */
		/* try again without bootblock offset */
		bh1 = qnx6_check_first_superblock(s, 0, silent);
		bh1 = qnx6_check_first_superblock(s, 0, silent);
		if (!bh1) {
		if (!bh1) {
			pr_err("qnx6: unable to read the first superblock\n");
			pr_err("unable to read the first superblock\n");
			goto outnobh;
			goto outnobh;
		}
		}
		/* seems that no bootblock at partition start */
		/* seems that no bootblock at partition start */
@@ -365,13 +365,13 @@ static int qnx6_fill_super(struct super_block *s, void *data, int silent)
	/* checksum check - start at byte 8 and end at byte 512 */
	/* checksum check - start at byte 8 and end at byte 512 */
	if (fs32_to_cpu(sbi, sb1->sb_checksum) !=
	if (fs32_to_cpu(sbi, sb1->sb_checksum) !=
			crc32_be(0, (char *)(bh1->b_data + 8), 504)) {
			crc32_be(0, (char *)(bh1->b_data + 8), 504)) {
		pr_err("qnx6: superblock #1 checksum error\n");
		pr_err("superblock #1 checksum error\n");
		goto out;
		goto out;
	}
	}


	/* set new blocksize */
	/* set new blocksize */
	if (!sb_set_blocksize(s, fs32_to_cpu(sbi, sb1->sb_blocksize))) {
	if (!sb_set_blocksize(s, fs32_to_cpu(sbi, sb1->sb_blocksize))) {
		pr_err("qnx6: unable to set blocksize\n");
		pr_err("unable to set blocksize\n");
		goto out;
		goto out;
	}
	}
	/* blocksize invalidates bh - pull it back in */
	/* blocksize invalidates bh - pull it back in */
@@ -393,20 +393,20 @@ static int qnx6_fill_super(struct super_block *s, void *data, int silent)
	/* next the second superblock */
	/* next the second superblock */
	bh2 = sb_bread(s, offset);
	bh2 = sb_bread(s, offset);
	if (!bh2) {
	if (!bh2) {
		pr_err("qnx6: unable to read the second superblock\n");
		pr_err("unable to read the second superblock\n");
		goto out;
		goto out;
	}
	}
	sb2 = (struct qnx6_super_block *)bh2->b_data;
	sb2 = (struct qnx6_super_block *)bh2->b_data;
	if (fs32_to_cpu(sbi, sb2->sb_magic) != QNX6_SUPER_MAGIC) {
	if (fs32_to_cpu(sbi, sb2->sb_magic) != QNX6_SUPER_MAGIC) {
		if (!silent)
		if (!silent)
			pr_err("qnx6: wrong signature (magic) in superblock #2.\n");
			pr_err("wrong signature (magic) in superblock #2.\n");
		goto out;
		goto out;
	}
	}


	/* checksum check - start at byte 8 and end at byte 512 */
	/* checksum check - start at byte 8 and end at byte 512 */
	if (fs32_to_cpu(sbi, sb2->sb_checksum) !=
	if (fs32_to_cpu(sbi, sb2->sb_checksum) !=
				crc32_be(0, (char *)(bh2->b_data + 8), 504)) {
				crc32_be(0, (char *)(bh2->b_data + 8), 504)) {
		pr_err("qnx6: superblock #2 checksum error\n");
		pr_err("superblock #2 checksum error\n");
		goto out;
		goto out;
	}
	}


@@ -416,23 +416,23 @@ static int qnx6_fill_super(struct super_block *s, void *data, int silent)
		sbi->sb_buf = bh1;
		sbi->sb_buf = bh1;
		sbi->sb = (struct qnx6_super_block *)bh1->b_data;
		sbi->sb = (struct qnx6_super_block *)bh1->b_data;
		brelse(bh2);
		brelse(bh2);
		pr_info("qnx6: superblock #1 active\n");
		pr_info("superblock #1 active\n");
	} else {
	} else {
		/* superblock #2 active */
		/* superblock #2 active */
		sbi->sb_buf = bh2;
		sbi->sb_buf = bh2;
		sbi->sb = (struct qnx6_super_block *)bh2->b_data;
		sbi->sb = (struct qnx6_super_block *)bh2->b_data;
		brelse(bh1);
		brelse(bh1);
		pr_info("qnx6: superblock #2 active\n");
		pr_info("superblock #2 active\n");
	}
	}
mmi_success:
mmi_success:
	/* sanity check - limit maximum indirect pointer levels */
	/* sanity check - limit maximum indirect pointer levels */
	if (sb1->Inode.levels > QNX6_PTR_MAX_LEVELS) {
	if (sb1->Inode.levels > QNX6_PTR_MAX_LEVELS) {
		pr_err("qnx6: too many inode levels (max %i, sb %i)\n",
		pr_err("too many inode levels (max %i, sb %i)\n",
		       QNX6_PTR_MAX_LEVELS, sb1->Inode.levels);
		       QNX6_PTR_MAX_LEVELS, sb1->Inode.levels);
		goto out;
		goto out;
	}
	}
	if (sb1->Longfile.levels > QNX6_PTR_MAX_LEVELS) {
	if (sb1->Longfile.levels > QNX6_PTR_MAX_LEVELS) {
		pr_err("qnx6: too many longfilename levels (max %i, sb %i)\n",
		pr_err("too many longfilename levels (max %i, sb %i)\n",
		       QNX6_PTR_MAX_LEVELS, sb1->Longfile.levels);
		       QNX6_PTR_MAX_LEVELS, sb1->Longfile.levels);
		goto out;
		goto out;
	}
	}
@@ -453,7 +453,7 @@ mmi_success:
	/* prefetch root inode */
	/* prefetch root inode */
	root = qnx6_iget(s, QNX6_ROOT_INO);
	root = qnx6_iget(s, QNX6_ROOT_INO);
	if (IS_ERR(root)) {
	if (IS_ERR(root)) {
		pr_err("qnx6: get inode failed\n");
		pr_err("get inode failed\n");
		ret = PTR_ERR(root);
		ret = PTR_ERR(root);
		goto out2;
		goto out2;
	}
	}
@@ -467,7 +467,7 @@ mmi_success:
	errmsg = qnx6_checkroot(s);
	errmsg = qnx6_checkroot(s);
	if (errmsg != NULL) {
	if (errmsg != NULL) {
		if (!silent)
		if (!silent)
			pr_err("qnx6: %s\n", errmsg);
			pr_err("%s\n", errmsg);
		goto out3;
		goto out3;
	}
	}
	return 0;
	return 0;
@@ -548,7 +548,7 @@ struct inode *qnx6_iget(struct super_block *sb, unsigned ino)
	inode->i_mode = 0;
	inode->i_mode = 0;


	if (ino == 0) {
	if (ino == 0) {
		pr_err("qnx6: bad inode number on dev %s: %u is out of range\n",
		pr_err("bad inode number on dev %s: %u is out of range\n",
		       sb->s_id, ino);
		       sb->s_id, ino);
		iget_failed(inode);
		iget_failed(inode);
		return ERR_PTR(-EIO);
		return ERR_PTR(-EIO);
@@ -558,7 +558,7 @@ struct inode *qnx6_iget(struct super_block *sb, unsigned ino)
	mapping = sbi->inodes->i_mapping;
	mapping = sbi->inodes->i_mapping;
	page = read_mapping_page(mapping, n, NULL);
	page = read_mapping_page(mapping, n, NULL);
	if (IS_ERR(page)) {
	if (IS_ERR(page)) {
		pr_err("qnx6: major problem: unable to read inode from dev %s\n",
		pr_err("major problem: unable to read inode from dev %s\n",
		       sb->s_id);
		       sb->s_id);
		iget_failed(inode);
		iget_failed(inode);
		return ERR_CAST(page);
		return ERR_CAST(page);
+6 −0
Original line number Original line Diff line number Diff line
@@ -10,6 +10,12 @@
 *
 *
 */
 */


#ifdef pr_fmt
#undef pr_fmt
#endif

#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt

#include <linux/fs.h>
#include <linux/fs.h>
#include <linux/pagemap.h>
#include <linux/pagemap.h>


+10 −10
Original line number Original line Diff line number Diff line
@@ -44,14 +44,14 @@ struct qnx6_super_block *qnx6_mmi_fill_super(struct super_block *s, int silent)
	   start with the first superblock */
	   start with the first superblock */
	bh1 = sb_bread(s, 0);
	bh1 = sb_bread(s, 0);
	if (!bh1) {
	if (!bh1) {
		pr_err("qnx6: Unable to read first mmi superblock\n");
		pr_err("Unable to read first mmi superblock\n");
		return NULL;
		return NULL;
	}
	}
	sb1 = (struct qnx6_mmi_super_block *)bh1->b_data;
	sb1 = (struct qnx6_mmi_super_block *)bh1->b_data;
	sbi = QNX6_SB(s);
	sbi = QNX6_SB(s);
	if (fs32_to_cpu(sbi, sb1->sb_magic) != QNX6_SUPER_MAGIC) {
	if (fs32_to_cpu(sbi, sb1->sb_magic) != QNX6_SUPER_MAGIC) {
		if (!silent) {
		if (!silent) {
			pr_err("qnx6: wrong signature (magic) in superblock #1.\n");
			pr_err("wrong signature (magic) in superblock #1.\n");
			goto out;
			goto out;
		}
		}
	}
	}
@@ -59,7 +59,7 @@ struct qnx6_super_block *qnx6_mmi_fill_super(struct super_block *s, int silent)
	/* checksum check - start at byte 8 and end at byte 512 */
	/* checksum check - start at byte 8 and end at byte 512 */
	if (fs32_to_cpu(sbi, sb1->sb_checksum) !=
	if (fs32_to_cpu(sbi, sb1->sb_checksum) !=
				crc32_be(0, (char *)(bh1->b_data + 8), 504)) {
				crc32_be(0, (char *)(bh1->b_data + 8), 504)) {
		pr_err("qnx6: superblock #1 checksum error\n");
		pr_err("superblock #1 checksum error\n");
		goto out;
		goto out;
	}
	}


@@ -69,7 +69,7 @@ struct qnx6_super_block *qnx6_mmi_fill_super(struct super_block *s, int silent)


	/* set new blocksize */
	/* set new blocksize */
	if (!sb_set_blocksize(s, fs32_to_cpu(sbi, sb1->sb_blocksize))) {
	if (!sb_set_blocksize(s, fs32_to_cpu(sbi, sb1->sb_blocksize))) {
		pr_err("qnx6: unable to set blocksize\n");
		pr_err("unable to set blocksize\n");
		goto out;
		goto out;
	}
	}
	/* blocksize invalidates bh - pull it back in */
	/* blocksize invalidates bh - pull it back in */
@@ -82,26 +82,26 @@ struct qnx6_super_block *qnx6_mmi_fill_super(struct super_block *s, int silent)
	/* read second superblock */
	/* read second superblock */
	bh2 = sb_bread(s, offset);
	bh2 = sb_bread(s, offset);
	if (!bh2) {
	if (!bh2) {
		pr_err("qnx6: unable to read the second superblock\n");
		pr_err("unable to read the second superblock\n");
		goto out;
		goto out;
	}
	}
	sb2 = (struct qnx6_mmi_super_block *)bh2->b_data;
	sb2 = (struct qnx6_mmi_super_block *)bh2->b_data;
	if (fs32_to_cpu(sbi, sb2->sb_magic) != QNX6_SUPER_MAGIC) {
	if (fs32_to_cpu(sbi, sb2->sb_magic) != QNX6_SUPER_MAGIC) {
		if (!silent)
		if (!silent)
			pr_err("qnx6: wrong signature (magic) in superblock #2.\n");
			pr_err("wrong signature (magic) in superblock #2.\n");
		goto out;
		goto out;
	}
	}


	/* checksum check - start at byte 8 and end at byte 512 */
	/* checksum check - start at byte 8 and end at byte 512 */
	if (fs32_to_cpu(sbi, sb2->sb_checksum)
	if (fs32_to_cpu(sbi, sb2->sb_checksum)
			!= crc32_be(0, (char *)(bh2->b_data + 8), 504)) {
			!= crc32_be(0, (char *)(bh2->b_data + 8), 504)) {
		pr_err("qnx6: superblock #1 checksum error\n");
		pr_err("superblock #1 checksum error\n");
		goto out;
		goto out;
	}
	}


	qsb = kmalloc(sizeof(*qsb), GFP_KERNEL);
	qsb = kmalloc(sizeof(*qsb), GFP_KERNEL);
	if (!qsb) {
	if (!qsb) {
		pr_err("qnx6: unable to allocate memory.\n");
		pr_err("unable to allocate memory.\n");
		goto out;
		goto out;
	}
	}


@@ -117,7 +117,7 @@ struct qnx6_super_block *qnx6_mmi_fill_super(struct super_block *s, int silent)
		sbi->sb_buf = bh1;
		sbi->sb_buf = bh1;
		sbi->sb = (struct qnx6_super_block *)bh1->b_data;
		sbi->sb = (struct qnx6_super_block *)bh1->b_data;
		brelse(bh2);
		brelse(bh2);
		pr_info("qnx6: superblock #1 active\n");
		pr_info("superblock #1 active\n");
	} else {
	} else {
		/* superblock #2 active */
		/* superblock #2 active */
		qnx6_mmi_copy_sb(qsb, sb2);
		qnx6_mmi_copy_sb(qsb, sb2);
@@ -129,7 +129,7 @@ struct qnx6_super_block *qnx6_mmi_fill_super(struct super_block *s, int silent)
		sbi->sb_buf = bh2;
		sbi->sb_buf = bh2;
		sbi->sb = (struct qnx6_super_block *)bh2->b_data;
		sbi->sb = (struct qnx6_super_block *)bh2->b_data;
		brelse(bh1);
		brelse(bh1);
		pr_info("qnx6: superblock #2 active\n");
		pr_info("superblock #2 active\n");
	}
	}
	kfree(qsb);
	kfree(qsb);