Commit cfe82c85 authored by Theodore Ts'o's avatar Theodore Ts'o
Browse files

ext4: remove ext4_new_meta_block() 



There were only two one callers of the function ext4_new_meta_block(),
which just a very simpler wrapper function around
ext4_new_meta_blocks().  Change those two functions to call
ext4_new_meta_blocks() directly, to save code and stack space usage.

Signed-off-by: default avatar"Theodore Ts'o" <tytso@mit.edu>
parent 815a1130

fs/ext4/balloc.c

+0 −17
Original line number Diff line number Diff line
@@ -724,23 +724,6 @@ ext4_fsblk_t ext4_new_meta_blocks(handle_t *handle, struct inode *inode, 
	return ret;

}



/*

 * ext4_new_meta_block() -- allocate block for meta data (indexing) blocks

 *

 * @handle:             handle to this transaction

 * @inode:              file inode

 * @goal:               given target block(filesystem wide)

 * @errp:               error code

 *

 * Return allocated block number on success

 */

ext4_fsblk_t ext4_new_meta_block(handle_t *handle, struct inode *inode,

		ext4_fsblk_t goal, int *errp)

{

	unsigned long count = 1;

	return ext4_new_meta_blocks(handle, inode, goal, &count, errp);

}



/**

 * ext4_count_free_blocks() -- count filesystem free blocks

 * @sb:		superblock

fs/ext4/ext4.h

+0 −2
Original line number Diff line number Diff line
@@ -998,8 +998,6 @@ extern ext4_grpblk_t ext4_block_group_offset(struct super_block *sb, 
extern int ext4_bg_has_super(struct super_block *sb, ext4_group_t group);

extern unsigned long ext4_bg_num_gdb(struct super_block *sb,

			ext4_group_t group);

extern ext4_fsblk_t ext4_new_meta_block(handle_t *handle, struct inode *inode,

			ext4_fsblk_t goal, int *errp);

extern ext4_fsblk_t ext4_new_meta_blocks(handle_t *handle, struct inode *inode,

			ext4_fsblk_t goal, unsigned long *count, int *errp);

extern int ext4_claim_free_blocks(struct ext4_sb_info *sbi, s64 nblocks);

fs/ext4/extents.c

+2 −1
Original line number Diff line number Diff line
@@ -189,9 +189,10 @@ ext4_ext_new_meta_block(handle_t *handle, struct inode *inode, 
			struct ext4_extent *ex, int *err)

{

	ext4_fsblk_t goal, newblock;

	unsigned long count = 1;



	goal = ext4_ext_find_goal(inode, path, le32_to_cpu(ex->ee_block));

	newblock = ext4_new_meta_block(handle, inode, goal, err);

	newblock = ext4_new_meta_blocks(handle, inode, goal, &count, err);

	return newblock;

}

fs/ext4/xattr.c

+3 −2
Original line number Diff line number Diff line
@@ -689,6 +689,7 @@ ext4_xattr_block_set(handle_t *handle, struct inode *inode, 
		     struct ext4_xattr_info *i,

		     struct ext4_xattr_block_find *bs)

{

	unsigned long count = 1;

	struct super_block *sb = inode->i_sb;

	struct buffer_head *new_bh = NULL;

	struct ext4_xattr_search *s = &bs->s;
@@ -810,8 +811,8 @@ inserted: 
			/* We need to allocate a new block */

			ext4_fsblk_t goal = ext4_group_first_block_no(sb,

						EXT4_I(inode)->i_block_group);

			ext4_fsblk_t block = ext4_new_meta_block(handle, inode,

							goal, &error);

			ext4_fsblk_t block = ext4_new_meta_blocks(handle, inode,

						  goal, &count, &error);

			if (error)

				goto cleanup;

			ea_idebug(inode, "creating block %d", block);

CRA Git | Maintained and supported by SUSTech CRA and CCSE