Whamcloud - gitweb
LU-9679 modules: Use LIST_HEAD for declaring list_heads
[fs/lustre-release.git] / lnet / klnds / socklnd / socklnd.c
index 3409264..6fd707c 100644 (file)
@@ -564,7 +564,7 @@ ksocknal_del_peer_locked(struct ksock_peer_ni *peer_ni, __u32 ip)
 static int
 ksocknal_del_peer(struct lnet_ni *ni, struct lnet_process_id id, __u32 ip)
 {
-       struct list_head zombies = LIST_HEAD_INIT(zombies);
+       LIST_HEAD(zombies);
        struct list_head *ptmp;
        struct list_head *pnxt;
        struct ksock_peer_ni *peer_ni;
@@ -1016,7 +1016,7 @@ ksocknal_create_conn(struct lnet_ni *ni, struct ksock_route *route,
                     struct socket *sock, int type)
 {
        rwlock_t *global_lock = &ksocknal_data.ksnd_global_lock;
-       struct list_head zombies = LIST_HEAD_INIT(zombies);
+       LIST_HEAD(zombies);
        struct lnet_process_id peerid;
        struct list_head *tmp;
        u64 incarnation;
@@ -1461,7 +1461,7 @@ ksocknal_close_conn_locked(struct ksock_conn *conn, int error)
                /* No more connections to this peer_ni */
 
                if (!list_empty(&peer_ni->ksnp_tx_queue)) {
-                               struct ksock_tx *tx;
+                       struct ksock_tx *tx;
 
                        LASSERT(conn->ksnc_proto == &ksocknal_protocol_v3x);
 
@@ -1491,8 +1491,7 @@ ksocknal_close_conn_locked(struct ksock_conn *conn, int error)
 
        spin_lock_bh(&ksocknal_data.ksnd_reaper_lock);
 
-       list_add_tail(&conn->ksnc_list,
-                     &ksocknal_data.ksnd_deathrow_conns);
+       list_add_tail(&conn->ksnc_list, &ksocknal_data.ksnd_deathrow_conns);
        wake_up(&ksocknal_data.ksnd_reaper_waitq);
 
        spin_unlock_bh(&ksocknal_data.ksnd_reaper_lock);
@@ -1521,8 +1520,8 @@ ksocknal_peer_failed(struct ksock_peer_ni *peer_ni)
        read_unlock(&ksocknal_data.ksnd_global_lock);
 
        if (notify)
-               lnet_notify(peer_ni->ksnp_ni, peer_ni->ksnp_id.nid, 0,
-                           last_alive);
+               lnet_notify(peer_ni->ksnp_ni, peer_ni->ksnp_id.nid,
+                           false, false, last_alive);
 }
 
 void
@@ -1531,7 +1530,7 @@ ksocknal_finalize_zcreq(struct ksock_conn *conn)
        struct ksock_peer_ni *peer_ni = conn->ksnc_peer;
        struct ksock_tx *tx;
        struct ksock_tx *tmp;
-       struct list_head zlist = LIST_HEAD_INIT(zlist);
+       LIST_HEAD(zlist);
 
        /* NB safe to finalize TXs because closing of socket will
         * abort all buffered data */
@@ -1781,7 +1780,7 @@ ksocknal_close_matching_conns(struct lnet_process_id id, __u32 ipaddr)
 }
 
 void
-ksocknal_notify(struct lnet_ni *ni, lnet_nid_t gw_nid, int alive)
+ksocknal_notify_gw_down(lnet_nid_t gw_nid)
 {
        /* The router is telling me she's been notified of a change in
         * gateway state....
@@ -1791,17 +1790,14 @@ ksocknal_notify(struct lnet_ni *ni, lnet_nid_t gw_nid, int alive)
                .pid    = LNET_PID_ANY,
        };
 
-        CDEBUG (D_NET, "gw %s %s\n", libcfs_nid2str(gw_nid),
-                alive ? "up" : "down");
+       CDEBUG(D_NET, "gw %s down\n", libcfs_nid2str(gw_nid));
 
-        if (!alive) {
-                /* If the gateway crashed, close all open connections... */
-                ksocknal_close_matching_conns (id, 0);
-                return;
-        }
+       /* If the gateway crashed, close all open connections... */
+       ksocknal_close_matching_conns(id, 0);
+       return;
 
-        /* ...otherwise do nothing.  We can only establish new connections
-         * if we have autroutes, and these connect on demand. */
+       /* We can only establish new connections
+        * if we have autroutes, and these connect on demand. */
 }
 
 void
