diff mbox series

Re: mount.nfs4 and logging

Message ID 8c44e6ce-7810-bba1-8779-127938fed1ab@gmch.uk
State New
Headers show
Series Re: mount.nfs4 and logging | expand

Commit Message

Chris Hall Sept. 20, 2020, 10:02 a.m. UTC
Steve,

Forwarded to you as suggested by Bruce Fields -- see below.

Chris


-------- Forwarded Message --------

On Sat, Sep 19, 2020 at 06:49:40PM +0100, Chris Hall wrote:
> On 19/09/2020 17:33, J. Bruce Fields wrote:
> >On Tue, Sep 15, 2020 at 02:06:23PM +0100, Chris Hall wrote:
> >>Also FWIW, I gather that this is configuration for the client-side
> >>'mount' of nfs exports, *only*.  I suppose it should be obvious that
> >>this has absolutely nothing to do with configuring (server-side)
> >>'mountd'.  Speaking as a fully paid up moron-in-a-hurry, it has
> >>taken me a while to work that out :-(  [I suggest that the comments
> >>in the .conf files and the man-page could say that nfs.conf is
> >>server-side and nfsmount.conf is client-side -- just a few words,
> >>for the avoidance of doubt.]
> >
> >That sounds sensible.  If you're feeling industrious, you can
> >
> >	git clone git://linux-nfs.org/~steved/nfs-utils
> >
> >and patch those files and mail us a patch....
> 
> Enclosed are suggested updates.

Oh, great, thanks.  Steve Dickson handles these, so send it to
steved@redhat.com, cc: linux-nfs@vger.kernel.org, and he should pick it
up.

--b.

> Chris

Comments

Steve Dickson Sept. 22, 2020, 5:30 p.m. UTC | #1
On 9/20/20 6:02 AM, Chris Hall wrote:
> 
> Steve,
> 
> Forwarded to you as suggested by Bruce Fields -- see below.
Thanks... I've been watching the discussion.

Would you mind reposting the patch in the proper format
and proper  Signed-off-by tag?

steved.
> 
> Chris
> 
> 
> -------- Forwarded Message --------
> 
> On Sat, Sep 19, 2020 at 06:49:40PM +0100, Chris Hall wrote:
>> On 19/09/2020 17:33, J. Bruce Fields wrote:
>> >On Tue, Sep 15, 2020 at 02:06:23PM +0100, Chris Hall wrote:
>> >>Also FWIW, I gather that this is configuration for the client-side
>> >>'mount' of nfs exports, *only*.  I suppose it should be obvious that
>> >>this has absolutely nothing to do with configuring (server-side)
>> >>'mountd'.  Speaking as a fully paid up moron-in-a-hurry, it has
>> >>taken me a while to work that out :-(  [I suggest that the comments
>> >>in the .conf files and the man-page could say that nfs.conf is
>> >>server-side and nfsmount.conf is client-side -- just a few words,
>> >>for the avoidance of doubt.]
>> >
>> >That sounds sensible.  If you're feeling industrious, you can
>> >
>> >    git clone git://linux-nfs.org/~steved/nfs-utils
>> >
>> >and patch those files and mail us a patch....
>>
>> Enclosed are suggested updates.
> 
> Oh, great, thanks.  Steve Dickson handles these, so send it to
> steved@redhat.com, cc: linux-nfs@vger.kernel.org, and he should pick it
> up.
> 
> --b.
> 
>> Chris
> 
>
diff mbox series

Patch

diff --git a/nfs.conf b/nfs.conf
index 186a5b19..9bfa0302 100644
--- a/nfs.conf
+++ b/nfs.conf
@@ -2,6 +2,8 @@ 
 # This is a general configuration for the
 # NFS daemons and tools
 #
+# Note that this is configuration for server-side NFS, only.
+#
 [general]
 # pipefs-directory=/var/lib/nfs/rpc_pipefs
 #
diff --git a/systemd/nfs.conf.man b/systemd/nfs.conf.man
index 3f1c7261..a647a29b 100644
--- a/systemd/nfs.conf.man
+++ b/systemd/nfs.conf.man
@@ -3,6 +3,9 @@ 
 nfs.conf \- general configuration for NFS daemons and tools
 .SH SYNOPSIS
 .I /etc/nfs.conf
+.PP
+Note that this is configuration for server-side NFS, only.  See 
+nfsmount.conf for the configuration of client-side NFS.
 .SH DESCRIPTION
 .PP
 This file contains site-specific configuration for various NFS daemons
diff --git a/utils/mount/nfsmount.conf b/utils/mount/nfsmount.conf
index 6bdc225a..79d23caa 100644
--- a/utils/mount/nfsmount.conf
+++ b/utils/mount/nfsmount.conf
@@ -1,6 +1,8 @@ 
 #
 # /etc/nfsmount.conf - see nfsmount.conf(5) for details
 #
+# Note that this is configuration for client-side mount operations, only.
+#
 # This is an NFS mount configuration file. This file can be broken
 # up into three different sections: Mount, Server and Global
 # 
@@ -115,16 +117,20 @@ 
 # Sets all attributes times to the same time (in seconds)
 # actimeo=30
 #
-# Server Mountd port mountport
+# Server Mountd port mountport - do *not* set this if *any* server
+#                                being used only supports nfs4
 # mountport=4001
 #
-# Server Mountd Protocol
+# Server Mountd Protocol       - do *not* set this if *any* server
+#                                being used only supports nfs4
 # mountproto=tcp
 #
-# Server Mountd Version
+# Server Mountd Version        - do *not* set this if *any* server
+#                                being used only supports nfs4
 # mountvers=3
 #
-# Server Mountd Host
+# Server Mountd Host           - do *not* set this if *any* server
+#                                being used only supports nfs4
 # mounthost=hostname
 #
 # Server Port
diff --git a/utils/mount/nfsmount.conf.man b/utils/mount/nfsmount.conf.man
index 3aa34564..f8dfb7d6 100644
--- a/utils/mount/nfsmount.conf.man
+++ b/utils/mount/nfsmount.conf.man
@@ -5,6 +5,9 @@  nfsmount.conf - Configuration file for NFS mounts
 .SH SYNOPSIS
 Configuration file for NFS mounts that allows options
 to be set globally, per server or per mount point.
+.PP
+Note that this is configuration for client-side mount operations,
+only.  See nfs.conf for the configuration of server-side NFS.
 .SH DESCRIPTION
 The configuration file is made up of multiple sections 
 followed by variables associated with that section.