On Thu, 2019-05-16 at 08:37 -0300, Jason Gunthorpe wrote: > On Thu, May 16, 2019 at 02:28:40PM +0300, Kamal Heib wrote: > > On Thu, 2019-05-16 at 08:16 -0300, Jason Gunthorpe wrote: > > > On Thu, May 16, 2019 at 01:53:08PM +0300, Kamal Heib wrote: > > > > For RoCE ports the call for ib_modify_port is not meaningful, > > > > so > > > > simplify the providers of RoCE by return OK in ib_core. > > > > > > > > Signed-off-by: Kamal Heib <kamalheib1@xxxxxxxxx> > > > > drivers/infiniband/core/device.c | 23 ++++++----- > > > > drivers/infiniband/hw/hns/hns_roce_main.c | 7 ---- > > > > drivers/infiniband/hw/mlx4/main.c | 8 ---- > > > > drivers/infiniband/hw/mlx5/main.c | 6 --- > > > > drivers/infiniband/hw/ocrdma/ocrdma_main.c | 1 - > > > > drivers/infiniband/hw/ocrdma/ocrdma_verbs.c | 6 --- > > > > drivers/infiniband/hw/ocrdma/ocrdma_verbs.h | 2 - > > > > drivers/infiniband/hw/qedr/main.c | 1 - > > > > drivers/infiniband/hw/qedr/verbs.c | 6 --- > > > > drivers/infiniband/hw/qedr/verbs.h | 2 - > > > > .../infiniband/hw/vmw_pvrdma/pvrdma_main.c | 1 - > > > > .../infiniband/hw/vmw_pvrdma/pvrdma_verbs.c | 38 --------- > > > > ---- > > > > .../infiniband/hw/vmw_pvrdma/pvrdma_verbs.h | 2 - > > > > drivers/infiniband/sw/rxe/rxe_verbs.c | 18 --------- > > > > 14 files changed, 14 insertions(+), 107 deletions(-) > > > > > > We have more roce only drivers than this, why isn't everything > > > changed? > > > > > > Jason > > > > Not all of them implements modify_port(). > > Then why didn't we just delete modify port from the other drivers? > > Jason This patch is doing that for all roce drivers that implement modify port, unless you mean none-roce drivers? Thanks, Kamal