@@ -1865,7 +1861,6 @@ ksocknal_query(struct lnet_ni *ni, lnet_nid_t nid, time64_t *when)
                 ksocknal_launch_all_connections_locked(peer_ni);
 
        write_unlock_bh(glock);
-        return;
 }
 
 static void
@@ -1939,7 +1934,7 @@ ksocknal_push(struct lnet_ni *ni, struct lnet_process_id id)
                        }
                        read_unlock(&ksocknal_data.ksnd_global_lock);
 
-                       if (i == 0) /* no match */
+                       if (i <= peer_off) /* no match */
                                break;
 
                        rc = 0;
@@ -2270,16 +2265,16 @@ ksocknal_base_shutdown(void)
               atomic_read (&libcfs_kmemory));
        LASSERT (ksocknal_data.ksnd_nnets == 0);
 
-        switch (ksocknal_data.ksnd_init) {
-        default:
-                LASSERT (0);
+       switch (ksocknal_data.ksnd_init) {
+       default:
+               LASSERT(0);
+               /* fallthrough */
 
-        case SOCKNAL_INIT_ALL:
-        case SOCKNAL_INIT_DATA:
-                LASSERT (ksocknal_data.ksnd_peers != NULL);
-                for (i = 0; i < ksocknal_data.ksnd_peer_hash_size; i++) {
+       case SOCKNAL_INIT_ALL:
+       case SOCKNAL_INIT_DATA:
+               LASSERT(ksocknal_data.ksnd_peers != NULL);
+               for (i = 0; i < ksocknal_data.ksnd_peer_hash_size; i++)
                        LASSERT(list_empty(&ksocknal_data.ksnd_peers[i]));
-                }
 
                LASSERT(list_empty(&ksocknal_data.ksnd_nets));
                LASSERT(list_empty(&ksocknal_data.ksnd_enomem_conns));
@@ -2473,49 +2468,49 @@ ksocknal_debug_peerhash(struct lnet_ni *ni)
 
        read_lock(&ksocknal_data.ksnd_global_lock);
 
-        for (i = 0; i < ksocknal_data.ksnd_peer_hash_size; i++) {
+       for (i = 0; i < ksocknal_data.ksnd_peer_hash_size; i++) {
                list_for_each(tmp, &ksocknal_data.ksnd_peers[i]) {
                        peer_ni = list_entry(tmp, struct ksock_peer_ni, ksnp_list);
 
-                        if (peer_ni->ksnp_ni == ni) break;
+                       if (peer_ni->ksnp_ni == ni)
+                               break;
 
-                        peer_ni = NULL;
-                }
-        }
+                       peer_ni = NULL;
+               }
+       }
 
         if (peer_ni != NULL) {
                struct ksock_route *route;
                struct ksock_conn  *conn;
 
-               CWARN ("Active peer_ni on shutdown: %s, ref %d, scnt %d, "
-                      "closing %d, accepting %d, err %d, zcookie %llu, "
-                      "txq %d, zc_req %d\n", libcfs_id2str(peer_ni->ksnp_id),
-                      atomic_read(&peer_ni->ksnp_refcount),
-                      peer_ni->ksnp_sharecount, peer_ni->ksnp_closing,
-                      peer_ni->ksnp_accepting, peer_ni->ksnp_error,
-                      peer_ni->ksnp_zc_next_cookie,
-                      !list_empty(&peer_ni->ksnp_tx_queue),
-                      !list_empty(&peer_ni->ksnp_zc_req_list));
+               CWARN("Active peer_ni on shutdown: %s, ref %d, "
+                     "closing %d, accepting %d, err %d, zcookie %llu, "
+                     "txq %d, zc_req %d\n", libcfs_id2str(peer_ni->ksnp_id),
+                     atomic_read(&peer_ni->ksnp_refcount),
+                     peer_ni->ksnp_closing,
+                     peer_ni->ksnp_accepting, peer_ni->ksnp_error,
+                     peer_ni->ksnp_zc_next_cookie,
+                     !list_empty(&peer_ni->ksnp_tx_queue),
+                     !list_empty(&peer_ni->ksnp_zc_req_list));
 
                list_for_each(tmp, &peer_ni->ksnp_routes) {
                        route = list_entry(tmp, struct ksock_route, ksnr_list);
-                       CWARN ("Route: ref %d, schd %d, conn %d, cnted %d, "
-                              "del %d\n", atomic_read(&route->ksnr_refcount),
-                              route->ksnr_scheduled, route->ksnr_connecting,
-                              route->ksnr_connected, route->ksnr_deleted);
+                       CWARN("Route: ref %d, schd %d, conn %d, cnted %d, "
+                             "del %d\n", atomic_read(&route->ksnr_refcount),
+                             route->ksnr_scheduled, route->ksnr_connecting,
+                             route->ksnr_connected, route->ksnr_deleted);
                }
 
                list_for_each(tmp, &peer_ni->ksnp_conns) {
                        conn = list_entry(tmp, struct ksock_conn, ksnc_list);
-                       CWARN ("Conn: ref %d, sref %d, t %d, c %d\n",
-                              atomic_read(&conn->ksnc_conn_refcount),
-                              atomic_read(&conn->ksnc_sock_refcount),
-                              conn->ksnc_type, conn->ksnc_closing);
+                       CWARN("Conn: ref %d, sref %d, t %d, c %d\n",
+                             atomic_read(&conn->ksnc_conn_refcount),
+                             atomic_read(&conn->ksnc_sock_refcount),
+                             conn->ksnc_type, conn->ksnc_closing);
                }
        }
 
        read_unlock(&ksocknal_data.ksnd_global_lock);
-       return;
 }
 
 void
