On 2/19/2020 6:01 AM, Erez Shitrit wrote:
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c index e5f438a..5c1cf68 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c @@ -1862,7 +1862,10 @@ static int ipoib_parent_init(struct net_device *ndev) priv->port); return result; } - priv->max_ib_mtu = ib_mtu_enum_to_int(attr.max_mtu); + if (rdma_core_cap_opa_port(priv->ca, priv->port))Why not to use something like you did in rdma_mtu_enum_to_int, in order to hide all the "_opa_" stuff from IPoIB ?
Sounds reasonable, we can do that in the v2. -Denny
