netdev - Re: [PATCH mlx5-next 0/4] Add support to multiple RDMA priorities for FDB rules

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 14 Dec 2021 00:04:09 +0000
From:   Saeed Mahameed <saeedm@...dia.com>
To:     Jason Gunthorpe <jgg@...dia.com>
CC:     Leon Romanovsky <leonro@...dia.com>,
        "kuba@...nel.org" <kuba@...nel.org>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: Re: [PATCH mlx5-next 0/4] Add support to multiple RDMA priorities for
 FDB rules

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@...dia.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.