On Thu, Apr 01, 2021 at 11:42:41AM +0300, Leon Romanovsky wrote: > On Mon, Mar 29, 2021 at 09:54:12AM -0400, dennis.dalessandro@xxxxxxxxxxxxxxxxxxxx wrote: > > From: Kaike Wan <kaike.wan@xxxxxxxxx> > > > > This is a follow on patch to add a phys_mtu field to the > > ib_port_attr structure to indicate the maximum physical MTU > > the underlying device supports. > > > > Extends the following: > > commit 6d72344cf6c4 ("IB/ipoib: Increase ipoib Datagram mode MTU's upper limit") > > > > Reviewed-by: Mike Marciniszyn <mike.marciniszyn@xxxxxxxxxxxxxxxxxxxx> > > Signed-off-by: Kaike Wan <kaike.wan@xxxxxxxxx> > > Signed-off-by: Dennis Dalessandro <dennis.dalessandro@xxxxxxxxxxxxxxxxxxxx> > > drivers/infiniband/hw/bnxt_re/ib_verbs.c | 1 + > > drivers/infiniband/hw/cxgb4/provider.c | 1 + > > drivers/infiniband/hw/efa/efa_verbs.c | 1 + > > drivers/infiniband/hw/hns/hns_roce_main.c | 1 + > > drivers/infiniband/hw/i40iw/i40iw_verbs.c | 1 + > > drivers/infiniband/hw/mlx4/main.c | 1 + > > drivers/infiniband/hw/mlx5/mad.c | 1 + > > drivers/infiniband/hw/mlx5/main.c | 2 ++ > > drivers/infiniband/hw/mthca/mthca_provider.c | 1 + > > drivers/infiniband/hw/ocrdma/ocrdma_verbs.c | 1 + > > drivers/infiniband/hw/qib/qib_verbs.c | 1 + > > drivers/infiniband/hw/usnic/usnic_ib_verbs.c | 1 + > > drivers/infiniband/hw/vmw_pvrdma/pvrdma_verbs.c | 1 + > > drivers/infiniband/sw/siw/siw_verbs.c | 1 + > > drivers/infiniband/ulp/ipoib/ipoib_main.c | 2 +- > > include/rdma/ib_verbs.h | 17 ----------------- > > 16 files changed, 16 insertions(+), 18 deletions(-) > > But why? What will it give us that almost all drivers have same > props->phys_mtu = ib_mtu_enum_to_int(props->max_mtu); line? Dennis? Jason