>I added in the following diff as part of applying this to get RDS to
>build.  It looks like this is all that's needed to fix the build?

Apparently I don't build RDS.  Yes, this looks right to me.  I'm assuming that
these calls are made only for established connections, where dev_addr is set
correctly.

>diff --git a/net/rds/iw.c b/net/rds/iw.c
>index db224f7..b28fa85 100644
>--- a/net/rds/iw.c
>+++ b/net/rds/iw.c
>@@ -184,8 +184,8 @@ static int rds_iw_conn_info_visitor(struct rds_connection
>*conn,
>               ic = conn->c_transport_data;
>               dev_addr = &ic->i_cm_id->route.addr.dev_addr;
>
>-              ib_addr_get_sgid(dev_addr, (union ib_gid *) &iinfo->src_gid);
>-              ib_addr_get_dgid(dev_addr, (union ib_gid *) &iinfo->dst_gid);
>+              rdma_addr_get_sgid(dev_addr, (union ib_gid *) &iinfo->src_gid);
>+              rdma_addr_get_dgid(dev_addr, (union ib_gid *) &iinfo->dst_gid);

This patch has the side effect of actually returning the correct 'GID' for an
iWarp device.  ib_addr_get_sgid should have been iw_addr_get_sgid here.

- 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