lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20240102130441.cq2lc4itpuaskg7l@quack3> Date: Tue, 2 Jan 2024 14:04:41 +0100 From: Jan Kara <jack@...e.cz> To: Kemeng Shi <shikemeng@...weicloud.com> Cc: tytso@....edu, adilger.kernel@...ger.ca, linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 5/8] ext4: remove unused return value of ext4_mb_release On Sun 26-11-23 00:11:40, Kemeng Shi wrote: > Remove unused return value of ext4_mb_release. > > Signed-off-by: Kemeng Shi <shikemeng@...weicloud.com> Looks good. Feel free to add: Reviewed-by: Jan Kara <jack@...e.cz> Honza > --- > fs/ext4/ext4.h | 2 +- > fs/ext4/mballoc.c | 4 +--- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h > index 8ccebe0d4..fb35cae16 100644 > --- a/fs/ext4/ext4.h > +++ b/fs/ext4/ext4.h > @@ -2902,7 +2902,7 @@ extern const struct seq_operations ext4_mb_seq_groups_ops; > extern const struct seq_operations ext4_mb_seq_structs_summary_ops; > extern int ext4_seq_mb_stats_show(struct seq_file *seq, void *offset); > extern int ext4_mb_init(struct super_block *); > -extern int ext4_mb_release(struct super_block *); > +extern void ext4_mb_release(struct super_block *); > extern ext4_fsblk_t ext4_mb_new_blocks(handle_t *, > struct ext4_allocation_request *, int *); > extern void ext4_discard_preallocations(struct inode *, unsigned int); > diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c > index f79e87ad3..8d6246db3 100644 > --- a/fs/ext4/mballoc.c > +++ b/fs/ext4/mballoc.c > @@ -3725,7 +3725,7 @@ static int ext4_mb_cleanup_pa(struct ext4_group_info *grp) > return count; > } > > -int ext4_mb_release(struct super_block *sb) > +void ext4_mb_release(struct super_block *sb) > { > ext4_group_t ngroups = ext4_get_groups_count(sb); > ext4_group_t i; > @@ -3801,8 +3801,6 @@ int ext4_mb_release(struct super_block *sb) > } > > free_percpu(sbi->s_locality_groups); > - > - return 0; > } > > static inline int ext4_issue_discard(struct super_block *sb, > -- > 2.30.0 > > -- Jan Kara <jack@...e.com> SUSE Labs, CR
Powered by blists - more mailing lists