Commit 25baba21 authored by Kamal Heib's avatar Kamal Heib Committed by Jason Gunthorpe

RDMA/siw: Fix setting active_{speed, width} attributes

Make sure to set the active_{speed, width} attributes to avoid reporting
the same values regardless of the underlying device.

Fixes: 303ae1cd ("rdma/siw: application interface")
Link: https://lore.kernel.org/r/20200218095911.26614-1-kamalheib1@gmail.comSigned-off-by: default avatarKamal Heib <kamalheib1@gmail.com>
Tested-by: default avatarBernard Metzler <bmt@zurich.ibm.com>
Reviewed-by: default avatarBernard Metzler <bmt@zurich.ibm.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent 65a16620
...@@ -165,11 +165,12 @@ int siw_query_port(struct ib_device *base_dev, u8 port, ...@@ -165,11 +165,12 @@ int siw_query_port(struct ib_device *base_dev, u8 port,
struct ib_port_attr *attr) struct ib_port_attr *attr)
{ {
struct siw_device *sdev = to_siw_dev(base_dev); struct siw_device *sdev = to_siw_dev(base_dev);
int rv;
memset(attr, 0, sizeof(*attr)); memset(attr, 0, sizeof(*attr));
attr->active_speed = 2; rv = ib_get_eth_speed(base_dev, port, &attr->active_speed,
attr->active_width = 2; &attr->active_width);
attr->gid_tbl_len = 1; attr->gid_tbl_len = 1;
attr->max_msg_sz = -1; attr->max_msg_sz = -1;
attr->max_mtu = ib_mtu_int_to_enum(sdev->netdev->mtu); attr->max_mtu = ib_mtu_int_to_enum(sdev->netdev->mtu);
...@@ -192,7 +193,7 @@ int siw_query_port(struct ib_device *base_dev, u8 port, ...@@ -192,7 +193,7 @@ int siw_query_port(struct ib_device *base_dev, u8 port,
* attr->subnet_timeout = 0; * attr->subnet_timeout = 0;
* attr->init_type_repy = 0; * attr->init_type_repy = 0;
*/ */
return 0; return rv;
} }
int siw_get_port_immutable(struct ib_device *base_dev, u8 port, int siw_get_port_immutable(struct ib_device *base_dev, u8 port,
......
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