Commit cb57bb84 authored by Matan Barak's avatar Matan Barak Committed by Doug Ledford

IB/cm: Use the source GID index type

Previosuly, cm and cma modules supported only IB and RoCE v1 GID type.
In order to support multiple GID types, the gid_type is passed to
cm_init_av_by_path and stored in the path record.

The rdma cm client would use a default GID type that will be saved in
rdma_id_private.
Signed-off-by: default avatarMatan Barak <matanb@mellanox.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent b39ffa1d
...@@ -364,7 +364,7 @@ static int cm_init_av_by_path(struct ib_sa_path_rec *path, struct cm_av *av) ...@@ -364,7 +364,7 @@ static int cm_init_av_by_path(struct ib_sa_path_rec *path, struct cm_av *av)
read_lock_irqsave(&cm.device_lock, flags); read_lock_irqsave(&cm.device_lock, flags);
list_for_each_entry(cm_dev, &cm.device_list, list) { list_for_each_entry(cm_dev, &cm.device_list, list) {
if (!ib_find_cached_gid(cm_dev->ib_device, &path->sgid, if (!ib_find_cached_gid(cm_dev->ib_device, &path->sgid,
IB_GID_TYPE_IB, ndev, &p, NULL)) { path->gid_type, ndev, &p, NULL)) {
port = cm_dev->port[p-1]; port = cm_dev->port[p-1];
break; break;
} }
...@@ -1600,6 +1600,8 @@ static int cm_req_handler(struct cm_work *work) ...@@ -1600,6 +1600,8 @@ static int cm_req_handler(struct cm_work *work)
struct ib_cm_id *cm_id; struct ib_cm_id *cm_id;
struct cm_id_private *cm_id_priv, *listen_cm_id_priv; struct cm_id_private *cm_id_priv, *listen_cm_id_priv;
struct cm_req_msg *req_msg; struct cm_req_msg *req_msg;
union ib_gid gid;
struct ib_gid_attr gid_attr;
int ret; int ret;
req_msg = (struct cm_req_msg *)work->mad_recv_wc->recv_buf.mad; req_msg = (struct cm_req_msg *)work->mad_recv_wc->recv_buf.mad;
...@@ -1639,11 +1641,24 @@ static int cm_req_handler(struct cm_work *work) ...@@ -1639,11 +1641,24 @@ static int cm_req_handler(struct cm_work *work)
cm_format_paths_from_req(req_msg, &work->path[0], &work->path[1]); cm_format_paths_from_req(req_msg, &work->path[0], &work->path[1]);
memcpy(work->path[0].dmac, cm_id_priv->av.ah_attr.dmac, ETH_ALEN); memcpy(work->path[0].dmac, cm_id_priv->av.ah_attr.dmac, ETH_ALEN);
ret = cm_init_av_by_path(&work->path[0], &cm_id_priv->av); ret = ib_get_cached_gid(work->port->cm_dev->ib_device,
work->port->port_num,
cm_id_priv->av.ah_attr.grh.sgid_index,
&gid, &gid_attr);
if (!ret) {
if (gid_attr.ndev)
dev_put(gid_attr.ndev);
work->path[0].gid_type = gid_attr.gid_type;
ret = cm_init_av_by_path(&work->path[0], &cm_id_priv->av);
}
if (ret) { if (ret) {
ib_get_cached_gid(work->port->cm_dev->ib_device, int err = ib_get_cached_gid(work->port->cm_dev->ib_device,
work->port->port_num, 0, &work->path[0].sgid, work->port->port_num, 0,
NULL); &work->path[0].sgid,
&gid_attr);
if (!err && gid_attr.ndev)
dev_put(gid_attr.ndev);
work->path[0].gid_type = gid_attr.gid_type;
ib_send_cm_rej(cm_id, IB_CM_REJ_INVALID_GID, ib_send_cm_rej(cm_id, IB_CM_REJ_INVALID_GID,
&work->path[0].sgid, sizeof work->path[0].sgid, &work->path[0].sgid, sizeof work->path[0].sgid,
NULL, 0); NULL, 0);
......
...@@ -228,6 +228,7 @@ struct rdma_id_private { ...@@ -228,6 +228,7 @@ struct rdma_id_private {
u8 tos; u8 tos;
u8 reuseaddr; u8 reuseaddr;
u8 afonly; u8 afonly;
enum ib_gid_type gid_type;
}; };
struct cma_multicast { struct cma_multicast {
...@@ -2314,6 +2315,7 @@ static int cma_resolve_iboe_route(struct rdma_id_private *id_priv) ...@@ -2314,6 +2315,7 @@ static int cma_resolve_iboe_route(struct rdma_id_private *id_priv)
ndev = dev_get_by_index(&init_net, addr->dev_addr.bound_dev_if); ndev = dev_get_by_index(&init_net, addr->dev_addr.bound_dev_if);
route->path_rec->net = &init_net; route->path_rec->net = &init_net;
route->path_rec->ifindex = addr->dev_addr.bound_dev_if; route->path_rec->ifindex = addr->dev_addr.bound_dev_if;
route->path_rec->gid_type = id_priv->gid_type;
} }
if (!ndev) { if (!ndev) {
ret = -ENODEV; ret = -ENODEV;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment