Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:Leap:15.2:Rings:1-MinimalX
autofs
0001-use_hostname_for_mounts-shouldn-t-prevent-...
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 0001-use_hostname_for_mounts-shouldn-t-prevent-selection-.patch of Package autofs
From b25cc873cda78c466ae75702be515b540e1bb7f3 Mon Sep 17 00:00:00 2001 From: NeilBrown <neilb@suse.com> Date: Mon, 29 Jan 2018 18:02:49 +1100 Subject: [PATCH] use_hostname_for_mounts shouldn't prevent selection among replicas If several replicas have been specified for a mount point, and use_hostname_for_mount is set to "yes", the selection between these replicas is currently disabled and the last in the list is always chosen. There is little point selecting between different addresses for the one host in this case, but it is still worth selecting between different hosts, particularly if different weights have been specified. This patch restores the "prune_host_list()" functionality when use_hostname_for_mount is set, and modifies it slightly so that only on IP address for any host:/path entry in the config file is willl be successfully probed. After a success, further addresses from the same entry are skipped. This is achieved by tracking an entry number ("ent_num") for each 'struct host'. Signed-off-by: NeilBrown <neilb@suse.com> --- include/replicated.h | 3 ++- modules/mount_nfs.c | 2 +- modules/replicated.c | 35 ++++++++++++++++++++--------------- 3 files changed, 23 insertions(+), 17 deletions(-) --- a/include/replicated.h +++ b/include/replicated.h @@ -55,6 +55,7 @@ struct host { char *name; + int ent_num; struct sockaddr *addr; size_t addr_len; unsigned int rr; @@ -68,7 +69,7 @@ struct host { }; void seed_random(void); -struct host *new_host(const char *, struct sockaddr *, size_t, +struct host *new_host(const char *, int, struct sockaddr *, size_t, unsigned int, unsigned int, unsigned int); void free_host_list(struct host **); int parse_location(unsigned, struct host **, const char *, unsigned int); --- a/modules/mount_nfs.c +++ b/modules/mount_nfs.c @@ -220,7 +220,7 @@ int mount_mount(struct autofs_point *ap, (vers & NFS_VERS_MASK) != 0 && (vers & NFS4_VERS_MASK) != 0) { unsigned int v4_probe_ok = 0; - struct host *tmp = new_host(hosts->name, + struct host *tmp = new_host(hosts->name, 0, hosts->addr, hosts->addr_len, hosts->proximity, hosts->weight, hosts->options); --- a/modules/replicated.c +++ b/modules/replicated.c @@ -83,7 +83,7 @@ void seed_random(void) return; } -struct host *new_host(const char *name, +struct host *new_host(const char *name, int ent_num, struct sockaddr *addr, size_t addr_len, unsigned int proximity, unsigned int weight, unsigned int options) @@ -116,6 +116,7 @@ struct host *new_host(const char *name, memset(new, 0, sizeof(struct host)); new->name = tmp1; + new->ent_num = ent_num; new->addr_len = addr_len; new->addr = tmp2; new->proximity = proximity; @@ -653,7 +654,7 @@ done: int prune_host_list(unsigned logopt, struct host **list, unsigned int vers, int port) { - struct host *this, *last, *first; + struct host *this, *last, *first, *prev; struct host *new = NULL; unsigned int proximity, selected_version = 0; unsigned int v2_tcp_count, v3_tcp_count, v4_tcp_count; @@ -665,12 +666,6 @@ int prune_host_list(unsigned logopt, str 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 */ first = *list; @@ -816,11 +811,18 @@ int prune_host_list(unsigned logopt, str first = last; this = first; + prev = NULL; while (this) { struct host *next = this->next; if (!this->name) { remove_host(list, this); add_host(&new, this); + } else if (defaults_use_hostname_for_mounts() && prev && + prev->ent_num == this->ent_num) { + /* When we use the hostname to mount, there is no + * point in probing every address it has, just one is + * enough. Skip the rest. + */ } else { status = get_supported_ver_and_cost(logopt, this, selected_version, port); @@ -828,6 +830,7 @@ int prune_host_list(unsigned logopt, str this->version = selected_version; remove_host(list, this); add_host(&new, this); + prev = this; } } this = next; @@ -840,7 +843,7 @@ int prune_host_list(unsigned logopt, str } static int add_new_host(struct host **list, - const char *host, unsigned int weight, + const char *host, int ent_num, unsigned int weight, struct addrinfo *host_addr, unsigned int rr, unsigned int options) { @@ -879,7 +882,7 @@ static int add_new_host(struct host **li else return 0; - new = new_host(host, host_addr->ai_addr, addr_len, prx, weight, options); + new = new_host(host, ent_num, host_addr->ai_addr, addr_len, prx, weight, options); if (!new) return 0; @@ -892,7 +895,7 @@ static int add_new_host(struct host **li return 1; } -static int add_host_addrs(struct host **list, const char *host, +static int add_host_addrs(struct host **list, const char *host, int ent_num, unsigned int weight, unsigned int options) { struct addrinfo hints, *ni, *this; @@ -927,7 +930,7 @@ static int add_host_addrs(struct host ** this = ni; while (this) { - ret = add_new_host(list, host, weight, this, 0, options); + ret = add_new_host(list, host, ent_num, weight, this, 0, options); if (!ret) break; this = this->ai_next; @@ -966,7 +969,7 @@ try_name: rr++; this = ni; while (this) { - ret = add_new_host(list, host, weight, this, rr, options); + ret = add_new_host(list, host, ent_num, weight, this, rr, options); if (!ret) break; this = this->ai_next; @@ -1059,6 +1062,7 @@ int parse_location(unsigned logopt, stru { char *str, *p, *delim; unsigned int empty = 1; + int ent_num = 1; if (!list) return 0; @@ -1116,7 +1120,7 @@ int parse_location(unsigned logopt, stru } if (p != delim) { - if (!add_host_addrs(hosts, p, weight, options)) { + if (!add_host_addrs(hosts, p, ent_num, weight, options)) { if (empty) { p = next; continue; @@ -1138,7 +1142,7 @@ int parse_location(unsigned logopt, stru *delim = '\0'; next = delim + 1; - if (!add_host_addrs(hosts, p, weight, options)) { + if (!add_host_addrs(hosts, p, ent_num, weight, options)) { p = next; continue; } @@ -1152,6 +1156,7 @@ int parse_location(unsigned logopt, stru return 0; } + ent_num ++; p = next; }
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor