From 5cbb78bf4bd871168ada54b0fd4b42b26c959f3e Mon Sep 17 00:00:00 2001 From: NeilBrown Date: Thu, 26 Mar 2015 11:27:27 -0400 Subject: mount: improve error message when statd cannot be started. If you try to mount and NFSv3 filesystem, and statd is not running and cannot be started (maybe rpcbind isn't running either), the error message is: mount.nfs: rpc.statd is not running but is required for remote locking. mount.nfs: Either use '-o nolock' to keep locks local, or start statd. mount.nfs: an incorrect mount option was specified That last line is incorrect and misleading: no incorret mount option was specified. This line comes from mount_error() in error.c. In this case that function doesn't really need to provide any more information. So introduce a concention that EALREADY means an error message has already been printed, and use it to suppress that message. Signed-off-by: NeilBrown Signed-off-by: Steve Dickson --- utils/mount/error.c | 3 +++ utils/mount/stropts.c | 1 + 2 files changed, 4 insertions(+) diff --git a/utils/mount/error.c b/utils/mount/error.c index e06f598..c9797fc 100644 --- a/utils/mount/error.c +++ b/utils/mount/error.c @@ -247,6 +247,9 @@ void mount_error(const char *spec, const char *mount_point, int error) nfs_error(_("%s: please report the error to" PACKAGE_BUGREPORT), progname); break; + case EALREADY: + /* Error message has already been provided */ + break; default: nfs_error(_("%s: %s"), progname, strerror(error)); diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c index 76ecb32..df67fac 100644 --- a/utils/mount/stropts.c +++ b/utils/mount/stropts.c @@ -298,6 +298,7 @@ static int nfs_verify_lock_option(struct mount_options *options) "required for remote locking."), progname); nfs_error(_("%s: Either use '-o nolock' to keep " "locks local, or start statd."), progname); + errno = EALREADY; /* Don't print further error message */ return 0; } -- cgit