On Fri 30-08-24 15:04:42, Christian Brauner wrote: > There's really no need to mention f_version. > > Signed-off-by: Christian Brauner <brauner@xxxxxxxxxx> Looks good. Feel free to add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > --- > fs/file_table.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/fs/file_table.c b/fs/file_table.c > index 3ef558f27a1c..bf1cbe47c93d 100644 > --- a/fs/file_table.c > +++ b/fs/file_table.c > @@ -159,7 +159,6 @@ static int init_file(struct file *f, int flags, const struct cred *cred) > mutex_init(&f->f_pos_lock); > f->f_flags = flags; > f->f_mode = OPEN_FMODE(flags); > - /* f->f_version: 0 */ > > /* > * We're SLAB_TYPESAFE_BY_RCU so initialize f_count last. While > > -- > 2.45.2 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR