From: Trond Myklebust <trondmy@hammerspace.com>
To: "anna.schumaker@netapp.com" <anna.schumaker@netapp.com>,
"chuck.lever@oracle.com" <chuck.lever@oracle.com>
Cc: "linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH 4/4] NFS: Replace fs_context-related dprintk() call sites with tracepoints
Date: Fri, 22 Jul 2022 18:00:58 +0000 [thread overview]
Message-ID: <a4e1c7e1f6229de8fcf0160fba68b4217e7ba82c.camel@hammerspace.com> (raw)
In-Reply-To: <165851074904.361126.11450659589753012982.stgit@morisot.1015granger.net>
On Fri, 2022-07-22 at 13:25 -0400, Chuck Lever wrote:
> Contributed as part of the long patch series that converts NFS from
> using dprintk to tracepoints for observability.
>
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
> Reviewed-by: Jeff Layton <jlayton@kernel.org>
> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
> ---
> fs/nfs/fs_context.c | 25 ++++++++++-------
> fs/nfs/nfstrace.h | 77
> +++++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 92 insertions(+), 10 deletions(-)
>
> diff --git a/fs/nfs/fs_context.c b/fs/nfs/fs_context.c
> index 9a16897e8dc6..35e400a557b9 100644
> --- a/fs/nfs/fs_context.c
> +++ b/fs/nfs/fs_context.c
> @@ -21,6 +21,8 @@
> #include "nfs.h"
> #include "internal.h"
>
> +#include "nfstrace.h"
> +
> #define NFSDBG_FACILITY NFSDBG_MOUNT
>
> #if IS_ENABLED(CONFIG_NFS_V3)
> @@ -284,7 +286,7 @@ static int nfs_verify_server_address(struct
> sockaddr *addr)
> }
> }
>
> - dfprintk(MOUNT, "NFS: Invalid IP address specified\n");
> + trace_nfs_mount_addr_err(addr);
> return 0;
> }
>
> @@ -378,7 +380,7 @@ static int nfs_parse_security_flavors(struct
> fs_context *fc,
> char *string = param->string, *p;
> int ret;
>
> - dfprintk(MOUNT, "NFS: parsing %s=%s option\n", param->key,
> param->string);
> + trace_nfs_mount_assign(param->key, string);
>
> while ((p = strsep(&string, ":")) != NULL) {
> if (!*p)
> @@ -480,7 +482,7 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> unsigned int len;
> int ret, opt;
>
> - dfprintk(MOUNT, "NFS: parsing nfs mount option '%s'\n",
> param->key);
> + trace_nfs_mount_option(param);
>
> opt = fs_parse(fc, nfs_fs_parameters, param, &result);
> if (opt < 0)
> @@ -683,6 +685,7 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> return ret;
> break;
> case Opt_vers:
> + trace_nfs_mount_assign(param->key, param->string);
> ret = nfs_parse_version_string(fc, param->string);
> if (ret < 0)
> return ret;
> @@ -694,6 +697,7 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> break;
>
> case Opt_proto:
> + trace_nfs_mount_assign(param->key, param->string);
> protofamily = AF_INET;
> switch (lookup_constant(nfs_xprt_protocol_tokens,
> param->string, -1)) {
> case Opt_xprt_udp6:
> @@ -729,6 +733,7 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> break;
>
> case Opt_mountproto:
> + trace_nfs_mount_assign(param->key, param->string);
> mountfamily = AF_INET;
> switch (lookup_constant(nfs_xprt_protocol_tokens,
> param->string, -1)) {
> case Opt_xprt_udp6:
> @@ -751,6 +756,7 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> break;
>
> case Opt_addr:
> + trace_nfs_mount_assign(param->key, param->string);
> len = rpc_pton(fc->net_ns, param->string, param-
> >size,
> &ctx->nfs_server.address,
> sizeof(ctx->nfs_server._address));
> @@ -759,16 +765,19 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> ctx->nfs_server.addrlen = len;
> break;
> case Opt_clientaddr:
> + trace_nfs_mount_assign(param->key, param->string);
> kfree(ctx->client_address);
> ctx->client_address = param->string;
> param->string = NULL;
> break;
> case Opt_mounthost:
> + trace_nfs_mount_assign(param->key, param->string);
> kfree(ctx->mount_server.hostname);
> ctx->mount_server.hostname = param->string;
> param->string = NULL;
> break;
> case Opt_mountaddr:
> + trace_nfs_mount_assign(param->key, param->string);
> len = rpc_pton(fc->net_ns, param->string, param-
> >size,
> &ctx->mount_server.address,
> sizeof(ctx->mount_server._address));
> @@ -846,7 +855,6 @@ static int nfs_fs_context_parse_param(struct
> fs_context *fc,
> */
> case Opt_sloppy:
> ctx->sloppy = true;
> - dfprintk(MOUNT, "NFS: relaxing parsing rules\n");
> break;
> }
>
> @@ -879,10 +887,8 @@ static int nfs_parse_source(struct fs_context
> *fc,
> size_t len;
> const char *end;
>
> - if (unlikely(!dev_name || !*dev_name)) {
> - dfprintk(MOUNT, "NFS: device name not specified\n");
> + if (unlikely(!dev_name || !*dev_name))
> return -EINVAL;
> - }
>
> /* Is the host name protected with square brakcets? */
> if (*dev_name == '[') {
> @@ -922,7 +928,7 @@ static int nfs_parse_source(struct fs_context
> *fc,
> if (!ctx->nfs_server.export_path)
> goto out_nomem;
>
> - dfprintk(MOUNT, "NFS: MNTPATH: '%s'\n", ctx-
> >nfs_server.export_path);
> + trace_nfs_mount_path(ctx->nfs_server.export_path);
> return 0;
>
> out_bad_devname:
> @@ -1116,7 +1122,6 @@ static int nfs23_parse_monolithic(struct
> fs_context *fc,
> return nfs_invalf(fc, "NFS: nfs_mount_data version supports
> only AUTH_SYS");
>
> out_nomem:
> - dfprintk(MOUNT, "NFS: not enough memory to handle mount
> options");
> return -ENOMEM;
>
> out_no_address:
> @@ -1248,7 +1253,7 @@ static int nfs4_parse_monolithic(struct
> fs_context *fc,
> if (IS_ERR(c))
> return PTR_ERR(c);
> ctx->nfs_server.export_path = c;
> - dfprintk(MOUNT, "NFS: MNTPATH: '%s'\n", c);
> + trace_nfs_mount_path(c);
>
> c = strndup_user(data->client_addr.data, 16);
> if (IS_ERR(c))
> diff --git a/fs/nfs/nfstrace.h b/fs/nfs/nfstrace.h
> index 012bd7339862..ccaeae42ee77 100644
> --- a/fs/nfs/nfstrace.h
> +++ b/fs/nfs/nfstrace.h
> @@ -1609,6 +1609,83 @@ TRACE_EVENT(nfs_fh_to_dentry,
> )
> );
>
> +TRACE_EVENT(nfs_mount_addr_err,
> + TP_PROTO(
> + const struct sockaddr *sap
> + ),
> +
> + TP_ARGS(sap),
> +
> + TP_STRUCT__entry(
> + __array(unsigned char, addr, sizeof(struct
> sockaddr_in6))
> + ),
> +
> + TP_fast_assign(
> + memcpy(__entry->addr, sap, sizeof(__entry->addr));
> + ),
> +
> + TP_printk("addr=%pISpc", __entry->addr)
Ditto problem with this.
> +);
> +
> +TRACE_EVENT(nfs_mount_assign,
> + TP_PROTO(
> + const char *option,
> + const char *value
> + ),
> +
> + TP_ARGS(option, value),
> +
> + TP_STRUCT__entry(
> + __string(option, option)
> + __string(value, value)
> + ),
> +
> + TP_fast_assign(
> + __assign_str(option, option);
> + __assign_str(value, value);
> + ),
> +
> + TP_printk("option %s=%s",
> + __get_str(option), __get_str(value)
> + )
> +);
> +
> +TRACE_EVENT(nfs_mount_option,
> + TP_PROTO(
> + const struct fs_parameter *param
> + ),
> +
> + TP_ARGS(param),
> +
> + TP_STRUCT__entry(
> + __string(option, param->key)
> + ),
> +
> + TP_fast_assign(
> + __assign_str(option, param->key);
> + ),
> +
> + TP_printk("option %s", __get_str(option))
> +);
> +
> +TRACE_EVENT(nfs_mount_path,
> + TP_PROTO(
> + const char *path
> + ),
> +
> + TP_ARGS(path),
> +
> + TP_STRUCT__entry(
> + __string(path, path)
> + ),
> +
> + TP_fast_assign(
> + __assign_str(path, path);
> + ),
> +
> + TP_printk("path='%s'", __get_str(path))
> +);
> +
> DECLARE_EVENT_CLASS(nfs_xdr_event,
> TP_PROTO(
> const struct xdr_stream *xdr,
>
>
--
Trond Myklebust
Linux NFS client maintainer, Hammerspace
trond.myklebust@hammerspace.com
next prev parent reply other threads:[~2022-07-22 18:01 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-22 17:25 [PATCH 0/4] Pre-requisites for client-side RPC-with-TLS support Chuck Lever
2022-07-22 17:25 ` [PATCH 1/4] SUNRPC: Fail faster on bad verifier Chuck Lever
2022-07-22 17:25 ` [PATCH 2/4] SUNRPC: Widen rpc_task::tk_flags Chuck Lever
2022-07-22 17:53 ` Trond Myklebust
2022-07-22 17:55 ` Chuck Lever III
2022-07-23 20:10 ` Trond Myklebust
2022-07-25 13:17 ` Chuck Lever III
2022-07-22 17:25 ` [PATCH 3/4] SUNRPC: Replace dprintk() call site in xs_data_ready Chuck Lever
2022-07-22 17:59 ` Trond Myklebust
2022-07-22 18:45 ` Chuck Lever III
2022-07-22 20:22 ` Steven Rostedt
2022-07-25 15:49 ` Arnaldo Carvalho de Melo
2022-07-25 16:06 ` Steven Rostedt
2022-07-22 17:25 ` [PATCH 4/4] NFS: Replace fs_context-related dprintk() call sites with tracepoints Chuck Lever
2022-07-22 18:00 ` Trond Myklebust [this message]
2022-07-22 19:05 ` 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=a4e1c7e1f6229de8fcf0160fba68b4217e7ba82c.camel@hammerspace.com \
--to=trondmy@hammerspace.com \
--cc=anna.schumaker@netapp.com \
--cc=chuck.lever@oracle.com \
--cc=linux-nfs@vger.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