@@ -2571,62 +2566,6 @@ ksocknal_shutdown(struct lnet_ni *ni)
 }
 
 static int
-ksocknal_enumerate_interfaces(struct ksock_net *net, char *iname)
-{
-       struct net_device *dev;
-
-       rtnl_lock();
-       for_each_netdev(&init_net, dev) {
-               /* The iname specified by an user land configuration can
-                * map to an ifa_label so always treat iname as an ifa_label.
-                * If iname is NULL then fall back to the net device name.
-                */
-               const char *name = iname ? iname : dev->name;
-               struct in_device *in_dev;
-
-               if (strcmp(dev->name, "lo") == 0) /* skip the loopback IF */
-                       continue;
-
-               if (!(dev_get_flags(dev) & IFF_UP)) {
-                       CWARN("Ignoring interface %s (down)\n", dev->name);
-                       continue;
-               }
-
-               in_dev = __in_dev_get_rtnl(dev);
-               if (!in_dev) {
-                       CWARN("Interface %s has no IPv4 status.\n", dev->name);
-                       continue;
-               }
-
-               for_ifa(in_dev)
-                       if (strcmp(name, ifa->ifa_label) == 0) {
-                               int idx = net->ksnn_ninterfaces;
-                               struct ksock_interface *ksi;
-
-                               if (idx >= ARRAY_SIZE(net->ksnn_interfaces)) {
-                                       rtnl_unlock();
-                                       return -E2BIG;
-                               }
-
-                               ksi = &net->ksnn_interfaces[idx];
-                               ksi->ksni_ipaddr = ntohl(ifa->ifa_local);
-                               ksi->ksni_netmask = ifa->ifa_mask;
-                               strlcpy(ksi->ksni_name,
-                                       name, sizeof(ksi->ksni_name));
-                               net->ksnn_ninterfaces++;
-                               break;
-                       }
-               endfor_ifa(in_dev);
-        }
-       rtnl_unlock();
-
-       if (net->ksnn_ninterfaces == 0)
-                CERROR("Can't find any usable interfaces\n");
-
-       return net->ksnn_ninterfaces > 0 ? 0 : -ENOENT;
-}
-
-static int
 ksocknal_search_new_ipif(struct ksock_net *net)
 {
        int new_ipif = 0;
@@ -2745,10 +2684,10 @@ ksocknal_startup(struct lnet_ni *ni)
 {
        struct ksock_net *net;
        struct lnet_ioctl_config_lnd_cmn_tunables *net_tunables;
+       struct ksock_interface *ksi = NULL;
+       struct lnet_inetdev *ifaces = NULL;
+       int i = 0;
        int rc;
-       int i;
-       struct net_device *net_dev;
-       int node_id;
 
         LASSERT (ni->ni_net->net_lnd == &the_ksocklnd);
 
@@ -2788,10 +2727,20 @@ ksocknal_startup(struct lnet_ni *ni)
                net_tunables->lct_peer_rtr_credits =
                        *ksocknal_tunables.ksnd_peerrtrcredits;
 
+       rc = lnet_inet_enumerate(&ifaces, ni->ni_net_ns);
+       if (rc < 0)
+               goto fail_1;
+
        if (!ni->ni_interfaces[0]) {
-               rc = ksocknal_enumerate_interfaces(net, NULL);
-               if (rc < 0)
-                       goto fail_1;
+               ksi = &net->ksnn_interfaces[0];
+
+               /* Use the first discovered interface */
+               net->ksnn_ninterfaces = 1;
+               ni->ni_dev_cpt = ifaces[0].li_cpt;
+               ksi->ksni_ipaddr = ifaces[0].li_ipaddr;
+               ksi->ksni_netmask = ifaces[0].li_netmask;
+               strlcpy(ksi->ksni_name, ifaces[0].li_name,
+                       sizeof(ksi->ksni_name));
        } else {
                /* Before Multi-Rail ksocklnd would manage
                 * multiple interfaces with its own tcp bonding.
@@ -2809,34 +2758,38 @@ ksocknal_startup(struct lnet_ni *ni)
                        if (!ni->ni_interfaces[i])
                                break;
 
-                       for (j = 0; j < net->ksnn_ninterfaces;  j++) {
-                               struct ksock_interface *ksi;
-
-                               ksi = &net->ksnn_interfaces[j];
-
-                               if (strcmp(ni->ni_interfaces[i],
-                                          ksi->ksni_name) == 0) {
-                                       CERROR("found duplicate %s\n",
-                                              ksi->ksni_name);
+                       for (j = 0; j < LNET_INTERFACES_NUM;  j++) {
+                               if (i != j && ni->ni_interfaces[j] &&
+                                   strcmp(ni->ni_interfaces[i],
+                                          ni->ni_interfaces[j]) == 0) {
                                        rc = -EEXIST;
+                                       CERROR("ksocklnd: found duplicate %s at %d and %d, rc = %d\n",
+                                              ni->ni_interfaces[i], i, j, rc);
                                        goto fail_1;
                                }
                        }
 
-                       rc = ksocknal_enumerate_interfaces(net, ni->ni_interfaces[i]);
-                       if (rc < 0)
-                               goto fail_1;
-               }
-       }
+                       for (j = 0; j < rc; j++) {
+                               if (strcmp(ifaces[j].li_name,
+                                          ni->ni_interfaces[i]) != 0)
+                                       continue;
 
-       net_dev = dev_get_by_name(&init_net,
-                                 net->ksnn_interfaces[0].ksni_name);
-       if (net_dev != NULL) {
-               node_id = dev_to_node(&net_dev->dev);
-               ni->ni_dev_cpt = cfs_cpt_of_node(lnet_cpt_table(), node_id);
-               dev_put(net_dev);
-       } else {
-               ni->ni_dev_cpt = CFS_CPT_ANY;
+                               ksi = &net->ksnn_interfaces[j];
+                               ni->ni_dev_cpt = ifaces[j].li_cpt;
+                               ksi->ksni_ipaddr = ifaces[j].li_ipaddr;
+                               ksi->ksni_netmask = ifaces[j].li_netmask;
+                               strlcpy(ksi->ksni_name, ifaces[j].li_name,
+                                       sizeof(ksi->ksni_name));
+                               net->ksnn_ninterfaces++;
+                               break;
+                       }
+               }
+               /* ni_interfaces don't map to all network interfaces */
+               if (!ksi || net->ksnn_ninterfaces != i) {
+                       CERROR("ksocklnd: requested %d but only %d interfaces found\n",
+                              i, net->ksnn_ninterfaces);
+                       goto fail_1;
+               }
        }
 
        /* call it before add it to ksocknal_data.ksnd_nets */
@@ -2844,8 +2797,8 @@ ksocknal_startup(struct lnet_ni *ni)
        if (rc != 0)
                goto fail_1;
 
-       ni->ni_nid = LNET_MKNID(LNET_NIDNET(ni->ni_nid),
-                               net->ksnn_interfaces[0].ksni_ipaddr);
+       LASSERT(ksi);
+       ni->ni_nid = LNET_MKNID(LNET_NIDNET(ni->ni_nid), ksi->ksni_ipaddr);
        list_add(&net->ksnn_list, &ksocknal_data.ksnd_nets);
 
         ksocknal_data.ksnd_nnets++;
@@ -2882,7 +2835,7 @@ static int __init ksocklnd_init(void)
        the_ksocklnd.lnd_ctl      = ksocknal_ctl;
        the_ksocklnd.lnd_send     = ksocknal_send;
        the_ksocklnd.lnd_recv     = ksocknal_recv;
-       the_ksocklnd.lnd_notify   = ksocknal_notify;
+       the_ksocklnd.lnd_notify_peer_down   = ksocknal_notify_gw_down;
        the_ksocklnd.lnd_query    = ksocknal_query;
        the_ksocklnd.lnd_accept   = ksocknal_accept;