[mlx5-next,0/5] Mellanox, mlx5 roce enable devlink parameter

Message

Saeed Mahameed Nov. 8, 2019, 11:45 p.m. UTC

Hi All,

Currently RoCE is always enabled by default in mlx5 and there is no option
for the user to disable it.

This change introduces new generic devlink param "enable_roce". 
User should set desired param value and reload the driver to get
desired configuration.

Two options for reload:
1) reload mlx5_ib driver.
2) reload via devlink, once the devlink mlx5 reload support [1]
lands in net-next branch, after this series is applied.

mlx5 devlink reload is not part of this patchset since it depends on
changes from net-next tree.

RoCE will still be enabled by default after this change.

In case of no objection this series will be applied to mlx5-next branch
and sent later as pull request to both rdma-next and net-next branches.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/saeed/linux.git/commit/?h=net-next-mlx5&id=0915d89b86e379b26457ba50656e9257193eaa60

Thanks,
Saeed.

---

Michael Guralnik (5):
  devlink: Add new "enable_roce" generic device param
  net/mlx5: Document flow_steering_mode devlink param
  net/mlx5: Handle "enable_roce" devlink param
  IB/mlx5: Rename profile and init methods
  IB/mlx5: Load profile according to RoCE enablement state

 .../device_drivers/mellanox/mlx5.rst          | 21 ++++++++++
 .../networking/devlink-params-mlx5.txt        | 17 ++++++++
 Documentation/networking/devlink-params.txt   |  4 ++
 drivers/infiniband/hw/mlx5/ib_rep.c           |  2 +-
 drivers/infiniband/hw/mlx5/ib_rep.h           |  2 +-
 drivers/infiniband/hw/mlx5/main.c             | 39 ++++++++++---------
 .../net/ethernet/mellanox/mlx5/core/devlink.c | 22 +++++++++++
 include/linux/mlx5/driver.h                   | 11 ++++++
 include/net/devlink.h                         |  4 ++
 net/core/devlink.c                            |  5 +++
 10 files changed, 106 insertions(+), 21 deletions(-)
 create mode 100644 Documentation/networking/devlink-params-mlx5.txt

Comments

Saeed Mahameed Nov. 11, 2019, 8:18 p.m. UTC | #1

On Fri, 2019-11-08 at 23:45 +0000, Saeed Mahameed wrote:
> Hi All,
> 
> Currently RoCE is always enabled by default in mlx5 and there is no
> option
> for the user to disable it.
> 
> This change introduces new generic devlink param "enable_roce". 
> User should set desired param value and reload the driver to get
> desired configuration.
> 
> Two options for reload:
> 1) reload mlx5_ib driver.
> 2) reload via devlink, once the devlink mlx5 reload support [1]
> lands in net-next branch, after this series is applied.
> 
> mlx5 devlink reload is not part of this patchset since it depends on
> changes from net-next tree.
> 
> RoCE will still be enabled by default after this change.
> 
> In case of no objection this series will be applied to mlx5-next
> branch
> and sent later as pull request to both rdma-next and net-next
> branches.
> 

Series applied to mlx5-next.

Thanks.