> +static void acm_parse_hosts_file(struct acm_ep *ep)
> +{
> +     FILE *f;
> +     char s[120];
> +     char addr[INET6_ADDRSTRLEN], gid[INET6_ADDRSTRLEN];
> +     uint8_t name[ACM_MAX_ADDRESS];
> +     struct in6_addr ip_addr, ib_addr;
> +     struct acm_dest *dest, *new_dest;
> +     uint8_t addr_type;
> +
> +     if (!(f = fopen(hosts_file, "r"))) {
> +             acm_log(0, "ERROR - couldn't open %s\n", hosts_file);
> +             return;
> +        }
> +
> +     while (fgets(s, sizeof s, f)) {
> +             if (s[0] == '#')
> +                     continue;
> +
> +             if (sscanf(s, "%46s%46s", addr, gid) != 2)
> +                     continue;
> +
> +             acm_log(2, "%s", s);
> +             if (inet_pton(AF_INET6, gid, &ib_addr) <= 0) {
> +                     acm_log(0, "ERROR - %s is not IB GID\n", gid);
> +                     continue;
> +             }
> +             if (inet_pton(AF_INET, addr, &ip_addr) > 0)
> +                     addr_type = ACM_ADDRESS_IP;
> +             else if (inet_pton(AF_INET6, addr, &ip_addr) > 0)
> +                     addr_type = ACM_ADDRESS_IP6;
> +             else {
> +                     acm_log(0, "ERROR - %s is not IP address\n", addr);
> +                     continue;
> +             }
> +
> +             memset(name, 0, ACM_MAX_ADDRESS);
> +             memcpy(name, &ib_addr, sizeof(ib_addr));
> +             dest = acm_get_dest(ep, ACM_ADDRESS_GID, name);
> +             if (!dest) {
> +                     acm_log(0, "ERROR - IB GID %s not found in cache\n", 
> gid);
> +                     continue;
> +             }

I just noticed that there's no matching acm_put_dest() to this call.  I'll fix 
this up.

> +
> +             memset(name, 0, ACM_MAX_ADDRESS);
> +             if (addr_type == ACM_ADDRESS_IP)
> +                     memcpy(name, &ip_addr, 4);
> +             else
> +                     memcpy(name, &ip_addr, sizeof(ip_addr));
> +             new_dest = acm_acquire_dest(ep, addr_type, name);
> +             if (!new_dest) {
> +                     acm_log(0, "ERROR - unable to create dest %s\n", addr);
> +                     continue;
> +             }
> +             new_dest->path = dest->path;
> +             new_dest->remote_qpn = dest->remote_qpn;
> +             new_dest->addr_timeout = dest->addr_timeout;
> +             new_dest->route_timeout = dest->route_timeout;
> +             new_dest->state = dest->state;

Rather than skipping addresses that we can't lookup the GID for, I *think* it 
would work to simply add new_dest, but set the state to ACM_ADDR_RESOLVED.

I can quickly throw a patch together to this effect, but won't be able to test 
it immediately.

- Sean
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to