Message ID | 20240813120712.2592310-8-shikemeng@huaweicloud.com |
---|---|
State | Superseded |
Headers | show |
Series | Fixes and cleanups to ext4 ialloc | expand |
…
> This could be a simple cleanup as complier may handle this.
compiler?
Would you like to append parentheses to the mentioned function name?
Regards,
Markus
on 8/15/2024 7:13 PM, Markus Elfring wrote: > … >> This could be a simple cleanup as complier may handle this. > > compiler? Right. > > > Would you like to append parentheses to the mentioned function name? Sure Will improve in next version. Thanks, Kemeng > > Regards, > Markus >
diff --git a/fs/ext4/ialloc.c b/fs/ext4/ialloc.c index 0cb109037384..e700236303cd 100644 --- a/fs/ext4/ialloc.c +++ b/fs/ext4/ialloc.c @@ -87,10 +87,10 @@ static int ext4_validate_inode_bitmap(struct super_block *sb, if (EXT4_SB(sb)->s_mount_state & EXT4_FC_REPLAY) return 0; - grp = ext4_get_group_info(sb, block_group); - if (buffer_verified(bh)) return 0; + + grp = ext4_get_group_info(sb, block_group); if (!grp || EXT4_MB_GRP_IBITMAP_CORRUPT(grp)) return -EFSCORRUPTED;
Check buffer_verified in advance to avoid unneeded ext4_get_group_info. This could be a simple cleanup as complier may handle this. Signed-off-by: Kemeng Shi <shikemeng@huaweicloud.com> --- fs/ext4/ialloc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)