From: Chuck Lever III <chuck.lever@oracle.com>
To: "trondmy@kernel.org" <trondmy@kernel.org>
Cc: Bruce Fields <bfields@redhat.com>,
Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v2 10/10] nfsd: Ignore rpcbind errors on nfsd startup
Date: Sun, 19 Dec 2021 18:15:07 +0000 [thread overview]
Message-ID: <831659F6-3005-459B-92ED-933BBCEE6FE9@oracle.com> (raw)
In-Reply-To: <20211219013803.324724-11-trondmy@kernel.org>
> On Dec 18, 2021, at 8:38 PM, trondmy@kernel.org wrote:
>
> From: Trond Myklebust <trond.myklebust@hammerspace.com>
>
> NFSv4 doesn't need rpcbind, so let's not refuse to start up just because
> the rpcbind registration failed.
Commit 7e55b59b2f32 ("SUNRPC/NFSD: Support a new option for ignoring
the result of svc_register") added vs_rpcb_optnl, which is already
set for nfsd4_version4. Is that not adequate?
> Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
> ---
> fs/nfsd/nfsctl.c | 7 ++++++-
> fs/nfsd/nfsd.h | 1 +
> fs/nfsd/nfssvc.c | 18 ++++++++++++++++--
> include/linux/sunrpc/svcsock.h | 5 +++--
> net/sunrpc/svcsock.c | 14 ++++++++------
> 5 files changed, 34 insertions(+), 11 deletions(-)
>
> diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c
> index 51a49e0cfe37..da9760479acd 100644
> --- a/fs/nfsd/nfsctl.c
> +++ b/fs/nfsd/nfsctl.c
> @@ -727,6 +727,7 @@ static ssize_t __write_ports_addfd(char *buf, struct net *net, const struct cred
> char *mesg = buf;
> int fd, err;
> struct nfsd_net *nn = net_generic(net, nfsd_net_id);
> + int flags = SVC_SOCK_DEFAULTS;
>
> err = get_int(&mesg, &fd);
> if (err != 0 || fd < 0)
> @@ -741,7 +742,11 @@ static ssize_t __write_ports_addfd(char *buf, struct net *net, const struct cred
> if (err != 0)
> return err;
>
> - err = svc_addsock(nn->nfsd_serv, fd, buf, SIMPLE_TRANSACTION_LIMIT, cred);
> + if (!nfsd_rpcbind_error_is_fatal())
> + flags |= SVC_SOCK_RPCBIND_NOERR;
> +
> + err = svc_addsock(nn->nfsd_serv, fd, buf, SIMPLE_TRANSACTION_LIMIT,
> + flags, cred);
> if (err < 0) {
> nfsd_destroy(net);
> return err;
> diff --git a/fs/nfsd/nfsd.h b/fs/nfsd/nfsd.h
> index 498e5a489826..e0356d3ecf65 100644
> --- a/fs/nfsd/nfsd.h
> +++ b/fs/nfsd/nfsd.h
> @@ -134,6 +134,7 @@ int nfsd_vers(struct nfsd_net *nn, int vers, enum vers_op change);
> int nfsd_minorversion(struct nfsd_net *nn, u32 minorversion, enum vers_op change);
> void nfsd_reset_versions(struct nfsd_net *nn);
> int nfsd_create_serv(struct net *net);
> +extern bool nfsd_rpcbind_error_is_fatal(void);
>
> extern int nfsd_max_blksize;
>
> diff --git a/fs/nfsd/nfssvc.c b/fs/nfsd/nfssvc.c
> index 6815c70b06af..6f22c72f340d 100644
> --- a/fs/nfsd/nfssvc.c
> +++ b/fs/nfsd/nfssvc.c
> @@ -289,17 +289,21 @@ static int nfsd_init_socks(struct net *net, const struct cred *cred)
> {
> int error;
> struct nfsd_net *nn = net_generic(net, nfsd_net_id);
> + int flags = SVC_SOCK_DEFAULTS;
>
> if (!list_empty(&nn->nfsd_serv->sv_permsocks))
> return 0;
>
> + if (!nfsd_rpcbind_error_is_fatal())
> + flags |= SVC_SOCK_RPCBIND_NOERR;
> +
> error = svc_create_xprt(nn->nfsd_serv, "udp", net, PF_INET, NFS_PORT,
> - SVC_SOCK_DEFAULTS, cred);
> + flags, cred);
> if (error < 0)
> return error;
>
> error = svc_create_xprt(nn->nfsd_serv, "tcp", net, PF_INET, NFS_PORT,
> - SVC_SOCK_DEFAULTS, cred);
> + flags, cred);
> if (error < 0)
> return error;
>
> @@ -340,6 +344,16 @@ static void nfsd_shutdown_generic(void)
> nfsd_file_cache_shutdown();
> }
>
> +static bool nfsd_rpcbind_error_fatal = false;
> +module_param(nfsd_rpcbind_error_fatal, bool, 0644);
> +MODULE_PARM_DESC(nfsd_rpcbind_error_fatal,
> + "rpcbind errors are fatal when starting nfsd.");
> +
> +bool nfsd_rpcbind_error_is_fatal(void)
> +{
> + return nfsd_rpcbind_error_fatal;
> +}
> +
> /*
> * Allow admin to disable lockd. This would typically be used to allow (e.g.)
> * a userspace NLM server of some sort to be used.
> diff --git a/include/linux/sunrpc/svcsock.h b/include/linux/sunrpc/svcsock.h
> index bcc555c7ae9c..f34c222cee9d 100644
> --- a/include/linux/sunrpc/svcsock.h
> +++ b/include/linux/sunrpc/svcsock.h
> @@ -61,8 +61,8 @@ void svc_drop(struct svc_rqst *);
> void svc_sock_update_bufs(struct svc_serv *serv);
> bool svc_alien_sock(struct net *net, int fd);
> int svc_addsock(struct svc_serv *serv, const int fd,
> - char *name_return, const size_t len,
> - const struct cred *cred);
> + char *name_return, const size_t len, int flags,
> + const struct cred *cred);
> void svc_init_xprt_sock(void);
> void svc_cleanup_xprt_sock(void);
> struct svc_xprt *svc_sock_create(struct svc_serv *serv, int prot);
> @@ -74,5 +74,6 @@ void svc_sock_destroy(struct svc_xprt *);
> #define SVC_SOCK_DEFAULTS (0U)
> #define SVC_SOCK_ANONYMOUS (1U << 0) /* don't register with pmap */
> #define SVC_SOCK_TEMPORARY (1U << 1) /* flag socket as temporary */
> +#define SVC_SOCK_RPCBIND_NOERR (1U << 2) /* Ignore pmap errors */
>
> #endif /* SUNRPC_SVCSOCK_H */
> diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c
> index 478f857cdaed..7f5b12a50bf9 100644
> --- a/net/sunrpc/svcsock.c
> +++ b/net/sunrpc/svcsock.c
> @@ -1309,14 +1309,15 @@ static struct svc_sock *svc_setup_socket(struct svc_serv *serv,
> inet = sock->sk;
>
> /* Register socket with portmapper */
> - if (pmap_register)
> + if (pmap_register) {
> err = svc_register(serv, sock_net(sock->sk), inet->sk_family,
> inet->sk_protocol,
> ntohs(inet_sk(inet)->inet_sport));
>
> - if (err < 0) {
> - kfree(svsk);
> - return ERR_PTR(err);
> + if (err < 0 && !(flags & SVC_SOCK_RPCBIND_NOERR)) {
> + kfree(svsk);
> + return ERR_PTR(err);
> + }
> }
>
> svsk->sk_sock = sock;
> @@ -1364,6 +1365,7 @@ EXPORT_SYMBOL_GPL(svc_alien_sock);
> * @fd: file descriptor of the new listener
> * @name_return: pointer to buffer to fill in with name of listener
> * @len: size of the buffer
> + * @flags: flags argument for svc_setup_socket()
> * @cred: credential
> *
> * Fills in socket name and returns positive length of name if successful.
> @@ -1371,7 +1373,7 @@ EXPORT_SYMBOL_GPL(svc_alien_sock);
> * value.
> */
> int svc_addsock(struct svc_serv *serv, const int fd, char *name_return,
> - const size_t len, const struct cred *cred)
> + const size_t len, int flags, const struct cred *cred)
> {
> int err = 0;
> struct socket *so = sockfd_lookup(fd, &err);
> @@ -1395,7 +1397,7 @@ int svc_addsock(struct svc_serv *serv, const int fd, char *name_return,
> err = -ENOENT;
> if (!try_module_get(THIS_MODULE))
> goto out;
> - svsk = svc_setup_socket(serv, so, SVC_SOCK_DEFAULTS);
> + svsk = svc_setup_socket(serv, so, flags);
> if (IS_ERR(svsk)) {
> module_put(THIS_MODULE);
> err = PTR_ERR(svsk);
> --
> 2.33.1
>
--
Chuck Lever
next prev parent reply other threads:[~2021-12-19 18:15 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-19 1:37 [PATCH v2 00/10] Assorted patches for knfsd trondmy
2021-12-19 1:37 ` [PATCH v2 01/10] nfsd: map EBADF trondmy
2021-12-19 1:37 ` [PATCH v2 02/10] nfsd: Add errno mapping for EREMOTEIO trondmy
2021-12-19 1:37 ` [PATCH v2 03/10] nfsd: Retry once in nfsd_open on an -EOPENSTALE return trondmy
2021-12-19 1:37 ` [PATCH v2 04/10] nfsd: Distinguish between required and optional NFSv3 post-op attributes trondmy
2021-12-19 1:37 ` [PATCH v2 05/10] nfs: Add export support for weak cache consistency attributes trondmy
2021-12-19 1:37 ` [PATCH v2 06/10] nfsd: NFSv3 should allow zero length writes trondmy
2021-12-19 1:38 ` [PATCH v2 07/10] nfsd: Add a tracepoint for errors in nfsd4_clone_file_range() trondmy
2021-12-19 1:38 ` [PATCH v2 08/10] nfsd: Replace use of rwsem with errseq_t trondmy
2021-12-19 1:38 ` [PATCH v2 09/10] nfsd: allow lockd to be forcibly disabled trondmy
2021-12-19 1:38 ` [PATCH v2 10/10] nfsd: Ignore rpcbind errors on nfsd startup trondmy
2021-12-19 18:15 ` Chuck Lever III [this message]
2021-12-19 20:49 ` Trond Myklebust
2021-12-20 15:51 ` Chuck Lever III
2021-12-20 18:35 ` Trond Myklebust
2021-12-20 19:02 ` Chuck Lever III
2021-12-20 19:52 ` Trond Myklebust
2021-12-20 20:12 ` Chuck Lever III
2021-12-19 18:34 ` [PATCH v2 09/10] nfsd: allow lockd to be forcibly disabled Chuck Lever III
2021-12-21 18:14 ` [PATCH v2 07/10] nfsd: Add a tracepoint for errors in nfsd4_clone_file_range() Chuck Lever III
2021-12-19 20:11 ` [PATCH v2 06/10] nfsd: NFSv3 should allow zero length writes Chuck Lever III
2022-01-05 16:10 ` [PATCH v2 05/10] nfs: Add export support for weak cache consistency attributes Daire Byrne
2021-12-19 20:10 ` [PATCH v2 04/10] nfsd: Distinguish between required and optional NFSv3 post-op attributes Chuck Lever III
2021-12-19 21:09 ` Trond Myklebust
2021-12-20 16:02 ` Chuck Lever III
2021-12-20 18:38 ` Trond Myklebust
2021-12-20 19:22 ` Chuck Lever III
2021-12-21 18:10 ` [PATCH v2 00/10] Assorted patches for knfsd Chuck Lever III
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=831659F6-3005-459B-92ED-933BBCEE6FE9@oracle.com \
--to=chuck.lever@oracle.com \
--cc=bfields@redhat.com \
--cc=linux-nfs@vger.kernel.org \
--cc=trondmy@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox