Re: [PATCH 02/13] kernfs: remove kernfs_create_file and kernfs_create_file_ns

From: Christian Brauner
Date: Mon Sep 13 2021 - 09:24:56 EST


On Mon, Sep 13, 2021 at 07:41:10AM +0200, Christoph Hellwig wrote:
> All callers actually use __kernfs_create_file.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> ---

Acked-by: Christian Brauner <christian.brauner@xxxxxxxxxx>

> include/linux/kernfs.h | 24 ------------------------
> 1 file changed, 24 deletions(-)
>
> diff --git a/include/linux/kernfs.h b/include/linux/kernfs.h
> index 1093abf7c28cc..cecfeedb7361d 100644
> --- a/include/linux/kernfs.h
> +++ b/include/linux/kernfs.h
> @@ -568,30 +568,6 @@ kernfs_create_dir(struct kernfs_node *parent, const char *name, umode_t mode,
> priv, NULL);
> }
>
> -static inline struct kernfs_node *
> -kernfs_create_file_ns(struct kernfs_node *parent, const char *name,
> - umode_t mode, kuid_t uid, kgid_t gid,
> - loff_t size, const struct kernfs_ops *ops,
> - void *priv, const void *ns)
> -{
> - struct lock_class_key *key = NULL;
> -
> -#ifdef CONFIG_DEBUG_LOCK_ALLOC
> - key = (struct lock_class_key *)&ops->lockdep_key;
> -#endif
> - return __kernfs_create_file(parent, name, mode, uid, gid,
> - size, ops, priv, ns, key);
> -}
> -
> -static inline struct kernfs_node *
> -kernfs_create_file(struct kernfs_node *parent, const char *name, umode_t mode,
> - loff_t size, const struct kernfs_ops *ops, void *priv)
> -{
> - return kernfs_create_file_ns(parent, name, mode,
> - GLOBAL_ROOT_UID, GLOBAL_ROOT_GID,
> - size, ops, priv, NULL);
> -}
> -
> static inline int kernfs_remove_by_name(struct kernfs_node *parent,
> const char *name)
> {
> --
> 2.30.2
>