aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteve Dickson <steved@redhat.com>2021-02-09 11:25:38 -0500
committerSteve Dickson <steved@redhat.com>2021-02-18 17:32:35 -0500
commitbcbd8553720b48607352e929f5edaddd9ee74d4c (patch)
treecf959fc937ccef2bce2f835ccc0e6ab99d24fb88
parent15dc0bead10d20c31e72ca94ce21eb66dc3528d5 (diff)
downloadnfs-utils-bcbd8553720b48607352e929f5edaddd9ee74d4c.tar.gz
exportd: multiple threads
Ported the multiple thread code from mountd (commit 11d34d11) Signed-off-by: Steve Dickson <steved@redhat.com>
-rw-r--r--nfs.conf3
-rw-r--r--systemd/nfs.conf.man9
-rw-r--r--utils/exportd/exportd.c118
-rw-r--r--utils/exportd/exportd.man7
4 files changed, 133 insertions, 4 deletions
diff --git a/nfs.conf b/nfs.conf
index 9fcf1bf0..4b344fa1 100644
--- a/nfs.conf
+++ b/nfs.conf
@@ -29,6 +29,9 @@
# port=0
# udp-port=0
#
+[exportd]
+# debug="all|auth|call|general|parse"
+# threads=1
[mountd]
# debug="all|auth|call|general|parse"
# manage-gids=n
diff --git a/systemd/nfs.conf.man b/systemd/nfs.conf.man
index 16e0ec4d..a4379fdf 100644
--- a/systemd/nfs.conf.man
+++ b/systemd/nfs.conf.man
@@ -129,6 +129,15 @@ but on the server, this will resolve to the path
.BR /my/root/filesystem .
.TP
+.B exportd
+Recognized values:
+.B threads
+
+See
+.BR exportd (8)
+for details.
+
+.TP
.B nfsdcltrack
Recognized values:
.BR storagedir .
diff --git a/utils/exportd/exportd.c b/utils/exportd/exportd.c
index 2f67e3bd..c8145033 100644
--- a/utils/exportd/exportd.c
+++ b/utils/exportd/exportd.c
@@ -15,6 +15,8 @@
#include <signal.h>
#include <string.h>
#include <getopt.h>
+#include <errno.h>
+#include <wait.h>
#include "nfslib.h"
#include "conffile.h"
@@ -26,6 +28,13 @@ extern void my_svc_run(void);
struct state_paths etab;
struct state_paths rmtab;
+/* Number of mountd threads to start. Default is 1 and
+ * that's probably enough unless you need hundreds of
+ * clients to be able to mount at once. */
+static int num_threads = 1;
+/* Arbitrary limit on number of threads */
+#define MAX_THREADS 64
+
int manage_gids;
int use_ipaddr = -1;
@@ -34,6 +43,7 @@ static struct option longopts[] =
{ "foreground", 0, 0, 'F' },
{ "debug", 1, 0, 'd' },
{ "help", 0, 0, 'h' },
+ { "num-threads", 1, 0, 't' },
{ NULL, 0, 0, 0 }
};
@@ -42,10 +52,85 @@ static struct option longopts[] =
*/
inline static void set_signals(void);
+/* Wait for all worker child processes to exit and reap them */
+static void
+wait_for_workers (void)
+{
+ int status;
+ pid_t pid;
+
+ for (;;) {
+
+ pid = waitpid(0, &status, 0);
+
+ if (pid < 0) {
+ if (errno == ECHILD)
+ return; /* no more children */
+ xlog(L_FATAL, "mountd: can't wait: %s\n",
+ strerror(errno));
+ }
+
+ /* Note: because we SIG_IGN'd SIGCHLD earlier, this
+ * does not happen on 2.6 kernels, and waitpid() blocks
+ * until all the children are dead then returns with
+ * -ECHILD. But, we don't need to do anything on the
+ * death of individual workers, so we don't care. */
+ xlog(L_NOTICE, "mountd: reaped child %d, status %d\n",
+ (int)pid, status);
+ }
+}
+
+/* Fork num_threads worker children and wait for them */
+static void
+fork_workers(void)
+{
+ int i;
+ pid_t pid;
+
+ xlog(L_NOTICE, "mountd: starting %d threads\n", num_threads);
+
+ for (i = 0 ; i < num_threads ; i++) {
+ pid = fork();
+ if (pid < 0) {
+ xlog(L_FATAL, "mountd: cannot fork: %s\n",
+ strerror(errno));
+ }
+ if (pid == 0) {
+ /* worker child */
+
+ /* Re-enable the default action on SIGTERM et al
+ * so that workers die naturally when sent them.
+ * Only the parent unregisters with pmap and
+ * hence needs to do special SIGTERM handling. */
+ struct sigaction sa;
+ sa.sa_handler = SIG_DFL;
+ sa.sa_flags = 0;
+ sigemptyset(&sa.sa_mask);
+ sigaction(SIGHUP, &sa, NULL);
+ sigaction(SIGINT, &sa, NULL);
+ sigaction(SIGTERM, &sa, NULL);
+
+ /* fall into my_svc_run in caller */
+ return;
+ }
+ }
+
+ /* in parent */
+ wait_for_workers();
+ xlog(L_NOTICE, "exportd: no more workers, exiting\n");
+ exit(0);
+}
+
static void
killer (int sig)
{
+ if (num_threads > 1) {
+ /* play Kronos and eat our children */
+ kill(0, SIGTERM);
+ wait_for_workers();
+ }
xlog (L_NOTICE, "Caught signal %d, exiting.", sig);
+
exit(0);
}
static void
@@ -78,10 +163,20 @@ static void
usage(const char *prog, int n)
{
fprintf(stderr,
- "Usage: %s [-f|--foreground] [-h|--help] [-d kind|--debug kind]\n", prog);
+ "Usage: %s [-f|--foreground] [-h|--help] [-d kind|--debug kind]\n"
+" [-t num|--num-threads=num]\n", prog);
exit(n);
}
+inline static void
+read_exportd_conf(char *progname)
+{
+ conf_init_file(NFS_CONFFILE);
+
+ xlog_set_debug(progname);
+
+ num_threads = conf_get_num("exportd", "threads", num_threads);
+}
int
main(int argc, char **argv)
{
@@ -98,10 +193,10 @@ main(int argc, char **argv)
/* Initialize logging. */
xlog_open(progname);
- conf_init_file(NFS_CONFFILE);
- xlog_set_debug(progname);
+ /* Read in config setting */
+ read_exportd_conf(progname);
- while ((c = getopt_long(argc, argv, "d:fh", longopts, NULL)) != EOF) {
+ while ((c = getopt_long(argc, argv, "d:fht:", longopts, NULL)) != EOF) {
switch (c) {
case 'd':
xlog_sconfig(optarg, 1);
@@ -112,6 +207,9 @@ main(int argc, char **argv)
case 'h':
usage(progname, 0);
break;
+ case 't':
+ num_threads = atoi (optarg);
+ break;
case '?':
default:
usage(progname, 1);
@@ -132,6 +230,18 @@ main(int argc, char **argv)
set_signals();
daemon_ready();
+ /* silently bounds check num_threads */
+ if (foreground)
+ num_threads = 1;
+ else if (num_threads < 1)
+ num_threads = 1;
+ else if (num_threads > MAX_THREADS)
+ num_threads = MAX_THREADS;
+
+ if (num_threads > 1)
+ fork_workers();
+
+
/* Open files now to avoid sharing descriptors among forked processes */
cache_open();
diff --git a/utils/exportd/exportd.man b/utils/exportd/exportd.man
index d786e575..1d65b5e0 100644
--- a/utils/exportd/exportd.man
+++ b/utils/exportd/exportd.man
@@ -44,6 +44,13 @@ Run in foreground (do not daemonize)
.TP
.B \-h " or " \-\-help
Display usage message.
+.TP
+.BR "\-t N" " or " "\-\-num\-threads=N " or " \-\-num\-threads N "
+This option specifies the number of worker threads that rpc.mountd
+spawns. The default is 1 thread, which is probably enough. More
+threads are usually only needed for NFS servers which need to handle
+mount storms of hundreds of NFS mounts in a few seconds, or when
+your DNS server is slow or unreliable.
.SH CONFIGURATION FILE
Many of the options that can be set on the command line can also be
controlled through values set in the