mbox series

[mlx5-next,0/4] Add support to multiple RDMA priorities for FDB rules

Message ID 20211201193621.9129-1-saeed@kernel.org (mailing list archive)
Headers show
Series Add support to multiple RDMA priorities for FDB rules | expand

Message

Saeed Mahameed Dec. 1, 2021, 7:36 p.m. UTC
From: Saeed Mahameed <saeedm@nvidia.com>

Currently, the driver ignores the user's priority for flow steering
rules in FDB namespace. Change it and create the rule in the right
priority.

It will allow to create FDB steering rules in up to 16 different
priorities.

Maor Gottlieb (4):
  net/mlx5: Separate FDB namespace
  net/mlx5: Refactor mlx5_get_flow_namespace
  net/mlx5: Create more priorities for FDB bypass namespace
  RDMA/mlx5: Add support to multiple priorities for FDB rules

 drivers/infiniband/hw/mlx5/fs.c               | 18 ++---
 drivers/infiniband/hw/mlx5/mlx5_ib.h          |  3 +-
 .../net/ethernet/mellanox/mlx5/core/fs_cmd.c  |  4 +-
 .../net/ethernet/mellanox/mlx5/core/fs_core.c | 76 +++++++++++++++----
 include/linux/mlx5/fs.h                       |  1 +
 5 files changed, 75 insertions(+), 27 deletions(-)

--
2.31.1

Comments

Jason Gunthorpe Dec. 7, 2021, 6:58 p.m. UTC | #1
On Wed, Dec 01, 2021 at 11:36:17AM -0800, Saeed Mahameed wrote:
> From: Saeed Mahameed <saeedm@nvidia.com>
> 
> Currently, the driver ignores the user's priority for flow steering
> rules in FDB namespace. Change it and create the rule in the right
> priority.
> 
> It will allow to create FDB steering rules in up to 16 different
> priorities.
> 
> Maor Gottlieb (4):
>   net/mlx5: Separate FDB namespace
>   net/mlx5: Refactor mlx5_get_flow_namespace
>   net/mlx5: Create more priorities for FDB bypass namespace
>   RDMA/mlx5: Add support to multiple priorities for FDB rules
> 
>  drivers/infiniband/hw/mlx5/fs.c               | 18 ++---
>  drivers/infiniband/hw/mlx5/mlx5_ib.h          |  3 +-
>  .../net/ethernet/mellanox/mlx5/core/fs_cmd.c  |  4 +-
>  .../net/ethernet/mellanox/mlx5/core/fs_core.c | 76 +++++++++++++++----
>  include/linux/mlx5/fs.h                       |  1 +
>  5 files changed, 75 insertions(+), 27 deletions(-)

Did you want this to go to the rdma tree? If so it seems fine, please
update the shared branch

Thanks,
Jason
Saeed Mahameed Dec. 14, 2021, 12:04 a.m. UTC | #2
On Tue, 2021-12-07 at 14:58 -0400, Jason Gunthorpe wrote:
> On Wed, Dec 01, 2021 at 11:36:17AM -0800, Saeed Mahameed wrote:
> > From: Saeed Mahameed <saeedm@nvidia.com>
> > 
> > Currently, the driver ignores the user's priority for flow steering
> > rules in FDB namespace. Change it and create the rule in the right
> > priority.
> > 
> > It will allow to create FDB steering rules in up to 16 different
> > priorities.
> > 
> > Maor Gottlieb (4):
> >   net/mlx5: Separate FDB namespace
> >   net/mlx5: Refactor mlx5_get_flow_namespace
> >   net/mlx5: Create more priorities for FDB bypass namespace
> >   RDMA/mlx5: Add support to multiple priorities for FDB rules
> > 
> >  drivers/infiniband/hw/mlx5/fs.c               | 18 ++---
> >  drivers/infiniband/hw/mlx5/mlx5_ib.h          |  3 +-
> >  .../net/ethernet/mellanox/mlx5/core/fs_cmd.c  |  4 +-
> >  .../net/ethernet/mellanox/mlx5/core/fs_core.c | 76
> > +++++++++++++++----
> >  include/linux/mlx5/fs.h                       |  1 +
> >  5 files changed, 75 insertions(+), 27 deletions(-)
> 
> Did you want this to go to the rdma tree? If so it seems fine, please
> update the shared branch
> 

Yes, applied.
Jason Gunthorpe Dec. 15, 2021, 12:31 p.m. UTC | #3
On Tue, Dec 14, 2021 at 12:04:09AM +0000, Saeed Mahameed wrote:
> On Tue, 2021-12-07 at 14:58 -0400, Jason Gunthorpe wrote:
> > On Wed, Dec 01, 2021 at 11:36:17AM -0800, Saeed Mahameed wrote:
> > > From: Saeed Mahameed <saeedm@nvidia.com>
> > > 
> > > Currently, the driver ignores the user's priority for flow steering
> > > rules in FDB namespace. Change it and create the rule in the right
> > > priority.
> > > 
> > > It will allow to create FDB steering rules in up to 16 different
> > > priorities.
> > > 
> > > Maor Gottlieb (4):
> > >   net/mlx5: Separate FDB namespace
> > >   net/mlx5: Refactor mlx5_get_flow_namespace
> > >   net/mlx5: Create more priorities for FDB bypass namespace
> > >   RDMA/mlx5: Add support to multiple priorities for FDB rules
> > > 
> > >  drivers/infiniband/hw/mlx5/fs.c               | 18 ++---
> > >  drivers/infiniband/hw/mlx5/mlx5_ib.h          |  3 +-
> > >  .../net/ethernet/mellanox/mlx5/core/fs_cmd.c  |  4 +-
> > >  .../net/ethernet/mellanox/mlx5/core/fs_core.c | 76
> > > +++++++++++++++----
> > >  include/linux/mlx5/fs.h                       |  1 +
> > >  5 files changed, 75 insertions(+), 27 deletions(-)
> > 
> > Did you want this to go to the rdma tree? If so it seems fine, please
> > update the shared branch
> > 
> 
> Yes, applied.

Okay, pulled, thanks

Jason