On Sat 01-04-23 15:53:03, wuchi wrote: > It's ok because the code will be optimized by the compiler, just > try to simple the code. > > Signed-off-by: wuchi <wuchi.zero@xxxxxxxxx> Sure. Feel free to add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > > diff --git a/fs/ext4/bitmap.c b/fs/ext4/bitmap.c > index 87c1c8ae9298..cd725bebe69e 100644 > --- a/fs/ext4/bitmap.c > +++ b/fs/ext4/bitmap.c > @@ -74,10 +74,7 @@ int ext4_block_bitmap_csum_verify(struct super_block *sb, > } else > calculated &= 0xFFFF; > > - if (provided == calculated) > - return 1; > - > - return 0; > + return provided == calculated; > } > > void ext4_block_bitmap_csum_set(struct super_block *sb, > -- > 2.20.1 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR