mbox series

[nfs-utils,0/3] Add docs for some mount options missing from nfs(5)

Message ID 20230601194617.2174639-1-smayhew@redhat.com (mailing list archive)
Headers show
Series Add docs for some mount options missing from nfs(5) | expand

Message

Scott Mayhew June 1, 2023, 7:46 p.m. UTC
Scott Mayhew (3):
  nfs(5): Document the softerr mount option.
  nfs(5): Document the write=lazy|eager|wait mount option.
  nfs(5): Document the trunkdiscovery/notrunkdiscovery mount option.

 utils/mount/nfs.man | 84 +++++++++++++++++++++++++++++++++------------
 1 file changed, 63 insertions(+), 21 deletions(-)

Comments

Steve Dickson June 5, 2023, 2:23 p.m. UTC | #1
On 6/1/23 3:46 PM, Scott Mayhew wrote:
> Scott Mayhew (3):
>    nfs(5): Document the softerr mount option.
My only thought about this one, is do we want document
an option we do people using since it will cause data
corruption?

Personally I would be for ripping out all of the soft doc
from the man page.

steved.

>    nfs(5): Document the write=lazy|eager|wait mount option.
>    nfs(5): Document the trunkdiscovery/notrunkdiscovery mount option.
> 
>   utils/mount/nfs.man | 84 +++++++++++++++++++++++++++++++++------------
>   1 file changed, 63 insertions(+), 21 deletions(-)
>
Trond Myklebust June 5, 2023, 2:53 p.m. UTC | #2
On Mon, 2023-06-05 at 10:23 -0400, Steve Dickson wrote:
> 
> 
> On 6/1/23 3:46 PM, Scott Mayhew wrote:
> > Scott Mayhew (3):
> >    nfs(5): Document the softerr mount option.
> My only thought about this one, is do we want document
> an option we do people using since it will cause data
> corruption?
> 
> Personally I would be for ripping out all of the soft doc
> from the man page.
> 

Softerr was explicitly designed for use when re-exporting NFS. If you
use a hard mount, you will find that if one of the NFS servers being
re-exported goes down, eventually all your knfsd threads lock up. By
timing out, and returning an error, 'softerr' prevents that scenario.

The difference between using 'soft' and 'softerr' here lies in the fact
that knfsd will convert the ETIMEOUT into NFS3ERR_JUKEBOX /
NFS4ERR_DELAY for the NFS clients, causing it to retry the operation.
OTOH, 'soft' returns EIO, which gets converted to NFS3ERR_IO /
NFS4ERR_IO, which does cause the client to error out and lose data.

IOW: this is an option that has a definite use case.
Steve Dickson June 12, 2023, 8:04 p.m. UTC | #3
On 6/1/23 3:46 PM, Scott Mayhew wrote:
> Scott Mayhew (3):
>    nfs(5): Document the softerr mount option.
>    nfs(5): Document the write=lazy|eager|wait mount option.
>    nfs(5): Document the trunkdiscovery/notrunkdiscovery mount option.
> 
>   utils/mount/nfs.man | 84 +++++++++++++++++++++++++++++++++------------
>   1 file changed, 63 insertions(+), 21 deletions(-)
> 
Committed... (tag: nfs-utils-2-6-4-rc2)

steved.