diff options
Diffstat (limited to 'rpc')
| -rw-r--r-- | rpc/rpc-lib/src/rpc-transport.c | 2 | ||||
| -rw-r--r-- | rpc/rpc-transport/rdma/src/name.c | 12 | ||||
| -rw-r--r-- | rpc/rpc-transport/socket/src/name.c | 26 | ||||
| -rw-r--r-- | rpc/rpc-transport/socket/src/socket.c | 3 | 
4 files changed, 17 insertions, 26 deletions
diff --git a/rpc/rpc-lib/src/rpc-transport.c b/rpc/rpc-lib/src/rpc-transport.c index e07bf0d76..8da898b61 100644 --- a/rpc/rpc-lib/src/rpc-transport.c +++ b/rpc/rpc-lib/src/rpc-transport.c @@ -585,7 +585,7 @@ rpc_transport_inet_options_build (dict_t **options, const char *hostname,                          "failed to set remote-port with %d", port);                  goto out;          } -        ret = dict_set_str (dict, "transport.address-family", "inet/inet6"); +        ret = dict_set_str (dict, "transport.address-family", "inet");          if (ret) {                  gf_log (THIS->name, GF_LOG_WARNING,                          "failed to set addr-family with inet"); diff --git a/rpc/rpc-transport/rdma/src/name.c b/rpc/rpc-transport/rdma/src/name.c index e5f7ba2ca..3718cd764 100644 --- a/rpc/rpc-transport/rdma/src/name.c +++ b/rpc/rpc-transport/rdma/src/name.c @@ -153,13 +153,11 @@ client_fill_address_family (rpc_transport_t *this, struct sockaddr *sockaddr)                          sockaddr->sa_family = AF_INET6;                  } else if (!strcasecmp (address_family, "inet-sdp")) {                          sockaddr->sa_family = AF_INET_SDP; -                } else if (!strcasecmp (address_family, "inet/inet6") -                           || !strcasecmp (address_family, "inet6/inet")) { -                        sockaddr->sa_family = AF_UNSPEC;                  } else {                          gf_log (this->name, GF_LOG_ERROR,                                  "unknown address-family (%s) specified",                                  address_family); +                        sockaddr->sa_family = AF_UNSPEC;                          return -1;                  }          } @@ -521,21 +519,19 @@ gf_rdma_server_get_local_sockaddr (rpc_transport_t *this,                          addr->sa_family = AF_INET_SDP;                  } else if (!strcasecmp (address_family, "unix")) {                          addr->sa_family = AF_UNIX; -                } else if (!strcasecmp (address_family, "inet/inet6") -                           || !strcasecmp (address_family, "inet6/inet")) { -                        addr->sa_family = AF_UNSPEC;                  } else {                          gf_log (this->name, GF_LOG_ERROR,                                  "unknown address family (%s) specified",                                  address_family); +                        addr->sa_family = AF_UNSPEC;                          ret = -1;                          goto err;                  }          } else {                  gf_log (this->name, GF_LOG_DEBUG,                          "option address-family not specified, defaulting " -                        "to inet/inet6"); -                addr->sa_family = AF_UNSPEC; +                        "to inet"); +                addr->sa_family = AF_INET;          }          switch (addr->sa_family) diff --git a/rpc/rpc-transport/socket/src/name.c b/rpc/rpc-transport/socket/src/name.c index 0b2ca5805..d37c83e18 100644 --- a/rpc/rpc-transport/socket/src/name.c +++ b/rpc/rpc-transport/socket/src/name.c @@ -134,24 +134,24 @@ client_fill_address_family (rpc_transport_t *this, sa_family_t *sa_family)                  if (!(remote_host_data || connect_path_data) ||                      (remote_host_data && connect_path_data)) {                          gf_log (this->name, GF_LOG_ERROR, -                                "transport.address-family not specified and " -                                "not able to determine the " -                                "same from other options (remote-host:%s and " -                                "transport.unix.connect-path:%s)", +                                "transport.address-family not specified. " +                                "Could not guess default value from (remote-host:%s or " +                                "transport.unix.connect-path:%s) options",                                  data_to_str (remote_host_data),                                  data_to_str (connect_path_data)); +                        *sa_family = AF_UNSPEC;                          goto out;                  }                  if (remote_host_data) {                          gf_log (this->name, GF_LOG_DEBUG,                                  "address-family not specified, guessing it " -                                "to be inet/inet6"); -                        *sa_family = AF_UNSPEC; +                                "to be inet from (remote-host: %s)", data_to_str (remote_host_data)); +                        *sa_family = AF_INET;                  } else {                          gf_log (this->name, GF_LOG_DEBUG,                                  "address-family not specified, guessing it " -                                "to be unix"); +                                "to be unix from (transport.unix.connect-path: %s)", data_to_str (connect_path_data));                          *sa_family = AF_UNIX;                  } @@ -165,13 +165,11 @@ client_fill_address_family (rpc_transport_t *this, sa_family_t *sa_family)                          *sa_family = AF_INET6;                  } else if (!strcasecmp (address_family, "inet-sdp")) {                          *sa_family = AF_INET_SDP; -                } else if (!strcasecmp (address_family, "inet/inet6") -                           || !strcasecmp (address_family, "inet6/inet")) { -                        *sa_family = AF_UNSPEC;                  } else {                          gf_log (this->name, GF_LOG_ERROR,                                  "unknown address-family (%s) specified",                                  address_family); +                        *sa_family = AF_UNSPEC;                          goto out;                  }          } @@ -542,18 +540,16 @@ server_fill_address_family (rpc_transport_t *this, sa_family_t *sa_family)                          *sa_family = AF_INET_SDP;                  } else if (!strcasecmp (address_family, "unix")) {                          *sa_family = AF_UNIX; -                } else if (!strcasecmp (address_family, "inet/inet6") -                           || !strcasecmp (address_family, "inet6/inet")) { -                        *sa_family = AF_UNSPEC;                  } else {                          gf_log (this->name, GF_LOG_ERROR,                                  "unknown address family (%s) specified", address_family); +                        *sa_family = AF_UNSPEC;                          goto out;                  }          } else {                  gf_log (this->name, GF_LOG_DEBUG, -                        "option address-family not specified, defaulting to inet/inet6"); -                *sa_family = AF_UNSPEC; +                        "option address-family not specified, defaulting to inet"); +                *sa_family = AF_INET;          }          ret = 0; diff --git a/rpc/rpc-transport/socket/src/socket.c b/rpc/rpc-transport/socket/src/socket.c index 2476b13f7..d193c93ce 100644 --- a/rpc/rpc-transport/socket/src/socket.c +++ b/rpc/rpc-transport/socket/src/socket.c @@ -2805,8 +2805,7 @@ struct volume_options options[] = {          },          { .key   = { "transport.address-family",                       "address-family" }, -          .value = {"inet", "inet6", "inet/inet6", "inet6/inet", -                    "unix", "inet-sdp" }, +          .value = {"inet", "inet6", "unix", "inet-sdp" },            .type  = GF_OPTION_TYPE_STR          },  | 
