diff options
Diffstat (limited to 'fs/efs/inode.c')
| -rw-r--r-- | fs/efs/inode.c | 51 | 
1 files changed, 24 insertions, 27 deletions
diff --git a/fs/efs/inode.c b/fs/efs/inode.c index a8e7797b947..079d20306ee 100644 --- a/fs/efs/inode.c +++ b/fs/efs/inode.c @@ -23,7 +23,6 @@ static sector_t _efs_bmap(struct address_space *mapping, sector_t block)  }  static const struct address_space_operations efs_aops = {  	.readpage = efs_readpage, -	.sync_page = block_sync_page,  	.bmap = _efs_bmap  }; @@ -58,7 +57,7 @@ struct inode *efs_iget(struct super_block *super, unsigned long ino)  	struct inode *inode;  	inode = iget_locked(super, ino); -	if (IS_ERR(inode)) +	if (!inode)  		return ERR_PTR(-ENOMEM);  	if (!(inode->i_state & I_NEW))  		return inode; @@ -90,16 +89,16 @@ struct inode *efs_iget(struct super_block *super, unsigned long ino)  	bh = sb_bread(inode->i_sb, block);  	if (!bh) { -		printk(KERN_WARNING "EFS: bread() failed at block %d\n", block); +		pr_warn("%s() failed at block %d\n", __func__, block);  		goto read_inode_error;  	}  	efs_inode = (struct efs_dinode *) (bh->b_data + offset);  	inode->i_mode  = be16_to_cpu(efs_inode->di_mode); -	inode->i_nlink = be16_to_cpu(efs_inode->di_nlink); -	inode->i_uid   = (uid_t)be16_to_cpu(efs_inode->di_uid); -	inode->i_gid   = (gid_t)be16_to_cpu(efs_inode->di_gid); +	set_nlink(inode, be16_to_cpu(efs_inode->di_nlink)); +	i_uid_write(inode, (uid_t)be16_to_cpu(efs_inode->di_uid)); +	i_gid_write(inode, (gid_t)be16_to_cpu(efs_inode->di_gid));  	inode->i_size  = be32_to_cpu(efs_inode->di_size);  	inode->i_atime.tv_sec = be32_to_cpu(efs_inode->di_atime);  	inode->i_mtime.tv_sec = be32_to_cpu(efs_inode->di_mtime); @@ -131,19 +130,16 @@ struct inode *efs_iget(struct super_block *super, unsigned long ino)  	for(i = 0; i < EFS_DIRECTEXTENTS; i++) {  		extent_copy(&(efs_inode->di_u.di_extents[i]), &(in->extents[i]));  		if (i < in->numextents && in->extents[i].cooked.ex_magic != 0) { -			printk(KERN_WARNING "EFS: extent %d has bad magic number in inode %lu\n", i, inode->i_ino); +			pr_warn("extent %d has bad magic number in inode %lu\n", +				i, inode->i_ino);  			brelse(bh);  			goto read_inode_error;  		}  	}  	brelse(bh); -    -#ifdef DEBUG -	printk(KERN_DEBUG "EFS: efs_iget(): inode %lu, extents %d, mode %o\n", -		inode->i_ino, in->numextents, inode->i_mode); -#endif - +	pr_debug("efs_iget(): inode %lu, extents %d, mode %o\n", +		 inode->i_ino, in->numextents, inode->i_mode);  	switch (inode->i_mode & S_IFMT) {  		case S_IFDIR:   			inode->i_op = &efs_dir_inode_operations;  @@ -163,7 +159,7 @@ struct inode *efs_iget(struct super_block *super, unsigned long ino)  			init_special_inode(inode, inode->i_mode, device);  			break;  		default: -			printk(KERN_WARNING "EFS: unsupported inode mode %o\n", inode->i_mode); +			pr_warn("unsupported inode mode %o\n", inode->i_mode);  			goto read_inode_error;  			break;  	} @@ -172,7 +168,7 @@ struct inode *efs_iget(struct super_block *super, unsigned long ino)  	return inode;  read_inode_error: -	printk(KERN_WARNING "EFS: failed to read inode %lu\n", inode->i_ino); +	pr_warn("failed to read inode %lu\n", inode->i_ino);  	iget_failed(inode);  	return ERR_PTR(-EIO);  } @@ -217,7 +213,7 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) {  		/* if we only have one extent then nothing can be found */  		if (in->numextents == 1) { -			printk(KERN_ERR "EFS: map_block() failed to map (1 extent)\n"); +			pr_err("%s() failed to map (1 extent)\n", __func__);  			return 0;  		} @@ -235,13 +231,12 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) {  			}  		} -		printk(KERN_ERR "EFS: map_block() failed to map block %u (dir)\n", block); +		pr_err("%s() failed to map block %u (dir)\n", __func__, block);  		return 0;  	} -#ifdef DEBUG -	printk(KERN_DEBUG "EFS: map_block(): indirect search for logical block %u\n", block); -#endif +	pr_debug("%s(): indirect search for logical block %u\n", +		 __func__, block);  	direxts = in->extents[0].cooked.ex_offset;  	indexts = in->numextents; @@ -263,7 +258,8 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) {  		if (dirext == direxts) {  			/* should never happen */ -			printk(KERN_ERR "EFS: couldn't find direct extent for indirect extent %d (block %u)\n", cur, block); +			pr_err("couldn't find direct extent for indirect extent %d (block %u)\n", +			       cur, block);  			if (bh) brelse(bh);  			return 0;  		} @@ -280,12 +276,12 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) {  			bh = sb_bread(inode->i_sb, iblock);  			if (!bh) { -				printk(KERN_ERR "EFS: bread() failed at block %d\n", iblock); +				pr_err("%s() failed at block %d\n", +				       __func__, iblock);  				return 0;  			} -#ifdef DEBUG -			printk(KERN_DEBUG "EFS: map_block(): read indirect extent block %d\n", iblock); -#endif +			pr_debug("%s(): read indirect extent block %d\n", +				 __func__, iblock);  			first = 0;  			lastblock = iblock;  		} @@ -295,7 +291,8 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) {  		extent_copy(&(exts[ioffset]), &ext);  		if (ext.cooked.ex_magic != 0) { -			printk(KERN_ERR "EFS: extent %d has bad magic number in block %d\n", cur, iblock); +			pr_err("extent %d has bad magic number in block %d\n", +			       cur, iblock);  			if (bh) brelse(bh);  			return 0;  		} @@ -307,7 +304,7 @@ efs_block_t efs_map_block(struct inode *inode, efs_block_t block) {  		}  	}  	if (bh) brelse(bh); -	printk(KERN_ERR "EFS: map_block() failed to map block %u (indir)\n", block); +	pr_err("%s() failed to map block %u (indir)\n", __func__, block);  	return 0;  }    | 
