On Wed 10-09-25 16:36:52, Christian Brauner wrote:
> Don't cargo-cult the same thing over and over.
>
> Signed-off-by: Christian Brauner <brauner@kernel.org>
Looks good. Feel free to add:
Reviewed-by: Jan Kara <jack@suse.cz>
Honza
> ---
> fs/namespace.c | 15 ++++++---------
> 1 file changed, 6 insertions(+), 9 deletions(-)
>
> diff --git a/fs/namespace.c b/fs/namespace.c
> index ddfd4457d338..14c5cdbdd6e1 100644
> --- a/fs/namespace.c
> +++ b/fs/namespace.c
> @@ -4177,18 +4177,15 @@ static struct mnt_namespace *alloc_mnt_ns(struct user_namespace *user_ns, bool a
> dec_mnt_namespaces(ucounts);
> return ERR_PTR(-ENOMEM);
> }
> - if (!anon) {
> - ret = ns_alloc_inum(&new_ns->ns);
> - if (ret) {
> - kfree(new_ns);
> - dec_mnt_namespaces(ucounts);
> - return ERR_PTR(ret);
> - }
> +
> + ret = ns_common_init(&new_ns->ns, &mntns_operations, !anon);
> + if (ret) {
> + kfree(new_ns);
> + dec_mnt_namespaces(ucounts);
> + return ERR_PTR(ret);
> }
> - new_ns->ns.ops = &mntns_operations;
> if (!anon)
> new_ns->seq = atomic64_inc_return(&mnt_ns_seq);
> - refcount_set(&new_ns->ns.count, 1);
> refcount_set(&new_ns->passive, 1);
> new_ns->mounts = RB_ROOT;
> INIT_LIST_HEAD(&new_ns->mnt_ns_list);
>
> --
> 2.47.3
>
--
Jan Kara <jack@suse.com>
SUSE Labs, CR