linux-kernel - RE: [PATCH] IB/mlx5: avoid binding a new mpi unit to the same devices repeatedly

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]

Message-ID: <VI1PR0502MB30082FAAD0D3EB6ACA9C6DF1D1560@VI1PR0502MB3008.eurprd05.prod.outlook.com>
Date:   Mon, 23 Jul 2018 16:21:28 +0000
From:   Parav Pandit <parav@...lanox.com>
To:     Qing Huang <qing.huang@...cle.com>,
        Daniel Jurgens <danielj@...lanox.com>,
        Or Gerlitz <gerlitz.or@...il.com>
CC:     Linux Kernel <linux-kernel@...r.kernel.org>,
        RDMA mailing list <linux-rdma@...r.kernel.org>,
        Jason Gunthorpe <jgg@...pe.ca>,
        Doug Ledford <dledford@...hat.com>,
        Leon Romanovsky <leon@...nel.org>,
        "gerald.gibson@...cle.com" <gerald.gibson@...cle.com>
Subject: RE: [PATCH] IB/mlx5: avoid binding a new mpi unit to the same devices
 repeatedly

Hi Qing,


> -----Original Message-----
> From: Qing Huang [mailto:qing.huang@...cle.com]
> Sent: Monday, July 23, 2018 10:36 AM
> To: Daniel Jurgens <danielj@...lanox.com>; Or Gerlitz
> <gerlitz.or@...il.com>; Parav Pandit <parav@...lanox.com>
> Cc: Linux Kernel <linux-kernel@...r.kernel.org>; RDMA mailing list <linux-
> rdma@...r.kernel.org>; Jason Gunthorpe <jgg@...pe.ca>; Doug Ledford
> <dledford@...hat.com>; Leon Romanovsky <leon@...nel.org>;
> gerald.gibson@...cle.com
> Subject: Re: [PATCH] IB/mlx5: avoid binding a new mpi unit to the same
> devices repeatedly
> 
> 
> 
> On 7/15/2018 12:48 PM, Daniel Jurgens wrote:
> > On 7/14/2018 10:57 AM, Or Gerlitz wrote:
> >> On Sat, Jul 14, 2018 at 2:50 AM, Qing Huang <qing.huang@...cle.com>
> wrote:
> >>> When a CX5 device is configured in dual-port RoCE mode, after
> >>> creating many VFs against port 1, creating the same number of VFs
> >>> against port 2 will flood kernel/syslog with something like
> >>> "mlx5_*:mlx5_ib_bind_slave_port:4266:(pid 5269): port 2 already
> >>> affiliated."
> >>>
> >>> So basically, when traversing mlx5_ib_dev_list,
> >>> mlx5_ib_add_slave_port() shouldn't repeatedly attempt to bind the
> >>> new mpi data unit to every device on the list until it finds an unbound
> device.
> >> Daniel,
> >>
> >> What is mpi data unit?
> > It's a structure to keep track affiliated port info in dual port RoCE mode,
> mpi meaning multi-port info. Parav can review this it my absence, otherwise I
> can take a closer look when I return to the office.
> Hi Daniel/Parav,
> 
> Have you got a chance to review this patch? Thanks!
Didn't have chance yet.
Will do this week.