| Message ID | 148227955423.31345.17642862410755193193.stgit@noble (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Dec 21, 2016 at 11:19:14AM +1100, NeilBrown wrote: > Reading etab may require hostname lookup, so it is not reliable > until the network is active. > But we want mountd to start before that so that it is ready > when the very first NFS request arrives. > So delay reading etab until that request arrives, by which time > the network must be online so hopefully hostname look will be reliable. > > An alternate would be to delay starting mountd and nfsd until the > network is on-line, but that will often be an unnecessary delay. One argument for that delay would be to get the grace period right: it's not really correct to start timing the grace period before you start accepting requests. In practice, with grace periods by default a minute and (I'm guessing) the delay here not even a few seconds, maybe it's a minor point. And in theory I guess knfsd could account for that by waiting to start the grace period timer until it receives its first rpc. Anyway, that's not an objection to applying this patch. --b. > > Signed-off-by: NeilBrown <neilb@suse.com> > --- > utils/mountd/mountd.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/utils/mountd/mountd.c b/utils/mountd/mountd.c > index 5d9466f5c651..61699e62a6f0 100644 > --- a/utils/mountd/mountd.c > +++ b/utils/mountd/mountd.c > @@ -852,8 +852,6 @@ main(int argc, char **argv) > sa.sa_handler = sig_hup; > sigaction(SIGHUP, &sa, NULL); > > - auth_init(); > - > if (!foreground) { > /* We first fork off a child. */ > if ((c = fork()) > 0) > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Dec 23 2016, J. Bruce Fields wrote: > On Wed, Dec 21, 2016 at 11:19:14AM +1100, NeilBrown wrote: >> Reading etab may require hostname lookup, so it is not reliable >> until the network is active. >> But we want mountd to start before that so that it is ready >> when the very first NFS request arrives. >> So delay reading etab until that request arrives, by which time >> the network must be online so hopefully hostname look will be reliable. >> >> An alternate would be to delay starting mountd and nfsd until the >> network is on-line, but that will often be an unnecessary delay. > > One argument for that delay would be to get the grace period right: it's > not really correct to start timing the grace period before you start > accepting requests. > > In practice, with grace periods by default a minute and (I'm guessing) > the delay here not even a few seconds, maybe it's a minor point. > > And in theory I guess knfsd could account for that by waiting to start > the grace period timer until it receives its first rpc. Interesting idea. One would need to be careful about the case where there are no clients wanting to recover state, so some timeout without seeing any RPC would be needed. It's times like this that I wish the grace period was a lot shorter, but was auto-extended whenever a state-recovery request arrived (maybe with some limit). But that isn't what the spec say :-( Thanks, NeilBrown > > Anyway, that's not an objection to applying this patch. > > --b. > >> >> Signed-off-by: NeilBrown <neilb@suse.com> >> --- >> utils/mountd/mountd.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/utils/mountd/mountd.c b/utils/mountd/mountd.c >> index 5d9466f5c651..61699e62a6f0 100644 >> --- a/utils/mountd/mountd.c >> +++ b/utils/mountd/mountd.c >> @@ -852,8 +852,6 @@ main(int argc, char **argv) >> sa.sa_handler = sig_hup; >> sigaction(SIGHUP, &sa, NULL); >> >> - auth_init(); >> - >> if (!foreground) { >> /* We first fork off a child. */ >> if ((c = fork()) > 0) >>
On Fri, Dec 23, 2016 at 10:16:57AM +1100, NeilBrown wrote: > On Fri, Dec 23 2016, J. Bruce Fields wrote: > > > On Wed, Dec 21, 2016 at 11:19:14AM +1100, NeilBrown wrote: > >> Reading etab may require hostname lookup, so it is not reliable > >> until the network is active. > >> But we want mountd to start before that so that it is ready > >> when the very first NFS request arrives. > >> So delay reading etab until that request arrives, by which time > >> the network must be online so hopefully hostname look will be reliable. > >> > >> An alternate would be to delay starting mountd and nfsd until the > >> network is on-line, but that will often be an unnecessary delay. > > > > One argument for that delay would be to get the grace period right: it's > > not really correct to start timing the grace period before you start > > accepting requests. > > > > In practice, with grace periods by default a minute and (I'm guessing) > > the delay here not even a few seconds, maybe it's a minor point. > > > > And in theory I guess knfsd could account for that by waiting to start > > the grace period timer until it receives its first rpc. > > Interesting idea. One would need to be careful about the case where > there are no clients wanting to recover state, so some timeout without > seeing any RPC would be needed. In theory the server should have enough information to skip the grace period in that case, I'm not sure if it always does. But there might be rare cases where a client that was expected to cover state doesn't, in which case we could impose an unnecessary wait on the first new client..... > It's times like this that I wish the grace period was a lot shorter, but > was auto-extended whenever a state-recovery request arrived (maybe with > some limit). But that isn't what the spec say :-( Actually I think it does permit that, but I don't have a citation to the exact text.--b. > > Thanks, > NeilBrown > > > > > > Anyway, that's not an objection to applying this patch. > > > > --b. > > > >> > >> Signed-off-by: NeilBrown <neilb@suse.com> > >> --- > >> utils/mountd/mountd.c | 2 -- > >> 1 file changed, 2 deletions(-) > >> > >> diff --git a/utils/mountd/mountd.c b/utils/mountd/mountd.c > >> index 5d9466f5c651..61699e62a6f0 100644 > >> --- a/utils/mountd/mountd.c > >> +++ b/utils/mountd/mountd.c > >> @@ -852,8 +852,6 @@ main(int argc, char **argv) > >> sa.sa_handler = sig_hup; > >> sigaction(SIGHUP, &sa, NULL); > >> > >> - auth_init(); > >> - > >> if (!foreground) { > >> /* We first fork off a child. */ > >> if ((c = fork()) > 0) > >> -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/utils/mountd/mountd.c b/utils/mountd/mountd.c index 5d9466f5c651..61699e62a6f0 100644 --- a/utils/mountd/mountd.c +++ b/utils/mountd/mountd.c @@ -852,8 +852,6 @@ main(int argc, char **argv) sa.sa_handler = sig_hup; sigaction(SIGHUP, &sa, NULL); - auth_init(); - if (!foreground) { /* We first fork off a child. */ if ((c = fork()) > 0)
Reading etab may require hostname lookup, so it is not reliable until the network is active. But we want mountd to start before that so that it is ready when the very first NFS request arrives. So delay reading etab until that request arrives, by which time the network must be online so hopefully hostname look will be reliable. An alternate would be to delay starting mountd and nfsd until the network is on-line, but that will often be an unnecessary delay. Signed-off-by: NeilBrown <neilb@suse.com> --- utils/mountd/mountd.c | 2 -- 1 file changed, 2 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html