From: Ian Kent <raven@themaw.net> To: NeilBrown <neilb@suse.com>, autofs@vger.kernel.org Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [ANNOUNCE] autofs 5.1.2 release Date: Wed, 20 Dec 2017 13:52:58 +0800 [thread overview] Message-ID: <5aee2303-d1c3-49fd-0ef9-362fb0256fd5@themaw.net> (raw) In-Reply-To: <876092krew.fsf@notabene.neil.brown.name> On 20/12/17 11:29, NeilBrown wrote: > > Hi Ian, > I've been looking at: > >> - add configuration option to use fqdn in mounts. > > (commit 9aeef772604) because using this new option causes a regression. > If you are using the "replicated server" functionality, then > use_hostname_for_mounts = yes > completely disables it. Yes, that's not quite right. It disables the probe and proximity check for each distinct host name used. Each of the entries in the list of hosts should still be attempted and given that NFS ping is also now used in the NFS mount module what's lost is the preferred ordering of the hosts list. > > This is caused by: > > diff --git a/modules/replicated.c b/modules/replicated.c > index 32860d5fe245..8437f5f3d5b2 100644 > --- a/modules/replicated.c > +++ b/modules/replicated.c > @@ -667,6 +667,12 @@ int prune_host_list(unsigned logopt, struct host **list, > if (!*list) > return 0; > > + /* If we're using the host name then there's no point probing > + * avialability and respose time. > + */ > + if (defaults_use_hostname_for_mounts()) > + return 1; > + > /* Use closest hosts to choose NFS version */ > > My question is: why what this particular change made. It was a while ago but there were complains about using the IP address for mounts. It was requested to provide a way to prevent that and force the use of the host name in mounts. > Why can't prune_host_list() be allowed to do it's thing > when use_hostname_for_mounts is set. We could if each host name resolved to a single IP address. I'd need to check that use_hostname_for_mounts doesn't get in the road but the host struct should have ->rr set to true if it has multiple addresses so changing it to work the way your recommending shouldn't be hard. I think there's a couple of places that would need to be checked. If the host does resolve to multiple addresses the situation is different. There's no way to stop the actual mount from trying an IP address that's not responding and proximity doesn't make sense either again because every time a lookup is done on the host name (eg. at mount time) the next address in its list will be returned which can and usually is different from what would have been checked. > I understand that it would be pointless choosing between > the different interfaces of a multi-homed host, but there is still value > in choosing between multiple distinct hosts. > > What, if anything, might go wrong if I simply reverse this chunk of the > patch? You'll get IP addresses in the logs in certain cases but that should be all. It would probably be better to ensure that the checks are done if the host name resolves to a single IP address. Ian
WARNING: multiple messages have this Message-ID (diff)
From: Ian Kent <raven@themaw.net> To: NeilBrown <neilb@suse.com>, autofs@vger.kernel.org Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [ANNOUNCE] autofs 5.1.2 release Date: Wed, 20 Dec 2017 13:52:58 +0800 [thread overview] Message-ID: <5aee2303-d1c3-49fd-0ef9-362fb0256fd5@themaw.net> (raw) In-Reply-To: <876092krew.fsf@notabene.neil.brown.name> On 20/12/17 11:29, NeilBrown wrote: > > Hi Ian, > I've been looking at: > >> - add configuration option to use fqdn in mounts. > > (commit 9aeef772604) because using this new option causes a regression. > If you are using the "replicated server" functionality, then > use_hostname_for_mounts = yes > completely disables it. Yes, that's not quite right. It disables the probe and proximity check for each distinct host name used. Each of the entries in the list of hosts should still be attempted and given that NFS ping is also now used in the NFS mount module what's lost is the preferred ordering of the hosts list. > > This is caused by: > > diff --git a/modules/replicated.c b/modules/replicated.c > index 32860d5fe245..8437f5f3d5b2 100644 > --- a/modules/replicated.c > +++ b/modules/replicated.c > @@ -667,6 +667,12 @@ int prune_host_list(unsigned logopt, struct host **list, > if (!*list) > return 0; > > + /* If we're using the host name then there's no point probing > + * avialability and respose time. > + */ > + if (defaults_use_hostname_for_mounts()) > + return 1; > + > /* Use closest hosts to choose NFS version */ > > My question is: why what this particular change made. It was a while ago but there were complains about using the IP address for mounts. It was requested to provide a way to prevent that and force the use of the host name in mounts. > Why can't prune_host_list() be allowed to do it's thing > when use_hostname_for_mounts is set. We could if each host name resolved to a single IP address. I'd need to check that use_hostname_for_mounts doesn't get in the road but the host struct should have ->rr set to true if it has multiple addresses so changing it to work the way your recommending shouldn't be hard. I think there's a couple of places that would need to be checked. If the host does resolve to multiple addresses the situation is different. There's no way to stop the actual mount from trying an IP address that's not responding and proximity doesn't make sense either again because every time a lookup is done on the host name (eg. at mount time) the next address in its list will be returned which can and usually is different from what would have been checked. > I understand that it would be pointless choosing between > the different interfaces of a multi-homed host, but there is still value > in choosing between multiple distinct hosts. > > What, if anything, might go wrong if I simply reverse this chunk of the > patch? You'll get IP addresses in the logs in certain cases but that should be all. It would probably be better to ensure that the checks are done if the host name resolves to a single IP address. Ian -- To unsubscribe from this list: send the line "unsubscribe autofs" in
next prev parent reply other threads:[~2017-12-20 5:53 UTC|newest] Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-06-15 3:15 [ANNOUNCE] autofs 5.1.2 release Ian Kent 2016-06-15 3:15 ` Ian Kent 2017-12-20 3:29 ` NeilBrown 2017-12-20 5:52 ` Ian Kent [this message] 2017-12-20 5:52 ` Ian Kent 2017-12-20 6:10 ` Ian Kent 2017-12-20 6:50 ` Ian Kent 2017-12-20 6:50 ` Ian Kent 2017-12-21 1:09 ` NeilBrown 2017-12-21 11:06 ` Ian Kent 2017-12-21 11:36 ` Ian Kent 2017-12-21 11:36 ` Ian Kent 2018-01-02 22:14 ` NeilBrown 2018-01-18 1:15 ` Ian Kent 2018-01-18 1:15 ` Ian Kent 2018-01-18 2:19 ` Ian Kent 2018-01-18 2:19 ` Ian Kent 2017-12-20 7:30 ` Ian Kent 2017-12-20 7:30 ` Ian Kent
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=5aee2303-d1c3-49fd-0ef9-362fb0256fd5@themaw.net \ --to=raven@themaw.net \ --cc=autofs@vger.kernel.org \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=neilb@suse.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.