[PATCH] ext4: if expression format
changes instances of if ((lhs = expression)) { to the preferred coding style lhs=expression; if (lhs) { Signed-off-by: Avantika Mathur <mathur@us.ibm.com> Cc: <linux-ext4@vger.kernel.org> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
7d1c520bb5
commit
7e0289766a
|
@ -186,7 +186,8 @@ static ext4_fsblk_t ext4_ext_find_goal(struct inode *inode,
|
||||||
depth = path->p_depth;
|
depth = path->p_depth;
|
||||||
|
|
||||||
/* try to predict block placement */
|
/* try to predict block placement */
|
||||||
if ((ex = path[depth].p_ext))
|
ex = path[depth].p_ext;
|
||||||
|
if (ex)
|
||||||
return ext_pblock(ex)+(block-le32_to_cpu(ex->ee_block));
|
return ext_pblock(ex)+(block-le32_to_cpu(ex->ee_block));
|
||||||
|
|
||||||
/* it looks like index is empty;
|
/* it looks like index is empty;
|
||||||
|
@ -543,7 +544,8 @@ static int ext4_ext_insert_index(handle_t *handle, struct inode *inode,
|
||||||
struct ext4_extent_idx *ix;
|
struct ext4_extent_idx *ix;
|
||||||
int len, err;
|
int len, err;
|
||||||
|
|
||||||
if ((err = ext4_ext_get_access(handle, inode, curp)))
|
err = ext4_ext_get_access(handle, inode, curp);
|
||||||
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
|
|
||||||
BUG_ON(logical == le32_to_cpu(curp->p_idx->ei_block));
|
BUG_ON(logical == le32_to_cpu(curp->p_idx->ei_block));
|
||||||
|
@ -665,7 +667,8 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode,
|
||||||
}
|
}
|
||||||
lock_buffer(bh);
|
lock_buffer(bh);
|
||||||
|
|
||||||
if ((err = ext4_journal_get_create_access(handle, bh)))
|
err = ext4_journal_get_create_access(handle, bh);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
|
|
||||||
neh = ext_block_hdr(bh);
|
neh = ext_block_hdr(bh);
|
||||||
|
@ -702,18 +705,21 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode,
|
||||||
set_buffer_uptodate(bh);
|
set_buffer_uptodate(bh);
|
||||||
unlock_buffer(bh);
|
unlock_buffer(bh);
|
||||||
|
|
||||||
if ((err = ext4_journal_dirty_metadata(handle, bh)))
|
err = ext4_journal_dirty_metadata(handle, bh);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
brelse(bh);
|
brelse(bh);
|
||||||
bh = NULL;
|
bh = NULL;
|
||||||
|
|
||||||
/* correct old leaf */
|
/* correct old leaf */
|
||||||
if (m) {
|
if (m) {
|
||||||
if ((err = ext4_ext_get_access(handle, inode, path + depth)))
|
err = ext4_ext_get_access(handle, inode, path + depth);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
path[depth].p_hdr->eh_entries =
|
path[depth].p_hdr->eh_entries =
|
||||||
cpu_to_le16(le16_to_cpu(path[depth].p_hdr->eh_entries)-m);
|
cpu_to_le16(le16_to_cpu(path[depth].p_hdr->eh_entries)-m);
|
||||||
if ((err = ext4_ext_dirty(handle, inode, path + depth)))
|
err = ext4_ext_dirty(handle, inode, path + depth);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@ -736,7 +742,8 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode,
|
||||||
}
|
}
|
||||||
lock_buffer(bh);
|
lock_buffer(bh);
|
||||||
|
|
||||||
if ((err = ext4_journal_get_create_access(handle, bh)))
|
err = ext4_journal_get_create_access(handle, bh);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
|
|
||||||
neh = ext_block_hdr(bh);
|
neh = ext_block_hdr(bh);
|
||||||
|
@ -780,7 +787,8 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode,
|
||||||
set_buffer_uptodate(bh);
|
set_buffer_uptodate(bh);
|
||||||
unlock_buffer(bh);
|
unlock_buffer(bh);
|
||||||
|
|
||||||
if ((err = ext4_journal_dirty_metadata(handle, bh)))
|
err = ext4_journal_dirty_metadata(handle, bh);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
brelse(bh);
|
brelse(bh);
|
||||||
bh = NULL;
|
bh = NULL;
|
||||||
|
@ -854,7 +862,8 @@ static int ext4_ext_grow_indepth(handle_t *handle, struct inode *inode,
|
||||||
}
|
}
|
||||||
lock_buffer(bh);
|
lock_buffer(bh);
|
||||||
|
|
||||||
if ((err = ext4_journal_get_create_access(handle, bh))) {
|
err = ext4_journal_get_create_access(handle, bh);
|
||||||
|
if (err) {
|
||||||
unlock_buffer(bh);
|
unlock_buffer(bh);
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
@ -874,11 +883,13 @@ static int ext4_ext_grow_indepth(handle_t *handle, struct inode *inode,
|
||||||
set_buffer_uptodate(bh);
|
set_buffer_uptodate(bh);
|
||||||
unlock_buffer(bh);
|
unlock_buffer(bh);
|
||||||
|
|
||||||
if ((err = ext4_journal_dirty_metadata(handle, bh)))
|
err = ext4_journal_dirty_metadata(handle, bh);
|
||||||
|
if (err)
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
/* create index in new top-level index: num,max,pointer */
|
/* create index in new top-level index: num,max,pointer */
|
||||||
if ((err = ext4_ext_get_access(handle, inode, curp)))
|
err = ext4_ext_get_access(handle, inode, curp);
|
||||||
|
if (err)
|
||||||
goto out;
|
goto out;
|
||||||
|
|
||||||
curp->p_hdr->eh_magic = EXT4_EXT_MAGIC;
|
curp->p_hdr->eh_magic = EXT4_EXT_MAGIC;
|
||||||
|
@ -1070,20 +1081,24 @@ int ext4_ext_correct_indexes(handle_t *handle, struct inode *inode,
|
||||||
*/
|
*/
|
||||||
k = depth - 1;
|
k = depth - 1;
|
||||||
border = path[depth].p_ext->ee_block;
|
border = path[depth].p_ext->ee_block;
|
||||||
if ((err = ext4_ext_get_access(handle, inode, path + k)))
|
err = ext4_ext_get_access(handle, inode, path + k);
|
||||||
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
path[k].p_idx->ei_block = border;
|
path[k].p_idx->ei_block = border;
|
||||||
if ((err = ext4_ext_dirty(handle, inode, path + k)))
|
err = ext4_ext_dirty(handle, inode, path + k);
|
||||||
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
|
|
||||||
while (k--) {
|
while (k--) {
|
||||||
/* change all left-side indexes */
|
/* change all left-side indexes */
|
||||||
if (path[k+1].p_idx != EXT_FIRST_INDEX(path[k+1].p_hdr))
|
if (path[k+1].p_idx != EXT_FIRST_INDEX(path[k+1].p_hdr))
|
||||||
break;
|
break;
|
||||||
if ((err = ext4_ext_get_access(handle, inode, path + k)))
|
err = ext4_ext_get_access(handle, inode, path + k);
|
||||||
|
if (err)
|
||||||
break;
|
break;
|
||||||
path[k].p_idx->ei_block = border;
|
path[k].p_idx->ei_block = border;
|
||||||
if ((err = ext4_ext_dirty(handle, inode, path + k)))
|
err = ext4_ext_dirty(handle, inode, path + k);
|
||||||
|
if (err)
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1142,7 +1157,8 @@ int ext4_ext_insert_extent(handle_t *handle, struct inode *inode,
|
||||||
le16_to_cpu(newext->ee_len),
|
le16_to_cpu(newext->ee_len),
|
||||||
le32_to_cpu(ex->ee_block),
|
le32_to_cpu(ex->ee_block),
|
||||||
le16_to_cpu(ex->ee_len), ext_pblock(ex));
|
le16_to_cpu(ex->ee_len), ext_pblock(ex));
|
||||||
if ((err = ext4_ext_get_access(handle, inode, path + depth)))
|
err = ext4_ext_get_access(handle, inode, path + depth);
|
||||||
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
ex->ee_len = cpu_to_le16(le16_to_cpu(ex->ee_len)
|
ex->ee_len = cpu_to_le16(le16_to_cpu(ex->ee_len)
|
||||||
+ le16_to_cpu(newext->ee_len));
|
+ le16_to_cpu(newext->ee_len));
|
||||||
|
@ -1192,7 +1208,8 @@ int ext4_ext_insert_extent(handle_t *handle, struct inode *inode,
|
||||||
has_space:
|
has_space:
|
||||||
nearex = path[depth].p_ext;
|
nearex = path[depth].p_ext;
|
||||||
|
|
||||||
if ((err = ext4_ext_get_access(handle, inode, path + depth)))
|
err = ext4_ext_get_access(handle, inode, path + depth);
|
||||||
|
if (err)
|
||||||
goto cleanup;
|
goto cleanup;
|
||||||
|
|
||||||
if (!nearex) {
|
if (!nearex) {
|
||||||
|
@ -1486,10 +1503,12 @@ int ext4_ext_rm_idx(handle_t *handle, struct inode *inode,
|
||||||
path--;
|
path--;
|
||||||
leaf = idx_pblock(path->p_idx);
|
leaf = idx_pblock(path->p_idx);
|
||||||
BUG_ON(path->p_hdr->eh_entries == 0);
|
BUG_ON(path->p_hdr->eh_entries == 0);
|
||||||
if ((err = ext4_ext_get_access(handle, inode, path)))
|
err = ext4_ext_get_access(handle, inode, path);
|
||||||
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
path->p_hdr->eh_entries = cpu_to_le16(le16_to_cpu(path->p_hdr->eh_entries)-1);
|
path->p_hdr->eh_entries = cpu_to_le16(le16_to_cpu(path->p_hdr->eh_entries)-1);
|
||||||
if ((err = ext4_ext_dirty(handle, inode, path)))
|
err = ext4_ext_dirty(handle, inode, path);
|
||||||
|
if (err)
|
||||||
return err;
|
return err;
|
||||||
ext_debug("index is empty, remove it, free block %llu\n", leaf);
|
ext_debug("index is empty, remove it, free block %llu\n", leaf);
|
||||||
bh = sb_find_get_block(inode->i_sb, leaf);
|
bh = sb_find_get_block(inode->i_sb, leaf);
|
||||||
|
@ -1930,7 +1949,8 @@ int ext4_ext_get_blocks(handle_t *handle, struct inode *inode,
|
||||||
mutex_lock(&EXT4_I(inode)->truncate_mutex);
|
mutex_lock(&EXT4_I(inode)->truncate_mutex);
|
||||||
|
|
||||||
/* check in cache */
|
/* check in cache */
|
||||||
if ((goal = ext4_ext_in_cache(inode, iblock, &newex))) {
|
goal = ext4_ext_in_cache(inode, iblock, &newex);
|
||||||
|
if (goal) {
|
||||||
if (goal == EXT4_EXT_CACHE_GAP) {
|
if (goal == EXT4_EXT_CACHE_GAP) {
|
||||||
if (!create) {
|
if (!create) {
|
||||||
/* block isn't allocated yet and
|
/* block isn't allocated yet and
|
||||||
|
@ -1969,7 +1989,8 @@ int ext4_ext_get_blocks(handle_t *handle, struct inode *inode,
|
||||||
*/
|
*/
|
||||||
BUG_ON(path[depth].p_ext == NULL && depth != 0);
|
BUG_ON(path[depth].p_ext == NULL && depth != 0);
|
||||||
|
|
||||||
if ((ex = path[depth].p_ext)) {
|
ex = path[depth].p_ext;
|
||||||
|
if (ex) {
|
||||||
unsigned long ee_block = le32_to_cpu(ex->ee_block);
|
unsigned long ee_block = le32_to_cpu(ex->ee_block);
|
||||||
ext4_fsblk_t ee_start = ext_pblock(ex);
|
ext4_fsblk_t ee_start = ext_pblock(ex);
|
||||||
unsigned short ee_len = le16_to_cpu(ex->ee_len);
|
unsigned short ee_len = le16_to_cpu(ex->ee_len);
|
||||||
|
|
Loading…
Reference in New Issue