]> git.openfabrics.org - ~shefty/rdma-dev.git/commitdiff
rdma/cm: restrict AF_IB loopback to binding to IB devices only
authorSean Hefty <sean.hefty@intel.com>
Wed, 7 Apr 2010 00:22:32 +0000 (17:22 -0700)
committerSean Hefty <sean.hefty@intel.com>
Wed, 7 Apr 2010 00:22:32 +0000 (17:22 -0700)
If a user specifies AF_IB as the source address for a loopback
connection, limit the resolution to IB devices only.

Signed-off-by: Sean Hefty <sean.hefty@intel.com>
drivers/infiniband/core/cma.c

index 304c69e28ebfc9c9b4557231a83c0383937a2c4c..9d83eaa89ede382008eb941f9799fca7d563f97b 100644 (file)
@@ -1802,26 +1802,40 @@ static void cma_set_loopback(struct sockaddr *addr)
 
 static int cma_bind_loopback(struct rdma_id_private *id_priv)
 {
-       struct cma_device *cma_dev;
+       struct cma_device *cma_dev, *cur_dev;
+       struct sockaddr *addr;
        struct ib_port_attr port_attr;
        union ib_gid gid;
        u16 pkey;
        int ret;
        u8 p;
 
+       cma_dev = NULL;
+       addr = (struct sockaddr *) &id_priv->id.route.addr.src_addr;
        mutex_lock(&lock);
-       if (list_empty(&dev_list)) {
+       list_for_each_entry(cur_dev, &dev_list, list) {
+               if (addr->sa_family == AF_IB &&
+                   rdma_node_get_transport(cur_dev->device->node_type) != RDMA_TRANSPORT_IB)
+                       continue;
+
+               if (!cma_dev)
+                       cma_dev = cur_dev;
+
+               for (p = 1; p <= cur_dev->device->phys_port_cnt; ++p) {
+                       if (!ib_query_port(cur_dev->device, p, &port_attr) &&
+                           port_attr.state == IB_PORT_ACTIVE) {
+                               cma_dev = cur_dev;
+                               goto port_found;
+                       }
+               }
+       }
+
+       if (!cma_dev) {
                ret = -ENODEV;
                goto out;
        }
-       list_for_each_entry(cma_dev, &dev_list, list)
-               for (p = 1; p <= cma_dev->device->phys_port_cnt; ++p)
-                       if (!ib_query_port(cma_dev->device, p, &port_attr) &&
-                           port_attr.state == IB_PORT_ACTIVE)
-                               goto port_found;
 
        p = 1;
-       cma_dev = list_entry(dev_list.next, struct cma_device, list);
 
 port_found:
        ret = ib_get_cached_gid(cma_dev->device, p, 0, &gid);