2017-02-24 00:17:13

by Rasmus Villemoes

[permalink] [raw]
Subject: [PATCH] fs: nfsd: remove superfluous KERN_INFO

dprintk already provides a KERN_* prefix; this KERN_INFO just shows up
as some odd characters in the output.

Signed-off-by: Rasmus Villemoes <[email protected]>
---
fs/nfsd/nfs4state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index a0dee8ae9f97..df86e888f833 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -2281,7 +2281,7 @@ gen_callback(struct nfs4_client *clp, struct nfsd4_setclientid *se, struct svc_r
out_err:
conn->cb_addr.ss_family = AF_UNSPEC;
conn->cb_addrlen = 0;
- dprintk(KERN_INFO "NFSD: this client (clientid %08x/%08x) "
+ dprintk("NFSD: this client (clientid %08x/%08x) "
"will not receive delegations\n",
clp->cl_clientid.cl_boot, clp->cl_clientid.cl_id);

--
2.1.4



2017-02-24 20:46:58

by J. Bruce Fields

[permalink] [raw]
Subject: Re: [PATCH] fs: nfsd: remove superfluous KERN_INFO

Thanks, applying.--b.

On Fri, Feb 24, 2017 at 01:15:55AM +0100, Rasmus Villemoes wrote:
> dprintk already provides a KERN_* prefix; this KERN_INFO just shows up
> as some odd characters in the output.
>
> Signed-off-by: Rasmus Villemoes <[email protected]>
> ---
> fs/nfsd/nfs4state.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
> index a0dee8ae9f97..df86e888f833 100644
> --- a/fs/nfsd/nfs4state.c
> +++ b/fs/nfsd/nfs4state.c
> @@ -2281,7 +2281,7 @@ gen_callback(struct nfs4_client *clp, struct nfsd4_setclientid *se, struct svc_r
> out_err:
> conn->cb_addr.ss_family = AF_UNSPEC;
> conn->cb_addrlen = 0;
> - dprintk(KERN_INFO "NFSD: this client (clientid %08x/%08x) "
> + dprintk("NFSD: this client (clientid %08x/%08x) "
> "will not receive delegations\n",
> clp->cl_clientid.cl_boot, clp->cl_clientid.cl_id);
>
> --
> 2.1.4