[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Z-10VGi6E8g3gi4Q@gallifrey>
Date: Wed, 2 Apr 2025 17:31:00 +0000
From: "Dr. David Alan Gilbert" <linux@...blig.org>
To: trondmy@...nel.org, anna@...nel.org, linux-nfs@...r.kernel.org
Cc: linux-kernel@...r.kernel.org
Subject: Re: [PATCH] NFS: Remove unused function nfs_umount
* linux@...blig.org (linux@...blig.org) wrote:
> From: "Dr. David Alan Gilbert" <linux@...blig.org>
>
> nfs_umount() has been unused since 2013's
> commit 4580a92d44e2 ("NFS: Use server-recommended security flavor by
> default (NFSv3)")
>
> Remove it.
>
> Signed-off-by: Dr. David Alan Gilbert <linux@...blig.org>
Hi,
I'd appreciate a review on this one please.
Dave (just going cleaning up his old patch list)
> ---
> fs/nfs/internal.h | 1 -
> fs/nfs/mount_clnt.c | 68 ---------------------------------------------
> 2 files changed, 69 deletions(-)
>
> diff --git a/fs/nfs/internal.h b/fs/nfs/internal.h
> index fae2c7ae4acc..36dcbb928c5f 100644
> --- a/fs/nfs/internal.h
> +++ b/fs/nfs/internal.h
> @@ -207,7 +207,6 @@ struct nfs_mount_request {
> };
>
> extern int nfs_mount(struct nfs_mount_request *info, int timeo, int retrans);
> -extern void nfs_umount(const struct nfs_mount_request *info);
>
> /* client.c */
> extern const struct rpc_program nfs_program;
> diff --git a/fs/nfs/mount_clnt.c b/fs/nfs/mount_clnt.c
> index 57c9dd700b58..db8dfb920394 100644
> --- a/fs/nfs/mount_clnt.c
> +++ b/fs/nfs/mount_clnt.c
> @@ -223,74 +223,6 @@ int nfs_mount(struct nfs_mount_request *info, int timeo, int retrans)
> goto out;
> }
>
> -/**
> - * nfs_umount - Notify a server that we have unmounted this export
> - * @info: pointer to umount request arguments
> - *
> - * MOUNTPROC_UMNT is advisory, so we set a short timeout, and always
> - * use UDP.
> - */
> -void nfs_umount(const struct nfs_mount_request *info)
> -{
> - static const struct rpc_timeout nfs_umnt_timeout = {
> - .to_initval = 1 * HZ,
> - .to_maxval = 3 * HZ,
> - .to_retries = 2,
> - };
> - struct rpc_create_args args = {
> - .net = info->net,
> - .protocol = IPPROTO_UDP,
> - .address = (struct sockaddr *)info->sap,
> - .addrsize = info->salen,
> - .timeout = &nfs_umnt_timeout,
> - .servername = info->hostname,
> - .program = &mnt_program,
> - .version = info->version,
> - .authflavor = RPC_AUTH_UNIX,
> - .flags = RPC_CLNT_CREATE_NOPING,
> - .cred = current_cred(),
> - };
> - struct rpc_message msg = {
> - .rpc_argp = info->dirpath,
> - };
> - struct rpc_clnt *clnt;
> - int status;
> -
> - if (strlen(info->dirpath) > MNTPATHLEN)
> - return;
> -
> - if (info->noresvport)
> - args.flags |= RPC_CLNT_CREATE_NONPRIVPORT;
> -
> - clnt = rpc_create(&args);
> - if (IS_ERR(clnt))
> - goto out_clnt_err;
> -
> - dprintk("NFS: sending UMNT request for %s:%s\n",
> - (info->hostname ? info->hostname : "server"), info->dirpath);
> -
> - if (info->version == NFS_MNT3_VERSION)
> - msg.rpc_proc = &clnt->cl_procinfo[MOUNTPROC3_UMNT];
> - else
> - msg.rpc_proc = &clnt->cl_procinfo[MOUNTPROC_UMNT];
> -
> - status = rpc_call_sync(clnt, &msg, 0);
> - rpc_shutdown_client(clnt);
> -
> - if (unlikely(status < 0))
> - goto out_call_err;
> -
> - return;
> -
> -out_clnt_err:
> - dprintk("NFS: failed to create UMNT RPC client, status=%ld\n",
> - PTR_ERR(clnt));
> - return;
> -
> -out_call_err:
> - dprintk("NFS: UMNT request failed, status=%d\n", status);
> -}
> -
> /*
> * XDR encode/decode functions for MOUNT
> */
> --
> 2.48.1
>
--
-----Open up your eyes, open up your mind, open up your code -------
/ Dr. David Alan Gilbert | Running GNU/Linux | Happy \
\ dave @ treblig.org | | In Hex /
\ _________________________|_____ http://www.treblig.org |_______/
Powered by blists - more mailing lists