linux-kernel - Re: [PATCH] vfs: inline new_inode_pseudo() and de-staticize alloc

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <vinprvobwukhgyjuurep2xdzanuqrf5wcvhzfhj7g5tpeflcsp@z3cyitdtssms>
Date: Wed, 19 Feb 2025 17:11:07 +0100
From: Jan Kara <jack@...e.cz>
To: Mateusz Guzik <mjguzik@...il.com>
Cc: brauner@...nel.org, viro@...iv.linux.org.uk, jack@...e.cz, 
	linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org
Subject: Re: [PATCH] vfs: inline new_inode_pseudo() and de-staticize
 alloc_inode()

On Wed 12-02-25 19:04:59, Mateusz Guzik wrote:
> The former is a no-op wrapper with the same argument.
> 
> I left it in place to not lose the information who needs it -- one day
> "pseudo" inodes may start differing from what alloc_inode() returns.
> 
> In the meantime no point taking a detour.
> 
> Signed-off-by: Mateusz Guzik <mjguzik@...il.com>

Looks good. Feel free to add:

Reviewed-by: Jan Kara <jack@...e.cz>

								Honza

> ---
>  fs/inode.c         | 29 ++++++++++++-----------------
>  include/linux/fs.h |  6 +++++-
>  2 files changed, 17 insertions(+), 18 deletions(-)
> 
> diff --git a/fs/inode.c b/fs/inode.c
> index 5587aabdaa5e..6e251e43bf70 100644
> --- a/fs/inode.c
> +++ b/fs/inode.c
> @@ -327,7 +327,17 @@ static void i_callback(struct rcu_head *head)
>  		free_inode_nonrcu(inode);
>  }
>  
> -static struct inode *alloc_inode(struct super_block *sb)
> +/**
> + *	alloc_inode 	- obtain an inode
> + *	@sb: superblock
> + *
> + *	Allocates a new inode for given superblock.
> + *	Inode wont be chained in superblock s_inodes list
> + *	This means :
> + *	- fs can't be unmount
> + *	- quotas, fsnotify, writeback can't work
> + */
> +struct inode *alloc_inode(struct super_block *sb)
>  {
>  	const struct super_operations *ops = sb->s_op;
>  	struct inode *inode;
> @@ -1159,21 +1169,6 @@ unsigned int get_next_ino(void)
>  }
>  EXPORT_SYMBOL(get_next_ino);
>  
> -/**
> - *	new_inode_pseudo 	- obtain an inode
> - *	@sb: superblock
> - *
> - *	Allocates a new inode for given superblock.
> - *	Inode wont be chained in superblock s_inodes list
> - *	This means :
> - *	- fs can't be unmount
> - *	- quotas, fsnotify, writeback can't work
> - */
> -struct inode *new_inode_pseudo(struct super_block *sb)
> -{
> -	return alloc_inode(sb);
> -}
> -
>  /**
>   *	new_inode 	- obtain an inode
>   *	@sb: superblock
> @@ -1190,7 +1185,7 @@ struct inode *new_inode(struct super_block *sb)
>  {
>  	struct inode *inode;
>  
> -	inode = new_inode_pseudo(sb);
> +	inode = alloc_inode(sb);
>  	if (inode)
>  		inode_sb_list_add(inode);
>  	return inode;
> diff --git a/include/linux/fs.h b/include/linux/fs.h
> index 640949116cf9..ac5d699e3aab 100644
> --- a/include/linux/fs.h
> +++ b/include/linux/fs.h
> @@ -3287,7 +3287,11 @@ static inline void __iget(struct inode *inode)
>  extern void iget_failed(struct inode *);
>  extern void clear_inode(struct inode *);
>  extern void __destroy_inode(struct inode *);
> -extern struct inode *new_inode_pseudo(struct super_block *sb);
> +struct inode *alloc_inode(struct super_block *sb);
> +static inline struct inode *new_inode_pseudo(struct super_block *sb)
> +{
> +	return alloc_inode(sb);
> +}
>  extern struct inode *new_inode(struct super_block *sb);
>  extern void free_inode_nonrcu(struct inode *inode);
>  extern int setattr_should_drop_suidgid(struct mnt_idmap *, struct inode *);
> -- 
> 2.43.0
> 
-- 
Jan Kara <jack@...e.com>
SUSE Labs, CR